Closed Bug 1940265 Opened 7 months ago Closed 2 months ago

Offer option to print rather than open phab auth code URL when using mach try pushing to lando

Categories

(Developer Infrastructure :: Try, enhancement, P3)

Desktop
All
enhancement

Tracking

(firefox142 fixed)

RESOLVED FIXED
142 Branch
Tracking Status
firefox142 --- fixed

People

(Reporter: Gijs, Assigned: glob)

Details

Attachments

(1 file)

moz-phab version (from moz-phab version): 1.8.1

STR:

  1. commit something in m-c
  2. ./mach try fuzzy
  3. select all the tasks you want
  4. accept out of fzf
  5. 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).

  1. 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.

(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.

Type: defect → enhancement
Component: moz-phab → Try
Product: Conduit → Developer Infrastructure
Severity: -- → S3
Priority: -- → P3
Severity: S3 → --

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.

Assignee: nobody → glob
Status: NEW → ASSIGNED

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.

Status: ASSIGNED → RESOLVED
Closed: 2 months ago
Resolution: --- → FIXED
Target Milestone: --- → 142 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: