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

TO-URL returns incorrect value for % symbol #2009

Closed
Siskin-Bot opened this issue Feb 15, 2020 · 0 comments
Closed

TO-URL returns incorrect value for % symbol #2009

Siskin-Bot opened this issue Feb 15, 2020 · 0 comments

Comments

@Siskin-Bot
Copy link
Collaborator

Siskin-Bot commented Feb 15, 2020

Submitted by: johnk

TO-URL does not encode % correctly when passed in a string or char

Reference to discussion on google groups:

https://groups.google.com/d/msg/rebol/rquLOryb5nM/Srahpr_71-0J

;In R2
>> to-url #"%"
== %
;In R3
>> to-url #"%"
== %25

Imported from: CureCode [ Version: r3 master Type: Bug Platform: All Category: Native Reproduce: Always Fixed-in:none ]
Imported from: metaeducation#2009

Comments:

Rebolbot commented on Apr 2, 2013:

Submitted by: Ladislav

This may as well be just a MOLD bug, I do not think the TO-URL behaviour is incorrect.


Rebolbot commented on Apr 2, 2013:

Submitted by: johnk

Reading this I agree that this should not be a bug.

https://en.wikipedia.org/wiki/Percent-encoding

Equivalent code in ruby

>> URI::encode "%"
=> "%"

Rebolbot commented on Apr 2, 2013:

Submitted by: BrianH

TO-URL, like all of the TO-datatype functions, only does datatype conversion. It doesn't and is not supposed to do any encoding. R3 has encoding bugs (there are many tickets about this) but it's not TO-URL's fault.


Rebolbot commented on Apr 3, 2013:

Submitted by: johnk

Thanks for the clarification Brian


Rebolbot commented on Apr 3, 2013:

Submitted by: Ladislav

See #2010, #2011 and #2012.


Rebolbot commented on Apr 4, 2013:

Submitted by: BrianH

See #482, #1327, #1333 and #1644 for the existing tickets.


Rebolbot added the Type.bug on Jan 12, 2016


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