microsoft / azure-pipelines-extensions

Collection of all RM and deployment extensions
http://www.visualstudio.com/explore/release-management-vs
MIT License
278 stars 422 forks source link

Add-SslCert check fails and tries to re-add ssl #1039

Open matbutler opened 2 years ago

matbutler commented 2 years ago

It appears to check $result.Get(4) for the Hostname:port, but its $result.Get(6) for me.

Windows 2022 IIS Version 10.0.20348.1

jiggybyte commented 2 years ago

Hitting the same thing, it's because there are some extra lines of blank space when it calls the show sslcert....can we get a fix for this? using Get(7) instead of Get(5) would solve it.

jabteles commented 1 year ago

+1 preventing to migrate to Server 2022. No more workarounds.

v-schhabra commented 8 months ago

Hi @jabteles, @jiggybyte The mentioned issue has been fixed in this PR https://github.com/microsoft/azure-pipelines-extensions/pull/1220 Could you please use the IISWebAppManagementV3 version of the task and test if it is working fine?

jabteles commented 8 months ago

Hi @jabteles, @jiggybyte The mentioned issue has been fixed in this PR #1220 Could you please use the IISWebAppManagementV3 version of the task and test if it is working fine?

I updated the extension on my Azure DevOps and still get the same error. I've confirmed it is using correct version (3.2.0)