Maniphest T108565

Shortening only a domain without a trailing slash fails
Closed, DuplicatePublic
Actions

Assigned To

None

Authored By

	polybuildr
	Aug 10 2015, 8:02 AM

Tags

Referenced Files

None

Subscribers

View All 28 Subscribers

Description

Based on the demo at http://urlshortener.wmflabs.org/, trying to shorten just http://en.wikipedia.org does not work - a trailing slash is required.

I'm not sure whether this is meant to be supported, but maybe the regex (?) needs a change?

Related Objects
Search...

		Status	Subtype	Assigned	Task
		Duplicate		None	T108565 Shortening only a domain without a trailing slash fails
		Resolved		Krinkle	T103379 Add 'url' module for native URL constructor (with conditionally loaded polyfill)

Event Timeline

polybuildr created this task.Aug 10 2015, 8:02 AM

polybuildr raised the priority of this task from to Needs Triage.

polybuildr updated the task description. (Show Details)

polybuildr added a project: MediaWiki-extensions-UrlShortener.

polybuildr subscribed.

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptAug 10 2015, 8:02 AM

This is because new mw.Uri('https://en.wikipedia.org'); throws an exception: Error: Bad constructor arguments.

https://gerrit.wikimedia.org/r/#/c/139584/ was Yuvi's attempt to fix this.

Legoktm added a subtask: T68617: mw.Uri should support URLs like http://wikimedia.org without trailing slash.Aug 10 2015, 6:02 PM

yuvipanda unsubscribed.Jun 30 2016, 2:01 PM

MarkTraceur unsubscribed.Jan 27 2017, 10:36 PM

Krinkle edited subtasks, added: T103379: Add 'url' module for native URL constructor (with conditionally loaded polyfill); removed: T68617: mw.Uri should support URLs like http://wikimedia.org without trailing slash.Apr 2 2019, 3:37 AM

This actually works on meta.wikimedia.org, but an error is still generated and the URL is not normalized (see T220718).

Jc86035 mentioned this in T108602: Equivalent URLs are not canonicalized and deduplicated.Apr 13 2019, 6:03 PM

Legoktm closed this task as a duplicate of T220718: URLs with no slashes after domain name are "invalid" but are still shortened.Apr 14 2019, 8:33 AM

Krinkle closed subtask T103379: Add 'url' module for native URL constructor (with conditionally loaded polyfill) as Resolved.Mar 2 2022, 5:22 PM

Krinkle reopened subtask T103379: Add 'url' module for native URL constructor (with conditionally loaded polyfill) as Open.Mar 7 2022, 7:09 PM

Krinkle closed subtask T103379: Add 'url' module for native URL constructor (with conditionally loaded polyfill) as Resolved.Mar 27 2022, 10:21 PM