Skip to content

bpo-38153: detect shake independently from sha3 (GH-16143)#16143

Merged
tiran merged 1 commit into
python:masterfrom
tiran:bpo-38153-2
Sep 14, 2019
Merged

bpo-38153: detect shake independently from sha3 (GH-16143)#16143
tiran merged 1 commit into
python:masterfrom
tiran:bpo-38153-2

Conversation

@tiran

@tiran tiran commented Sep 14, 2019

Copy link
Copy Markdown
Member

Signed-off-by: Christian Heimes <christian@python.org>
@tiran tiran changed the title bpo-38153: detect shake independently from sha3 bpo-38153: detect shake independently from sha3 (GH-16143) Sep 14, 2019
@tiran
tiran merged commit eb2b0c6 into python:master Sep 14, 2019
@tiran
tiran deleted the bpo-38153-2 branch September 14, 2019 15:29
@miss-islington

Copy link
Copy Markdown
Contributor

Thanks @tiran for the PR 🌮🎉.. I'm working now to backport this PR to: 3.8.
🐍🍒⛏🤖

@miss-islington

Copy link
Copy Markdown
Contributor

I'm having trouble backporting to 3.8. Reason: 'Error 110 while writing to socket. Connection timed out.'. Please retry by removing and re-adding the needs backport to 3.8 label.

@miss-islington

Copy link
Copy Markdown
Contributor

Thanks @tiran for the PR 🌮🎉.. I'm working now to backport this PR to: 3.8.
🐍🍒⛏🤖

@miss-islington

Copy link
Copy Markdown
Contributor

Sorry, @tiran, I could not cleanly backport this to 3.8 due to a conflict.
Please backport using cherry_picker on command line.
cherry_picker eb2b0c694aef6122fdf95015abb24e0d095b6401 3.8

@miss-islington

Copy link
Copy Markdown
Contributor

Thanks @tiran for the PR 🌮🎉.. I'm working now to backport this PR to: 3.8.
🐍🍒⛏🤖

@bedevere-bot

Copy link
Copy Markdown

GH-16179 is a backport of this pull request to the 3.8 branch.

miss-islington added a commit that referenced this pull request Sep 16, 2019
XOF digests (SHAKE) are not available in OpenSSL 1.1.0 but SHA3 fixed-length digests are.

Signed-off-by: Christian Heimes <christian@python.org>
(cherry picked from commit eb2b0c6)

Co-authored-by: Christian Heimes <christian@python.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants