Skip to content

Further improve NSEC base32 handling. - #1372

Merged
rthalley merged 2 commits into
rthalley:mainfrom
bwelling:base32
Oct 20, 2026
Merged

rthalley merged 2 commits into
rthalley:mainfrom
bwelling:base32

Conversation

@bwelling

Copy link
Copy Markdown
Collaborator

If running under Python 3.15 or newer, we can use no-padding mode for encode and decode, rather than doing the padding ourselves.

Also, move the comment describing the extra padding checks for <3.15 closer to the code doing the checks.

If running under Python 3.15 or newer, we can use no-padding mode for
encode and decode, rather than doing the padding ourselves.

Also, move the comment describing the extra padding checks for <3.15
closer to the code doing the checks.
@rthalley
rthalley merged commit 9978a71 into rthalley:main Oct 20, 2026
10 checks passed
@bwelling
bwelling deleted the base32 branch October 21, 2026 00:57
rthalley pushed a commit that referenced this pull request Oct 22, 2026
* Further improve NSEC base32 handling.

If running under Python 3.15 or newer, we can use no-padding mode for
encode and decode, rather than doing the padding ourselves.

Also, move the comment describing the extra padding checks for <3.15
closer to the code doing the checks.

* Invoke base64.b32hexdecode correctly.

(cherry picked from commit 9978a71)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants