Skip to content

S3 take sign config#483

Merged
TingDaoK merged 12 commits intomainfrom
s3-take-sign-config
Jul 10, 2023
Merged

S3 take sign config#483
TingDaoK merged 12 commits intomainfrom
s3-take-sign-config

Conversation

@TingDaoK
Copy link
Copy Markdown
Contributor

@TingDaoK TingDaoK commented Jun 16, 2023

  • S3 configurations now takes signing config for more control of the signing process.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@jmklix jmklix added the needs-review This issue or pull request needs review from a core team member. label Jul 6, 2023
Comment thread awscrt/s3.py Outdated
Comment thread awscrt/s3.py
Comment thread awscrt/s3.py
Comment thread awscrt/s3.py Outdated
Comment thread awscrt/s3.py Outdated
Comment thread awscrt/s3.py Outdated
Comment thread awscrt/s3.py Outdated
Comment thread awscrt/s3.py Outdated
credential_provider (Optional[AwsCredentialsProvider]): Credentials providers source the
:class:`~awscrt.auth.AwsCredentials` needed to sign an authenticated AWS request.
signing_config (Optional[AwsSigningConfig]):
Configuration for signing of the client. Use `create_default_s3_signing_config` to create the default config.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

trivial: do a link

Suggested change
Configuration for signing of the client. Use `create_default_s3_signing_config` to create the default config.
Configuration for signing of the client. Use :func:`create_default_s3_signing_config()` to create the default config.

you can check how the docs look locally by running ./scripts/make-docs.py

Comment thread awscrt/s3.py Outdated
:class:`~awscrt.auth.AwsCredentials` needed to sign an authenticated AWS request, for this request only.
If None is provided, the credential provider in the client will be used.
signing_config (Optional[AwsSigningConfig]):
Configuration for signing of the request to override the configuration from client. Use `create_default_s3_signing_config` to create the default config.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

same about a link

@TingDaoK TingDaoK enabled auto-merge (squash) July 10, 2023 19:07
@TingDaoK TingDaoK merged commit 91d3dce into main Jul 10, 2023
@TingDaoK TingDaoK deleted the s3-take-sign-config branch July 10, 2023 19:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-review This issue or pull request needs review from a core team member.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants