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

Add support for secret key #405

Merged
merged 3 commits into from
Dec 28, 2022
Merged

Add support for secret key #405

merged 3 commits into from
Dec 28, 2022

Conversation

fantix
Copy link
Member

@fantix fantix commented Nov 25, 2022

Secret keys are JWTs prefixed with nbwt_ or edbt_ for authentication,
default to secret_key in $config_dir/$cloud_profile.json for cloud instances.

Depends on edgedb/shared-client-testcases#17, edgedb/shared-client-testcases#18

@fantix fantix requested a review from elprans November 25, 2022 16:14
Secret keys are JWTs prefixed with nbwt_ or edbt_ for authentication,
default to access_token in $config_dir/cloud.json for cloud instances.
@fantix fantix force-pushed the secret-key branch 2 times, most recently from a4755d9 to 8a37aa2 Compare December 27, 2022 23:46
@fantix fantix merged commit b10cc30 into master Dec 28, 2022
@fantix fantix deleted the secret-key branch December 28, 2022 21:26
fantix added a commit that referenced this pull request Feb 9, 2023
Secret keys are JWTs prefixed with nbwt_ or edbt_ for authentication,
default to secret_key in $config_dir/$cloud_profile.json for cloud instances.
fantix added a commit that referenced this pull request Feb 9, 2023
Changes
=======

* Add support for secret key authentication, and Cloud instance (#405)
  (by @fantix in df1f6fb, 076e8c8, 27e7a4f for #405, @elprans in 65c9c37 for #412)

Docs
====

* Minor changes (#413)
  (by @0xsirsaif in 14363fb for #413)
@fantix fantix mentioned this pull request Feb 9, 2023
@aljazerzen aljazerzen mentioned this pull request Feb 23, 2024
This was referenced Aug 2, 2024
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.

2 participants