Page MenuHomePhabricator

E-mail notification should work with protocol relative urls
Closed, ResolvedPublic

Description

Discovered on wikitech, when the server hostname is configured with a relative protocol, the mails sent are broken.

It should either default to a configuration variable, a user preference or whatever the request triggering the mail had as protocol.

The problem with the latter is that it results in inconsistent e-mail (ie. user John get's mixed urls in his emails, some http some https one of which he is mostly likely not logged in at)


Version: unspecified
Severity: normal

Details

Reference
bz29993
TitleReferenceAuthorSource BranchDest Branch
Make image work with BotPassword auth for officewikitoolforge-repos/officewikibot-pywikibot!1bd808work/bd808/botpasswordmain
Customize query in GitLab

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 21 2014, 11:36 PM
bzimport added a project: MediaWiki-Email.
bzimport set Reference to bz29993.
bzimport added a subscriber: Unknown Object (MLST).

This should be fixed with r93891, but I didn't test that. Can someone confirm?

  • Bug 30204 has been marked as a duplicate of this bug. ***

(In reply to comment #1)

This should be fixed with r93891, but I didn't test that. Can someone confirm?

Deployed the fix, and confirmed it on testwiki by asking Ryan to edit my talk page.

Turns out to be only semi-fixed. The primary URL is fixed now, but the footer reads:

To change your email notification settings, visit
//test.wikipedia.org/wiki/Special:Preferences

To change your watchlist settings, visit
//test.wikipedia.org/wiki/Special:Watchlist/edit

To delete the page from your watchlist, visit
http://test.wikipedia.org/w/index.php?title=User_talk:Catrope&action=unwatch

Feedback and further assistance:
//test.wikipedia.org/wiki/Help:Contents

(In reply to comment #4)

Turns out to be only semi-fixed. The primary URL is fixed now, but the footer
reads:

To change your email notification settings, visit
//test.wikipedia.org/wiki/Special:Preferences

Those were particularly nasty, because the i18n message looked like this:

To change your email notification settings, visit
{{fullurl:{{#special:Preferences}}}}

Fixed in a somewhat elaborate way, as part of a reorganization that needed to be done anyway, in r94995.