Page MenuHomePhabricator

/logout redirects to /index
Closed, ResolvedPublic

Description

/logout view redirects to /index, which doesn't exist and shows 404 after successful logging out.

Waiting on https://github.com/valhallasw/flask-mwoauth/pull/25 to be merged, then https://gerrit.wikimedia.org/r/c/labs/tools/wikinity/+/522868 should be reverted.

Event Timeline

Urbanecm triaged this task as High priority.Apr 6 2019, 2:29 PM

Change 522868 had a related patch set uploaded (by Urbanecm; owner: Urbanecm):
[labs/tools/wikinity@master] Add /index redirecting to /

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

Change 522868 merged by jenkins-bot:
[labs/tools/wikinity@master] Add /index redirecting to /

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

Urbanecm lowered the priority of this task from High to Low.Jul 14 2019, 1:05 PM

The immediate problem should be fixed now, lowering priority. However, the change above just introduced an awful workaround for https://github.com/valhallasw/flask-mwoauth/pull/25. As soon as https://github.com/valhallasw/flask-mwoauth/pull/25 is merged or the underlying issue is fixed somehow else, the patch above should be reverted.

Urbanecm moved this task from Backlog to Patch proposed upstream on the Upstream board.
Urbanecm updated the task description. (Show Details)

Change 633260 had a related patch set uploaded (by Urbanecm; owner: Urbanecm):
[labs/tools/wikinity@master] Revert "Add /index redirecting to /"

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

Change 633260 merged by jenkins-bot:
[labs/tools/wikinity@master] Revert "Add /index redirecting to /"

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