github/azure-sdk-for-cpp
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

AMQP: do not use scope.MinimumExpiration when getting SAS Token (#5376)

Browse Source 
* AMQP: do not use scope.MinimumExpiration when getting SAS Token

* Remove unused parameters

* Match style

---------

Co-authored-by: Anton Kolesnyk <antkmsft@users.noreply.github.com>
This commit is contained in:
Anton Kolesnyk 2024-02-23 15:11:40 -08:00 committed by GitHub
parent 300e518eed
commit 00b30e3d86
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 3 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
sdk/core/azure-core-amqp/src/amqp/connection.cpp
View File

@ -661,11 +661,7 @@ namespace Azure { namespace Core { namespace Amqp { namespace _detail {
try
{
Credentials::TokenRequestContext requestContext;
bool isSasToken = IsSasCredential();
if (isSasToken)
{
requestContext.MinimumExpiration = std::chrono::minutes(60);
}
requestContext.Scopes = m_options.AuthenticationScopes;
auto accessToken{GetCredential()->GetToken(requestContext, context)};

3
sdk/core/azure-core-amqp/src/amqp/connection_string_credential.cpp
View File

@ -129,8 +129,9 @@ namespace Azure { namespace Core { namespace Amqp { namespace _internal {
Context const& context) const
{
Credentials::AccessToken rv;
rv.ExpiresOn = Azure::DateTime::clock::now() + tokenRequestContext.MinimumExpiration;
rv.ExpiresOn = Azure::DateTime::clock::now() + std::chrono::minutes(60);
rv.Token = GenerateSasToken(static_cast<std::chrono::system_clock::time_point>(rv.ExpiresOn));
(void)tokenRequestContext;
(void)context;
return rv;
}