Page MenuHomePhabricator

Remove the "HTTPS to HTTP" url filter in the IRC feed
Closed, ResolvedPublic

Description

https://gerrit.wikimedia.org/r/#/c/217858/

This needs to be announced in tech news etc.

Deployment date: Monday, May 2nd, 2016.

Event Timeline

Krenair raised the priority of this task from to Needs Triage.
Krenair updated the task description. (Show Details)
Krenair subscribed.

Change 217858 had a related patch set uploaded (by Alex Monk):
Do not rewrite https -> http for IRC notifications

https://gerrit.wikimedia.org/r/217858

This is about recent changes IRC feed, right? When do you expect this change to take effect?

Note the IRC feed URLs have been set explicitly to HTTP because of T31925: https-links should not be outputted to irc.wikimedia.org (Regression). Using https:// broke patrolling bots.

Looks like a User-notice has been sent to https://meta.wikimedia.org/wiki/Tech/News/2016/02 , one might want to notify wikitech-l and maybe labs-l as well.

IIRC a lot of bots relying on IRC are legacy tools and some might not have any developer to do the adjustment. But maybe we can be very firm about it and highlight the RCStream system which is supposed to takeover IRC.

If there is major outcry, guess we will want to revert to http and plan on phasing out the IRC service entirely and have tools to be migrated to RCStream.

Krinkle renamed this task from Stop rewriting URLs to unencrypted HTTP in the IRC feed to Remove the "HTTPS to HTTP" url filter in the IRC feed.Mar 16 2016, 11:46 PM
Krinkle removed a project: Traffic.
Andrew triaged this task as Medium priority.Apr 14 2016, 7:46 PM

19:04 < mutante> that irc link thing there. it has a comment "can be merged anytime. no impact"
19:04 < mutante> and a scheduled deployment date of May 2nd
19:04 < RoanKattouw> Yeah, if Krinkle says that I'll happily merge+deploy
19:04 < mutante> it should have been in the earlier window but irc.wm wasn't ready due to other bugs
19:04 < grrrit-wm> (CR) Catrope: [C: 2] Remove obsolete 'https -> http' rewrite for IRC notifications [mediawiki-config] -

https://gerrit.wikimedia.org/r/217858 (https://phabricator.wikimedia.org/T122933) (owner: Faidon Liambotis)

19:04 < mutante> thanks a lot :)

per "Don't merge before May 2nd, 2016." , just on time

Dzahn claimed this task.

claiming resolved, but _please_ confirm

Change 217858 merged by jenkins-bot:
Remove obsolete 'https -> http' rewrite for IRC notifications

https://gerrit.wikimedia.org/r/217858

@Johan the change happened now, late but still on May 2nd as announced