Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

webapp: ssl bind fails due to \r in thumbprint #16090

Closed
dev-playbook opened this issue Nov 30, 2020 · 4 comments
Closed

webapp: ssl bind fails due to \r in thumbprint #16090

dev-playbook opened this issue Nov 30, 2020 · 4 comments
Assignees
Labels

Comments

@dev-playbook
Copy link

When I did this....

wwwSslThumbprint=$(az webapp config ssl list --query "[?name=='www.$customRootDomain'].thumbprint" --output tsv)

echo $wwwSslThumbprint

az webapp config ssl bind --certificate-thumbprint $wwwSslThumbprint --ssl-type SNI --verbose

I get this error. 'not found. for thumbprint'{thumbprint}

But if when I did this...

wwwSslThumbprint=$(az webapp config ssl list --query "[?name=='www.$customRootDomain'].thumbprint" --output tsv | sed "s/\r//g")

echo $wwwSslThumbprint

az webapp config ssl bind --certificate-thumbprint $wwwSslThumbprint --ssl-type SNI --verbose

It works!

Note that I've had to remove the carriage return in the extracted thumbprint by using sed "s/\r//g".

az version info

{
  "azure-cli": "2.15.1",
  "azure-cli-core": "2.15.1",    
  "azure-cli-telemetry": "1.0.6",
  "extensions": {}
}

Microsoft Windows [Version 10.0.18363.1198]

Ubuntu wsl version: 2


Document Details

Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

@ghost ghost added needs-triage This is a new issue that needs to be triaged to the appropriate team. question The issue doesn't require a change to the product in order to be resolved. Most issues start as that labels Nov 30, 2020
@yungezz yungezz added Service Attention This issue is responsible by Azure service team. Web Apps az webapp labels Dec 1, 2020
@ghost ghost removed the needs-triage This is a new issue that needs to be triaged to the appropriate team. label Dec 1, 2020
@ghost
Copy link

ghost commented Dec 1, 2020

Thanks for the feedback! We are routing this to the appropriate team for follow-up. cc @AzureAppServiceCLI, @antcp.

Issue Details

When I did this....

wwwSslThumbprint=$(az webapp config ssl list --query "[?name=='www.$customRootDomain'].thumbprint" --output tsv)

echo $wwwSslThumbprint

az webapp config ssl bind --certificate-thumbprint $wwwSslThumbprint --ssl-type SNI --verbose

I get this error. 'not found. for thumbprint'{thumbprint}

But if when I did this...

wwwSslThumbprint=$(az webapp config ssl list --query "[?name=='www.$customRootDomain'].thumbprint" --output tsv | sed "s/\r//g")

echo $wwwSslThumbprint

az webapp config ssl bind --certificate-thumbprint $wwwSslThumbprint --ssl-type SNI --verbose

It works!

Note that I've had to remove the carriage return in the extracted thumbprint by using sed "s/\r//g".

az version info

{
  "azure-cli": "2.15.1",
  "azure-cli-core": "2.15.1",    
  "azure-cli-telemetry": "1.0.6",
  "extensions": {}
}

Microsoft Windows [Version 10.0.18363.1198]

Ubuntu wsl version: 2


Document Details

Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

Author: dev-playbook
Assignees: -
Labels:

Service Attention, Web Apps, needs-triage, question

Milestone: -

@yungezz
Copy link
Member

yungezz commented Dec 1, 2020

route to appropriate team

@btardif btardif added this to the S180 milestone Dec 2, 2020
@btardif
Copy link
Member

btardif commented Dec 2, 2020

@panchagnula looks like we need to trim trailing carriage return on the thumbprint

@panchagnula panchagnula changed the title ssl bind fails due to \r in thumbprint webapp: ssl bind fails due to \r in thumbprint Dec 2, 2020
@yonzhan yonzhan modified the milestones: S180, S182 Dec 26, 2020
@yonzhan yonzhan removed this from the S182 milestone Feb 6, 2021
@jiasli
Copy link
Member

jiasli commented Jul 29, 2021

Please see the explanation at #13573 (comment).

@jiasli jiasli closed this as completed Jul 29, 2021
@jiasli jiasli added this to the Jul 2021 (2021-08-03) milestone Jul 29, 2021
@jiasli jiasli assigned jiasli and unassigned ThejaChoudary Jul 29, 2021
@jiasli jiasli added WSL and removed Service Attention This issue is responsible by Azure service team. Web Apps az webapp question The issue doesn't require a change to the product in order to be resolved. Most issues start as that labels Jul 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

7 participants