Offer option to print rather than open phab auth code URL when using mach try pushing to lando
Categories
(Developer Infrastructure :: Try, enhancement, P3)
Tracking
(firefox142 fixed)
Tracking | Status | |
---|---|---|
firefox142 | --- | fixed |
People
(Reporter: Gijs, Assigned: glob)
Details
Attachments
(1 file)
moz-phab version (from moz-phab version
): 1.8.1
STR:
- commit something in m-c
- ./mach try fuzzy
- select all the tasks you want
- accept out of
fzf
- a tab opens in the default browser with the lando auth page for the 8-digit code thingy
Unfortunately, the tab opens in a container inside Firefox that does not have the right cookies for me to auth (and I don't want it to have them).
- try to reload the page in a different container
This will fail as the codes are one-time-use.
AR:
errors, failure to auth, and the commandline just sits there waiting for auth and has to be ctrl-C'd
ER:
some way to load the URL in an environment that either already has the right session cookie or where it can be created by authenticating.
Per discussion on slack, the easiest solve here might be an option that will only print the URL instead of open it.
Comment 1•7 months ago
|
||
(In reply to :Gijs (he/him) from comment #0)
Per discussion on slack, the easiest solve here might be an option that will only print the URL instead of open it.
This seems reasonable to me. I took a quick look and the Python webbrowser
module doesn't support containers. I wasn't aware the codes were 1-time use only.
Updated•7 months ago
|
Updated•7 months ago
|
There's situations where opening the auth0 verification URL in the default
browser is problematic; eg. for those using containers and the default
container isn't one that's use for auth0. As the URLs can only be opened once,
there isn't the opportunity to open the URL in another container/browser/etc.
Add a simple environment var 'LANDO_AUTH_NO_BROWSER' to disable the URL opening
action.
Updated•2 months ago
|
The just-landed fix for this requires adding an option to your machrc
file (which may require creating one).
See https://firefox-source-docs.mozilla.org/mach/settings.html and mach settings
.
Comment 5•2 months ago
|
||
bugherder |
Description
•