Last modified: 2011-08-12 16:24:57 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T31976, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 29976 - protocol relative uri for meta edituri
protocol relative uri for meta edituri
Status: RESOLVED FIXED
Product: MediaWiki
Classification: Unclassified
API (Other open bugs)
unspecified
All All
: Normal normal (vote)
: ---
Assigned To: Roan Kattouw
: patch, patch-need-review
Depends on:
Blocks: 20342
  Show dependency treegraph
 
Reported: 2011-07-20 07:33 UTC by Derk-Jan Hartman
Modified: 2011-08-12 16:24 UTC (History)
9 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments
dze path is simple (505 bytes, patch)
2011-07-20 20:25 UTC, Derk-Jan Hartman
Details

Description Derk-Jan Hartman 2011-07-20 07:33:37 UTC
Test shows:

<link rel="EditURI" type="application/rsd+xml" href="https://secure.wikimedia.org/wikipedia/test/w/api.php?action=rsd" />
Comment 1 Bryan Tong Minh 2011-07-20 07:41:41 UTC
Putting this in the API component, even though this is in the HTML output of index.php, as it refers to an API module.
Comment 2 Brion Vibber 2011-07-20 17:33:57 UTC
I'd say if this needs to be changed, just use a local relative link with the path only. It'll be just as accurate, smaller in output, and slightly more likely to be supported by clients doing discovery.
Comment 3 Derk-Jan Hartman 2011-07-20 20:25:26 UTC
Created attachment 8808 [details]
dze path is simple

Relates to Bug 25648.

Fear is that some of the clients actually using this URL might not support any relative url form.

We probably need a set of testcases so that we can file upstream bugs.
Comment 4 Bryan Tong Minh 2011-07-26 19:25:40 UTC
We could force it to always refer to HTTP.
Comment 5 Roan Kattouw 2011-07-26 21:57:33 UTC
(In reply to comment #4)
> We could force it to always refer to HTTP.
With my pending wfExpandUrl() rewrite, we'd be able to do that, yes, with something like wfExpandUrl( $url, PROT_HTTP );
Comment 6 Roan Kattouw 2011-07-27 09:06:28 UTC
(In reply to comment #5)
> (In reply to comment #4)
> > We could force it to always refer to HTTP.
> With my pending wfExpandUrl() rewrite, we'd be able to do that, yes, with
> something like wfExpandUrl( $url, PROT_HTTP );
wfExpandUrl() rewrite done in r93258, will finish this soonish.
Comment 7 Roan Kattouw 2011-08-12 16:24:57 UTC
Fixed a while ago in r93820 and r93821.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links