-
Notifications
You must be signed in to change notification settings - Fork 5.1k
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
[Web App] Password is not a required property when creating Certificates #5029
Comments
Thanks for the feedback! We are routing this to the appropriate team for follow-up. cc @AzureAppServiceCLI, @antcp. |
any update on this? |
Anyone looking at this? it's blocking a fair amount of downstream applications by the looks of it. |
@ArcturusZhang is anything happening on this? |
@ArcturusZhang, ping? |
For now you please use empty string for password this is managed certificates. We plan to update the swagger to make this optional, but don't have an ETA to share at this time. |
We just update our nuget to 2.0.1 (api-version=2018-02-01) from 1.7.1-preview
https://www.nuget.org/packages/Microsoft.Azure.Management.WebSites/2.0.1
and it seems we have made the “Password” property a must in the new api-version.
azure-rest-api-specs/specification/web/resource-manager/Microsoft.Web/stable/2018-02-01/Certificates.json
Line 308 in 67edb3f
Does it reflect the correct set of validation on the Service side, as the same API was passing on nuget 1.7.1-preview? Or is it a bug in the rest Spec, which is throwing the below exception now
"ExceptionData": "Microsoft.Rest.ValidationException: 'Password' cannot be null.\r\n
at Microsoft.Azure.Management.WebSites.Models.Certificate.Validate()\r\n
at Microsoft.Azure.Management.WebSites.CertificatesOperations.d__8.MoveNext()\r\n
The text was updated successfully, but these errors were encountered: