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

Implementing the new tokens of the original SunbirdAPI #13

Closed
wants to merge 53 commits into from

Conversation

ConradKash
Copy link
Collaborator

No description provided.

@gitguardian
Copy link

gitguardian bot commented Sep 9, 2023

⚠️ GitGuardian has uncovered 14 secrets following the scan of your pull request.

Please consider investigating the findings and remediating the incidents. Failure to do so may lead to compromising the associated services or software components.

🔎 Detected hardcoded secrets in your pull request
GitGuardian id Secret Commit Filename
7962162 Bearer Token 45c939c app/inference_services/base.py View secret
7962162 Bearer Token dd9ba60 app/inference_services/base.py View secret
7962162 Bearer Token 2d2c823 app/inference_services/base.py View secret
7962162 Bearer Token 3f84d2a app/inference_services/base.py View secret
7962162 Bearer Token db88236 app/inference_services/base.py View secret
7962162 Bearer Token 1755c9b app/inference_services/base.py View secret
7962162 Bearer Token 3f84d2a app/inference_services/base.py View secret
7962162 Bearer Token b0f1dec app/inference_services/base.py View secret
7962162 Bearer Token a481aa3 app/inference_services/base.py View secret
7962162 Bearer Token 58e2d05 app/inference_services/base.py View secret
7962162 Bearer Token 0e2436b app/inference_services/base.py View secret
7962162 Bearer Token 45c939c app/inference_services/base.py View secret
7962162 Bearer Token 26a3707 app/inference_services/base.py View secret
7962162 Bearer Token c26537f app/inference_services/base.py View secret
🛠 Guidelines to remediate hardcoded secrets
  1. Understand the implications of revoking this secret by investigating where it is used in your code.
  2. Replace and store your secrets safely. Learn here the best practices.
  3. Revoke and rotate these secrets.
  4. If possible, rewrite git history. Rewriting git history is not a trivial act. You might completely break other contributing developers' workflow and you risk accidentally deleting legitimate data.

To avoid such incidents in the future consider


🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.

Our GitHub checks need improvements? Share your feedbacks!

@ConradKash ConradKash removed the request for review from Ignatius-K September 11, 2023 20:06
@ConradKash
Copy link
Collaborator Author

Hey @cmplx-xyttmt I am going to create another branh please mege this one.
I am going to start working on the Jinja tempplate and twilwing as i integrate the new api key

@cmplx-xyttmt
Copy link
Member

Hey @cmplx-xyttmt I am going to create another branh please mege this one. I am going to start working on the Jinja tempplate and twilwing as i integrate the new api key

Hello, I can't merge this branch because it has commits that were already merged. That will mess up the git history. Like I said, first go to main and pull the latest changes, then create a new branch where you can implement the changes here and do the frontend.

@cmplx-xyttmt cmplx-xyttmt deleted the imp-text-translation branch September 12, 2023 08: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.

2 participants