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.

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

DECODE-URL ignores restriction on passwords #1333

Closed
Siskin-Bot opened this issue Feb 15, 2020 · 1 comment
Closed

DECODE-URL ignores restriction on passwords #1333

Siskin-Bot opened this issue Feb 15, 2020 · 1 comment

Comments

@Siskin-Bot
Copy link
Collaborator

Siskin-Bot commented Feb 15, 2020

Submitted by: meijeru

Inspection of SYSTEM/INTRINSIC/PARSE-URL reveals that the charset defined for the optional password component (PASS-CHAR) is not used in the rules. This charset would forbid whitespace in the password.

>> decode-url to-url "ftp://a:b c@d"
== [scheme: 'ftp pass: "b c" user: "a" host: "d"]

Imported from: CureCode [ Version: alpha 94 Type: Bug Platform: All Category: Mezzanine Reproduce: Always Fixed-in:none ]
Imported from: metaeducation#1333

Comments:


Rebolbot mentioned this issue on Jan 12, 2016:
DECODE-URL and url! syntax don't obey the url encoding rules
TO-URL returns incorrect value for % symbol
LOAD does not handle correctly URLs containing encoded delimiters
MOLD does not handle correctly URLs containing encoded characters


Rebolbot added the Type.bug on Jan 12, 2016


Oldes commented on Jun 6, 2019:

[scheme: 'ftp pass: "b c" user: "a" host: "d"] result above (which I get in old 2.100.111.3.1 R3 version) seems to be correct or am I missing something?


Oldes commented on Jun 6, 2019:

Ah... I see... right... the pass-char value is not used.


IngoHohmann mentioned this issue on Feb 5, 2020:
Semicolons should be legal in URL


@Oldes
Copy link
Owner

Oldes commented Apr 14, 2022

There is a new version of the decode-url now. Space is allowed as long as it is %-encoded

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants