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

#75 | Private and Public Keys Have Been Encrypted & Flows and Tests Have Been Refactored #104

Merged
merged 5 commits into from
Jul 10, 2023

Conversation

agitrubard
Copy link
Collaborator

@agitrubard agitrubard commented Jun 25, 2023

Previously, the private and public keys used for token generation were openly stored in the database. Due to the potential security vulnerability, it was considered more appropriate to hash these keys and store them in the database. To address this issue, a private repository was created specifically for the purpose of hashing operations. Within this repository, hashing workflows were developed to ensure that the hashing process is not directly accessible from external sources. Subsequently, this library was integrated into the backend project. After the integration, the workflows were updated, and the keys are now stored in a hashed form, rather than being openly accessible. The database scripts were also updated accordingly.

AYS BE Encryption Utility Library : https://github.com/afet-yonetim-sistemi/ays-be-encryption-utility

Copy link
Contributor

@moaydogdu moaydogdu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks cool!

@agitrubard agitrubard linked an issue Jul 2, 2023 that may be closed by this pull request
Copy link
Contributor

@Rapter1990 Rapter1990 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we define README.md to understand the module easily if possible?

@Rapter1990 Rapter1990 added the in review This issue in review label Jul 3, 2023
@agitrubard agitrubard removed the in review This issue in review label Jul 4, 2023
@agitrubard
Copy link
Collaborator Author

Can we define README.md to understand the module easily if possible?

@Rapter1990 What kind of description did we need to make?

@agitrubard agitrubard changed the title Private and Public Keys Have Been Encrypted & Flows and Tests Have Been Refactored #75 | Private and Public Keys Have Been Encrypted & Flows and Tests Have Been Refactored Jul 9, 2023
Copy link
Contributor

@Rapter1990 Rapter1990 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@Rapter1990 Rapter1990 added the in review This issue in review label Jul 9, 2023
Copy link
Contributor

@emreylmaz emreylmaz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@agitrubard agitrubard merged commit 0a4a0e8 into main Jul 10, 2023
@agitrubard agitrubard deleted the feature/75/data-hashing branch July 10, 2023 20:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in review This issue in review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Veritabanında Tutulan Token Anahtarların Hashlenerek Tutulması
6 participants