Page MenuHomePhabricator

In ext. links don't change "\" to "/"
Closed, ResolvedPublic

Description

Author: wikipedia

Description:
The Link
[http://www.bosch-presse.de/cgi-bin/cgiWebLinkImage.exe/3-10031.JPG.jpg?sid=5733990&action=jpeg&image=\\Obelix\arena2\Bilder\bosch_pb\1298\jpg_org\jpg\3-10031.jpg]
is not beeing rendered, but shown like it would be inside a <nowiki> section.


Version: unspecified
Severity: normal
URL: http://de.wikipedia.org/w/index.php?title=Benutzer:Habakuk/Bilder&oldid=3609588

Details

Reference
bz1001
TitleReferenceAuthorSource BranchDest Branch
replace reviewer jelto with wmf-sre-collab grouprepos/sre/miscweb/research-landing-page!15jeltoreplace-jelto-readmemaster
add .gitlab-ci.ymlrepos/sre/miscweb/bugzilla!1jeltoadd-cimain
add /repos/sre/miscweb/bugzilla to Trusted Runnersrepos/releng/gitlab-trusted-runner!42jeltoadd-bugzillamain
use timestamp in image tagrepos/sre/miscweb/annualreport!5jeltouse-timestamp-tagsmaster
use timestamp in image tagrepos/sre/miscweb/bienvenida!2jeltouse-timestamp-tagsmaster
use timestamp in image tagrepos/sre/miscweb/transparencyreport!2jeltouse-timestamp-tagsmaster
allow annualreport on trusted runnersrepos/releng/gitlab-trusted-runner!22jeltoadd-miscweb-annualreportmain
Add Ci image build and publishrepos/sre/miscweb/annualreport!2jeltoci-image-buildmaster
add blubber image build for annual report sitesrepos/sre/miscweb/annualreport!1jeltoblubber-buildmaster
Customize query in GitLab

Related Objects

StatusSubtypeAssignedTask
ResolvedNone
ResolvedNone

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 21 2014, 8:00 PM
bzimport added a project: MediaWiki-Parser.
bzimport set Reference to bz1001.
bzimport added a subscriber: Unknown Object (MLST).

zigger wrote:

de.wikipedia.org with the bug (parsing stops at the \ character and the
surrounding brace-pair is displayed) is running v1.3.7wm. test.wikipedia.org on
1.4beta1 treats the \ character as part of the URL ok, but they are getting
changed to the / character in the href attribute instead of being encoded.
Related to bug 343.

http://www.faqs.org/rfcs/rfc2396.html

2.4.3. Excluded US-ASCII Characters

Although they are disallowed within the URI syntax, we include here a
description of those US-ASCII characters that have been excluded and
the reasons for their exclusion.

[snip]

Other characters are excluded because gateways and other transport
agents are known to sometimes modify such characters, or they are
used as delimiters.

unwise      = "{" | "}" | "|" | "\" | "^" | "[" | "]" | "`"

Data corresponding to excluded characters must be escaped in order to
be properly represented within a URI.

So I'm pretty sure that a literal "\" is not allowed in a URL.

bugzillas+padREMOVETHISdu wrote:

Yeah, "\" is not allowed. So it should be converted to "%5C". That's what
comment 1 says. Converting it to "/" might not always work, but I'm not sure
about it. Is it known that all "major" web servers treat "%5C" as equivalent to "/"?

The server at www.bosch-presse.de which is linked to by the URL of this bug
seems to treat "%5C" and "/" equivalently, though.

bugzillas+padREMOVETHISdu wrote:

This bug only deals with links with a "\". For other problematic characters see
bug 343.

Changing the URL to an older version of the page that uses an ext. link with "\".