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: Mask API keys Fixes #1213 #1331

Merged
merged 8 commits into from
Jul 31, 2024

Conversation

yogeshojha
Copy link
Owner

This PR masks all the api keys, tokens, and sensitive URLs

  • Hides Hackerone API token
  • Hides API tokens from API vault (OpenAI/Netlas)
  • Hides notification token from Notification Settings

@yogeshojha yogeshojha self-assigned this Jul 28, 2024
@yogeshojha yogeshojha linked an issue Jul 28, 2024 that may be closed by this pull request
1 task
Copy link
Contributor

👋 Hi @yogeshojha,
Thank you for sending this pull request.
Please make sure you have followed our contribution guidelines.
We will review this PR as soon as possible. Thank you for your patience.

@yogeshojha yogeshojha changed the base branch from master to release/2.2.0 July 28, 2024 02:05
@yogeshojha yogeshojha merged commit b372f8a into release/2.2.0 Jul 31, 2024
6 checks passed
@yogeshojha yogeshojha changed the title Mask API keys Fixes #1213 feat: Mask API keys Fixes #1213 Aug 31, 2024
@yogeshojha yogeshojha deleted the 1213-feat-mask-api-keys-in-settings-view branch September 14, 2024 02:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feat: Mask API keys in settings view.
1 participant