Skip to content

Fix/presigned urls#157

Merged
lfbrehm merged 2 commits intomainfrom
fix/presigned_urls
Apr 17, 2024
Merged

Fix/presigned urls#157
lfbrehm merged 2 commits intomainfrom
fix/presigned_urls

Conversation

@lfbrehm
Copy link
Copy Markdown
Member

@lfbrehm lfbrehm commented Apr 17, 2024

Summary

Fixes presigned urls with aruna tokens

Description

When creating a presigned url, the token id returned by the authorization was not shared with the underlying get_credentials function in get_presigend_upload/get_presigned_download

@lfbrehm lfbrehm requested review from St4NNi and das-Abroxas April 17, 2024 11:53
Copy link
Copy Markdown
Contributor

@das-Abroxas das-Abroxas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm 👍

@lfbrehm lfbrehm merged commit 5c41040 into main Apr 17, 2024
@lfbrehm lfbrehm deleted the fix/presigned_urls branch April 17, 2024 12:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants