-
-
Notifications
You must be signed in to change notification settings - Fork 36
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
feat: allow create SAS token for azure storage #306
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mostly LGTM!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mostly LGTM!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks a lot!
pub fn sign_query( | ||
&self, | ||
req: &mut impl SignableRequest, | ||
expire: Duration, | ||
cred: &Credential, | ||
) -> Result<()> { | ||
let ctx = self.build(req, SigningMethod::Query(expire), cred)?; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Xuanwo Please note that we have modified the signature of this function, and we need to pay attention to compatibility when releasing the version.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! We should release v0.10 instead!
close #301
This PR is support azure SAS token(account SAS), we will try to generate SAS token when call
sign_query
without settingSharedAccessSignature(token)
in Credential.