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

Revisit "+" in the type #436

Open
pombredanne opened this issue Mar 28, 2025 · 2 comments
Open

Revisit "+" in the type #436

pombredanne opened this issue Mar 28, 2025 · 2 comments
Assignees
@pombredanne
Labels
Ecma specification Work on the core specification PURL core specification Format and syntax that define PURL (excludes PURL type definitions) PURL encoding

Comments

@pombredanne
Copy link
Member

A plus '+' in a type looks like a wart and is not used in any type, and may go against encoding rules and would make things weird given that + is also treated as an encoded space
@pombredanne pombredanne self-assigned this Mar 28, 2025
@jkowalleck
Copy link
Member

jkowalleck commented Mar 30, 2025 •
edited
Loading

given that + is also treated as an encoded space

since when?
in URL - yes.
in PURL - not yet. PURL clearly states to use percent-encoding for al permitted charactes. space is a permitted character and is encoded as %20

@jkowalleck jkowalleck added PURL core specification Format and syntax that define PURL (excludes PURL type definitions) PURL encoding Ecma specification Work on the core specification labels Mar 30, 2025
@jkowalleck jkowalleck mentioned this issue Mar 30, 2025
Open
@matt-phylum
Copy link
Contributor

PURL never said to do it, but some implementations (still) have bugs where + is changed into or is changed into +. #261

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pombredanne pombredanne

Labels
Ecma specification Work on the core specification PURL core specification Format and syntax that define PURL (excludes PURL type definitions) PURL encoding
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@pombredanne @jkowalleck @matt-phylum