Skip to content

Use crypto/rand for mask key#846

Closed
ghost wants to merge 1 commit into
mainfrom
unknown repository
Closed

Use crypto/rand for mask key#846
ghost wants to merge 1 commit into
mainfrom
unknown repository

Conversation

@ghost

@ghost ghost commented Aug 26, 2023

Copy link
Copy Markdown

This PR fixes issues with a PR in progress:

  • Call crypto/rand Reader directly to get four random bytes.
  • Use seeded math/rand Reader for reproducible results in tests.

@coreydaley

Copy link
Copy Markdown

I have pulled your pull request into #840

@coreydaley coreydaley closed this Aug 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant