Paths

Table of Contentst

Allow "bin/auth recover" to succeed before phabricator.base-uri is set
ClosedPublic
Actions

Authored by epriestley on Nov 20 2013, 5:29 PM.

Tags

None

Referenced Files

	F13814138: D7615.id17188.diff
	Thu, Sep 19, 3:38 AM

	F13814137: D7615.id17187.diff
	Thu, Sep 19, 3:37 AM

	Unknown Object (File)
	Tue, Sep 17, 3:45 AM

	Unknown Object (File)
	Fri, Sep 13, 5:43 PM

	Unknown Object (File)
	Fri, Sep 13, 5:42 PM

	Unknown Object (File)
	Fri, Sep 13, 5:41 PM

	Unknown Object (File)
	Fri, Sep 13, 5:41 PM

	Unknown Object (File)
	Tue, Sep 10, 4:43 AM

View All Files

Subscribers

aran

Details

Reviewers

btrahan

Maniphest Tasks

T4132: Allow `bin/auth recover` to operate even if `phabricator.base-uri` is not set

Commits

Restricted Diffusion Commit
rP6eb02af314a4: Allow "bin/auth recover" to succeed before phabricator.base-uri is set

Summary

Fixes T4132. If you run "bin/auth recover" before setting the base URI, it throws when trying to generate a production URI.

Instead, just show the path. We can't figure out the domain, and I think this is less confusing than showing "your.phabricator.example.com", etc.

Test Plan

Ran bin/auth recover <user> for valid and missing base-uri.

Diff Detail

Branch

authrecover

Lint

Lint Passed

Unit

Tests Passed

Event Timeline

btrahan accepted this revision.Nov 20 2013, 6:33 PM

Closed by commit rP6eb02af314a4 (authored by @epriestley).

Revision Contents
Changeset List

Path

Size

src/

applications/

auth/

management/

PhabricatorAuthManagementRecoverWorkflow.php

3 lines

people/

storage/

PhabricatorUser.php

12 lines

Diff 17187

View Options

src/applications/auth/management/PhabricatorAuthManagementRecoverWorkflow.php