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

Separate headers for sensitive values (orgKey) in registerCluster response #1331

Merged
merged 1 commit into from
Jun 13, 2023

Conversation

carrolp
Copy link
Contributor

@carrolp carrolp commented Jun 13, 2023

Cluster registration, deletion, and enableRegistrationUrl apis now return sensitive values as separate headers object in addition to in the url query params.

Using the url alone, with included authorization value (orgKey) is still possible, but now deprecated. A future major release will make the breaking change that removes the query parameter from the URL and require use of the headers returned separately for authorized requests. The use of headers for sensitive values (e.g. authorization values) will improve security.

… sensitive values as separate headers object in addition to in the url query params.

Using the url alone, with included authorization value (orgKey) is still possible, but now deprecated.
A future major release will make the breaking change that removes the query parameter from the URL and require use of the headers returned separately for authorized requests.
The use of headers for sensitive values (e.g. authorization values) will improve security.
@carrolp carrolp changed the title Cluster registration, deletion, and enableRegistrationUrl apis return… Separate headers for sensitive values (orgKey) in registerCluster response Jun 13, 2023
@carrolp carrolp marked this pull request as ready for review June 13, 2023 20:54
@carrolp carrolp requested a review from kdai7 June 13, 2023 20:55
@carrolp carrolp merged commit 266402f into master Jun 13, 2023
2 checks passed
@carrolp carrolp deleted the register-script-20230607 branch June 13, 2023 21:10
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

2 participants