Skip to content

Escape % in manifest filenames as required by RFC - #211

Open
maxnoe wants to merge 1 commit into
LibraryOfCongress:masterfrom
maxnoe:percent-encode
Open

maxnoe wants to merge 1 commit into
LibraryOfCongress:masterfrom
maxnoe:percent-encode

Conversation

@maxnoe

@maxnoe maxnoe commented Nov 2, 2026

Copy link
Copy Markdown

The code was missing the escaping for the % itself, as required by the RFC:

o If a filepath includes a Line Feed (LF), a Carriage Return (CR),
a Carriage-Return Line Feed (CRLF), or a percent sign (%), those
characters (and only those) MUST be percent-encoded following
[RFC3986].

@sonarqubecloud

sonarqubecloud Bot commented Nov 2, 2026

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
B Maintainability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

This branch has not been deployed

No deployments
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.

1 participant