Page MenuHomePhabricator

Provide a way to show OAuth authorization dialog with a layout appropriate for a popup
Closed, ResolvedPublicFeature

Description

I would like to open a dialog box for OAuth authorization instead of redirecting to the whole page and then back. So it would be great if I could pass a parameter to make the dialog be the only thing in the window, and not just a small part of it.

(Implemented with display=popup, as per the OIDC spec.)

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 22 2014, 3:27 AM
bzimport set Reference to bz69246.
bzimport added a subscriber: Unknown Object (MLST).

We can't allow framing of the page, but it should be a trivial change to prevent loading the javascript that creates the dialog, so you can have the authorization page in a small popup like Yahoo used to use.

Yes, I had a popup in mind. I am using now a popup, but the issue is that then the overlay inside the page is not over the whole window. So there could be a query string parameter which would make overlay inside the window appear from edge to edge.

Aklapper changed the subtype of this task from "Task" to "Feature Request".Feb 4 2022, 12:23 PM

This would ideally extend to the login page, since authorization might involve logging the user in. So it has some overlap with T71596: Provide JavaScript login widget. Opening the login dialog in a separate window would be an acceptable workaround, though.

Change #1031627 had a related patch set uploaded (by Bartosz Dziewoński; author: Bartosz Dziewoński):

[mediawiki/extensions/OAuth@master] SpecialMWOAuth: Support &display=popup

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

Example of the popup style being used in a tool:


(implementation: https://gitlab.wikimedia.org/repos/ci-tools/patchdemo/-/merge_requests/615)

Change #1031627 merged by jenkins-bot:

[mediawiki/extensions/OAuth@master] SpecialMWOAuth: Support &display=popup

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

matmarex claimed this task.

This has been implemented. If you make use of this mode, we'd appreciate comments (on this task) about what you or your users think about it.

We should probably advertise this to tool authors. Not sure what's the best mechanism for that. Tech News? wikitech-l? There's a Developer-notice tag but I don't think it does much.

Tgr renamed this task from Provide a way to open OAuth authorization dialog over whole page to Provide a way to show OAuth authorization dialog with a layout appropriate for a popup.May 29 2024, 10:40 AM
Tgr updated the task description. (Show Details)