Skip to content
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

Update docs for sts auth #2229

Merged
merged 4 commits into from
Mar 12, 2025
Merged

Update docs for sts auth #2229

merged 4 commits into from
Mar 12, 2025

Conversation

phoebusm
Copy link
Collaborator

Reference Issues/PRs

https://man312219.monday.com/boards/7852509418/pulses/8615374613

What does this implement or fix?

Update STS auth docs

Any other comments?

Checklist

Checklist for code changes...
  • Have you updated the relevant docstrings, documentation and copyright notice?
  • Is this contribution tested against all ArcticDB's features?
  • Do all exceptions introduced raise appropriate error messages?
  • Are API changes highlighted in the PR description?
  • Is the PR labelled as enhancement or bug so it appears in autogenerated release notes?

@phoebusm phoebusm added the documentation Improvements or additions to documentation label Mar 10, 2025
Copy link
Collaborator

@maxim-morozov maxim-morozov left a comment

Choose a reason for hiding this comment

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

You would need to resolve the conflict as there is a new version of the aws.md. See with the newer version where this section fits better

### Custom CA cert support

Due to a [bug](https://github.com/aws/aws-sdk-cpp/issues/2920) in AWS C++ SDK, STS authentication users on below OS are required to turn off [S3Storage.VerifySSL](https://docs.arcticdb.io/latest/runtime_config/#s3storageverifyssl):
Copy link
Collaborator

Choose a reason for hiding this comment

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

let's not use bug. Due to a known issue....

@phoebusm phoebusm force-pushed the docs/update_sts_docs branch from cb15937 to b9f2328 Compare March 11, 2025 11:31
@phoebusm phoebusm added the minor Feature change, should increase minor version label Mar 12, 2025
@phoebusm phoebusm merged commit 3e40f14 into master Mar 12, 2025
147 checks passed
@phoebusm phoebusm deleted the docs/update_sts_docs branch March 12, 2025 15:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation minor Feature change, should increase minor version
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants