Skip to content

Use random_bytes - #20915

Merged
rullzer merged 1 commit into
masterfrom
enh/use_random_bytes_in_crypto
May 11, 2020
Merged

Use random_bytes#20915
rullzer merged 1 commit into
masterfrom
enh/use_random_bytes_in_crypto

Conversation

@rullzer

Copy link
Copy Markdown
Member

Since we don't care if it is human readbale.
The code is backwards compatible with the old format.

Signed-off-by: Roeland Jago Douma roeland@famdouma.nl

@ChristophWurstChristophWurst left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

🐘

Comment threadlib/private/Security/Crypto.php
Since we don't care if it is human readbale.
The code is backwards compatible with the old format.
Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
@rullzer
rullzerforce-pushed the enh/use_random_bytes_in_crypto branch from 421f78d to 35ff4aaCompareMay 11, 2020 10:47

@nickvergessennickvergessen left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Didn't test, but looks good.

@nickvergessen

Copy link
Copy Markdown
Member

Backport till 17?

@rullzer

Copy link
Copy Markdown
MemberAuthor

Backport till 17?

sounds good!

@rullzer

Copy link
Copy Markdown
MemberAuthor

/backport to stable19

@rullzer

Copy link
Copy Markdown
MemberAuthor

/backport to stable18

@rullzer

Copy link
Copy Markdown
MemberAuthor

/backport to stable17

@rullzer
rullzer merged commit 7211d44 into masterMay 11, 2020
@rullzer
rullzer deleted the enh/use_random_bytes_in_crypto branch May 11, 2020 12:21
@lynn-stephenson

Copy link
Copy Markdown
Contributor

Glad to see this issue was fixed! :D

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviewsenhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@rullzer@nickvergessen@lynn-stephenson@georgehrke@ChristophWurst