Skip to content

Allow querystring_auth to be overridden on a per-call basis#1536

Open
brianhelba wants to merge 1 commit into
jschneier:masterfrom
brianhelba:s3-url-querystring-auth
Open

Allow querystring_auth to be overridden on a per-call basis#1536
brianhelba wants to merge 1 commit into
jschneier:masterfrom
brianhelba:s3-url-querystring-auth

Conversation

@brianhelba

Copy link
Copy Markdown
Contributor

This preserves the existing behavior by default, but adds the ability for individual calls to S3Storage.url to specify a different value of querystring_auth.

This supports the use case of buckets with mixed public and private content.

This preserves the existing behavior by default, but adds the
ability for individual calls to `S3Storage.url` to specify
a different value of `querystring_auth`.

This supports the use case of buckets with mixed public and
private content.
@brianhelba

Copy link
Copy Markdown
Contributor Author

I'm not sure how to write a useful test for this. The existing test_url_unsigned would be the appropriate place to add some additional testing, but the current test structure doesn't really assert anything useful about whether signing actually occurred.

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.

1 participant