Issue 42981: Error messages raised by _hashlib_scrypt_impl() are slightly misleading

➜

This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

This issue has been migrated to GitHub: https://github.com/python/cpython/issues/87147

classification

Title:	Error messages raised by _hashlib_scrypt_impl() are slightly misleading
Type:		Stage:	resolved
Components:	Extension Modules	Versions:

process

Status:	closed	Resolution:	wont fix
Dependencies:		Superseder:
Assigned To:		Nosy List:	illia-v
Priority:	normal	Keywords:	patch

Created on 2021-01-20 19:02 by illia-v, last changed 2022-04-11 14:59 by admin. This issue is now closed.

Pull Requests
URL	Status	Linked	Edit
PR 24274	closed	illia-v, 2021-01-20 19:10

Messages (1)
msg385360 - (view)	Author: Illia Volochii (illia-v) *	Date: 2021-01-20 19:02
Error messages state that maxmem and dklen must be smaller than INT_MAX, but values equal to INT_MAX are allowed too. Also, another error message states that n must be a power of 2, but it does not mention that 1 or 2⁰ is not allowed.

msg385360 - (view)

Author: Illia Volochii (illia-v) *

Date: 2021-01-20 19:02

Error messages state that maxmem and dklen must be smaller than INT_MAX, but values equal to INT_MAX are allowed too.

Also, another error message states that n must be a power of 2, but it does not mention that 1 or 2⁰ is not allowed.

History
Date	User	Action	Args
2022-04-11 14:59:40	admin	set	github: 87147
2021-02-20 18:38:06	illia-v	set	status: open -> closed resolution: wont fix stage: patch review -> resolved
2021-01-20 19:10:09	illia-v	set	keywords: + patch stage: patch review pull_requests: + pull_request23097
2021-01-20 19:02:38	illia-v	create