Closed Bug 483660 Opened 15 years ago Closed 6 years ago

Consider not setting gEscapeUTF8 to true

Categories

(Core :: Networking, defect, P3)

x86
macOS
defect

Tracking

()

RESOLVED WONTFIX

People

(Reporter: bzbarsky, Unassigned)

References

(Blocks 1 open bug)

Details

(Whiteboard: [necko-backlog])

Right now, nsStandardURL escapes all non-ASCII bytes in its internal representation, as far as I can tell.  It certainly does so for refs, which causes issues like bug 135309.

Is there a reason we're not storing in IRI form internally, escaping to URI as needed when someone actually needs an ASCII spec?  We have GetAsciiSpec anyway...
That said, is flipping that pref basically all we need to get IRI behavior?
Blocks: 135309
Flags: blocking1.9.2?
Blocks: 436006
Flags: blocking1.9.2? → blocking1.9.2-
Blocks: 582361
I think we're really not pursuing iri at this point. reopen if I've got that wrong
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → WONTFIX
This blocks observable bugs in our HTML support, so if we're not going to change necko here we need to figure out some other approach...  I still think fixing this in necko (via aligning with the URL spec, not IRI) is the right thing.
Status: RESOLVED → REOPENED
Resolution: WONTFIX → ---
Whiteboard: [necko-backlog]
Bulk change to priority: https://bugzilla.mozilla.org/show_bug.cgi?id=1399258
Priority: -- → P1
Bulk change to priority: https://bugzilla.mozilla.org/show_bug.cgi?id=1399258
Priority: P1 → P3
How exactly are we not aligned with the URL Standard here? The URL Standard requires UTF-8 percent encoding of non-ASCII input (except for the query, where we use the document encoding to do so). I believe Gecko/Necko is aligned with that.
Flags: needinfo?(bzbarsky)
> How exactly are we not aligned with the URL Standard here?

No idea, sorry.  I have been tracking neither the URL standard nor Necko on this issue.  I expect Valentin knows where things stand.
Flags: needinfo?(bzbarsky) → needinfo?(valentin.gosu)
This pref was removed in Bug 1321705.
What Anne said in comment 6 is what we do.
Status: REOPENED → RESOLVED
Closed: 8 years ago6 years ago
Flags: needinfo?(valentin.gosu)
Resolution: --- → FIXED
See Also: → 1321705
Resolution: FIXED → WONTFIX
You need to log in before you can comment on or make changes to this bug.