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

fix: change rand lib(math/rand => crypto/rand) #465

Merged
merged 1 commit into from
May 20, 2023

Conversation

Doozers
Copy link
Contributor

@Doozers Doozers commented May 17, 2023

No description provided.

@trafico-bot trafico-bot bot added the 🔍 Ready for Review Pull Request is not reviewed yet label May 17, 2023
@Doozers Doozers force-pushed the fix/isma/rand-lib branch from cd88f65 to a309523 Compare May 17, 2023 09:33
@Doozers Doozers marked this pull request as ready for review May 17, 2023 09:40
@moul
Copy link
Owner

moul commented May 18, 2023

We should add a migration step to disable any previously created tokens that are no longer being used.

@trafico-bot trafico-bot bot added ✅ Approved Pull Request has been approved and can be merged and removed 🔍 Ready for Review Pull Request is not reviewed yet labels May 18, 2023
@moul moul closed this May 20, 2023
@moul moul reopened this May 20, 2023
@moul moul merged commit f9c8f60 into moul:master May 20, 2023
@trafico-bot trafico-bot bot added ✨ Merged Pull Request has been merged successfully and removed ✅ Approved Pull Request has been approved and can be merged labels May 20, 2023
@github-actions
Copy link

🎉 This PR is included in version 1.19.5 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ Merged Pull Request has been merged successfully released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants