-
Notifications
You must be signed in to change notification settings - Fork 911
Adding checks during SAS creation #20379
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
Conversation
souravgupta-msft
commented
Mar 9, 2023
- The purpose of this PR is explained in this or a referenced issue.
- The PR does not update generated files.
- These files are managed by the codegen framework at Azure/autorest.go.
- Tests are included and/or updated for code changes.
- Updates to CHANGELOG.md are included.
- MIT license headers are included in each file.
Hi @souravgupta-msft,
When the SAS should be associated with stored access policy the documentation says expiry and permissions must be omitted if those information are specified in the associated stored access policy. Currently this change leads to an error during signing which has worked previously. My code looks like
Anything that I have missed that I need to do? |