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

feat(fs-aws): add credential config for public and requester pays #1409

Merged
merged 2 commits into from
Mar 4, 2024

Conversation

blacha
Copy link
Owner

@blacha blacha commented Mar 4, 2024

Motivation

Requester pays and public buckets are pretty common ways of accessing s3 locations, access needs to be configured so that a bucket can be accessed

with either:

  • "assume this role then use requester pays"
  • "use the current role but as requester pays"

or as a public bucket with no credentials.

Modification

allow credentials to be defined as requester pays or public

@blacha blacha marked this pull request as ready for review March 4, 2024 23:06
@blacha blacha merged commit 0386b61 into master Mar 4, 2024
4 checks passed
@blacha blacha deleted the feat/public-bucket branch March 4, 2024 23:12
@github-actions github-actions bot mentioned this pull request Mar 4, 2024
blacha pushed a commit that referenced this pull request Nov 4, 2024
🤖 I have created a release *beep* *boop*
---


<details><summary>fs-aws: 11.3.0</summary>

##
[11.3.0](fs-aws-v11.2.2...fs-aws-v11.3.0)
(2024-11-03)


### Features

* **fs-aws:** add callback when credentials are dynamically loaded
([29a1e3c](29a1e3c))
* **fs-aws:** add credential config for public and requester pays
([#1409](#1409))
([0386b61](0386b61))


### Dependencies

* The following workspace dependencies were updated
  * dependencies
    * @chunkd/source-aws bumped from 11.0.3 to 11.0.4
</details>

<details><summary>source-aws: 11.0.4</summary>

##
[11.0.4](source-aws-v11.0.3...source-aws-v11.0.4)
(2024-11-03)


### Bug Fixes

* **source-aws:** prevent s3 client from being logged when errors are
thrown ([#1475](#1475))
([d146aa1](d146aa1))
</details>

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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