Skip to content

Requester Pays Best Practice#1021

Merged
cholmes merged 14 commits intodevfrom
req-pays-practice
Mar 2, 2021
Merged

Requester Pays Best Practice#1021
cholmes merged 14 commits intodevfrom
req-pays-practice

Conversation

@cholmes
Copy link
Copy Markdown
Contributor

@cholmes cholmes commented Mar 2, 2021

Related Issue(s): #896

Proposed Changes:

  1. Added best practice section with recommendations to put STAC json in public bucket even if data is in requester pays
  2. recommendation to use cloud-specific request protocols
  3. NOTE - did not yet add a link to the storage extension, since it's just a PR, and one that will get closed. Can add the recommendation in when we get a location for it.

PR Checklist:

  • This PR is made against the dev branch (all proposed changes except releases should be against dev, not master).
  • This PR has no breaking changes.
  • I have added my changes to the CHANGELOG.

Co-authored-by: Phil Varner <[email protected]>
cholmes and others added 3 commits March 2, 2021 07:05
Co-authored-by: Phil Varner <[email protected]>
Co-authored-by: Phil Varner <[email protected]>
Co-authored-by: Phil Varner <[email protected]>
Co-authored-by: Phil Varner <[email protected]>
Copy link
Copy Markdown
Contributor

@lossyrob lossyrob left a comment

Choose a reason for hiding this comment

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

Approved, reworded and fixed some typos.

cholmes and others added 4 commits March 2, 2021 08:46
Co-authored-by: Rob Emanuele <[email protected]>
Co-authored-by: Rob Emanuele <[email protected]>
Co-authored-by: Rob Emanuele <[email protected]>
Co-authored-by: Rob Emanuele <[email protected]>
@cholmes cholmes merged commit e5ed34f into dev Mar 2, 2021
@cholmes cholmes deleted the req-pays-practice branch March 2, 2021 16:49
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.

4 participants