Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix: wrap base64-encoded PEM with 64-char line boundary #1292

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

fix: wrap base64-encoded PEM with 64-char line boundary #1292

wants to merge 1 commit into from

Commits on Mar 14, 2024

  1. fix: Wrap base64-encoded PEM with 64-char line boundary 

    According to [RFC7468](https://datatracker.ietf.org/doc/html/rfc7468)

> Generators MUST wrap the base64-encoded lines so that each line
  consists of exactly 64 characters except for the final line, which
  will encode the remainder of the data (within the 64-character line
  boundary), and they MUST NOT emit extraneous whitespace.

Parsers can avoid branching and prevent timing sidechannel attacks. Ref https://arxiv.org/pdf/2108.04600.pdf

Fixes compatibility with Deno as it enforces stricter handling of PEM.
    @littledivy
    littledivy committed Mar 14, 2024
    Configuration menu
    Copy the full SHA
    5675af4 View commit details
    Browse the repository at this point in the history