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

[Backport release-2.24] Add config option to override the payer namespace for REST requests. (#5105) #5126

Merged
merged 1 commit into from
Jun 20, 2024

Conversation

KiterLuc
Copy link
Contributor

Backport bdfc19c from #5105.


TYPE: CONFIG
DESC: Add rest.payer_namespace config option to set the namespace to be charged for REST requests.

…5105)

[SC-49142](https://app.shortcut.com/tiledb-inc/story/49142/add-support-for-rest-x-payer-via-config)

This PR introduces a new config option that allows users to set the
namespace to charge for REST requests when it cannot be automatically
determined or when the user needs to set a specific override.

Big thanks to @Shelnutt2 for implementing this. I added a test and made
some small changes on top of it.

---
TYPE: CONFIG
DESC: Add `rest.payer_namespace` config option to set the namespace to
be charged for REST requests.

---------

Co-authored-by: Seth Shelnutt <seth@tiledb.io>
Co-authored-by: KiterLuc <67824247+KiterLuc@users.noreply.github.com>
Co-authored-by: eric-hughes-tiledb <82400964+eric-hughes-tiledb@users.noreply.github.com>
Co-authored-by: Luc Rancourt <lucrancourt@gmail.com>
@ihnorton ihnorton merged commit 1e0a5ab into release-2.24 Jun 20, 2024
61 of 62 checks passed
@ihnorton ihnorton deleted the backport-5105-to-release-2.24 branch June 20, 2024 02:55
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.

None yet

3 participants