Page MenuHomePhabricator
Feed Advanced Search

Feb 18 2015

epriestley merged task T7309: Give scratch tokens when enabling two-factor authentication into T6549: Backup codes for multi-factor authentication.
Feb 18 2015, 4:30 AM · Auth
zhaofengli added a comment to T7309: Give scratch tokens when enabling two-factor authentication.

It's a Google-specific routine in the app (and only works when you set up your Google account with it), not a part of the TOTP standard. Servers implement scratch codes in their own ways.

Feb 18 2015, 1:52 AM · Auth
• chasemp added a comment to T7309: Give scratch tokens when enabling two-factor authentication.

IIRC the google authenticator app itself gave me one-time use codes when I first set it up. What client side app are you using?

Feb 18 2015, 1:46 AM · Auth
zhaofengli added a project to T7309: Give scratch tokens when enabling two-factor authentication: Auth.
Feb 18 2015, 1:14 AM · Auth

Feb 17 2015

epriestley added a parent task for T5955: Refactor Conduit auth to be stateless, token-based, and support wire encodings: T7288: configuring certificates for bots is hard/confusing.
Feb 17 2015, 12:13 AM · Restricted Project, Auth, Conduit

Feb 13 2015

btrahan updated subscribers of T5955: Refactor Conduit auth to be stateless, token-based, and support wire encodings.
Feb 13 2015, 12:46 AM · Restricted Project, Auth, Conduit

Feb 10 2015

epriestley closed T6484: Conduit API for SSH key retreival as Resolved by committing Restricted Diffusion Commit.
Feb 10 2015, 11:44 PM · Conduit, Auth

Feb 5 2015

FallingBullets added a comment to T6707: 'Attaching' two external accounts of the same type results in AphrontCountQueryException.

Any progress on this?

Feb 5 2015, 10:49 AM · Wikimedia, Auth

Feb 2 2015

• jason.chen closed T7103: Authentication via Reverse Proxy? as Invalid.

Hi @epriestley that ticket is exactly what I'm after. Much thanks.

Feb 2 2015, 9:36 PM · Auth
epriestley triaged T7103: Authentication via Reverse Proxy? as Low priority.
Feb 2 2015, 1:29 PM · Auth

Jan 14 2015

devurandom added a comment to T814: Support HTTP Basic Auth as an authentication mechanism.

REMOTE_USER can come from a lot more sources than just HTTP Basic Auth. E.g. I would use it for GSSAPI authentication. Could you maybe change the bug title?

Jan 14 2015, 10:10 PM · Restricted Project, OpenStack, Auth
epriestley closed T6971: Unhandled Exception ("RuntimeException") – Undefined variable: provider as Resolved by committing Restricted Diffusion Commit.
Jan 14 2015, 7:04 PM · Auth
epriestley added a revision to T6971: Unhandled Exception ("RuntimeException") – Undefined variable: provider: Restricted Differential Revision.
Jan 14 2015, 7:01 PM · Auth
chad added a project to T6971: Unhandled Exception ("RuntimeException") – Undefined variable: provider: Auth.
Jan 14 2015, 2:51 PM · Auth
epriestley updated subscribers of T814: Support HTTP Basic Auth as an authentication mechanism.
Jan 14 2015, 2:02 PM · Restricted Project, OpenStack, Auth
epriestley merged task T6970: REMOTE_USER authentication into T814: Support HTTP Basic Auth as an authentication mechanism.
Jan 14 2015, 2:02 PM · Auth
devurandom created T6970: REMOTE_USER authentication.
Jan 14 2015, 1:40 PM · Auth

Jan 12 2015

epriestley added a revision to T4137: user_externalaccount table goes out of sync when user try to login before adding it using "import from LDAP": Restricted Differential Revision.
Jan 12 2015, 10:52 PM · Support Impact, Auth, LDAP

Jan 9 2015

epriestley added a revision to T6870: Return to previous page after login via header: Restricted Differential Revision.
Jan 9 2015, 12:24 AM · Auth

Jan 8 2015

epriestley closed T6870: Return to previous page after login via header as Resolved by committing Restricted Diffusion Commit.
Jan 8 2015, 11:56 PM · Auth
lpriestley added a revision to T6870: Return to previous page after login via header: Restricted Differential Revision.
Jan 8 2015, 11:23 PM · Auth

Jan 6 2015

epriestley renamed T814: Support HTTP Basic Auth as an authentication mechanism from Basic Auth to Support HTTP Basic Auth as an authentication mechanism.
Jan 6 2015, 12:52 AM · Restricted Project, OpenStack, Auth
epriestley assigned T6870: Return to previous page after login via header to lpriestley.
  • In PhabricatorAuthApplication->buildMainMenuItems(), add ?next=<the current path> to the "Log In" item.
Jan 6 2015, 12:31 AM · Auth

Jan 5 2015

epriestley added a comment to T5955: Refactor Conduit auth to be stateless, token-based, and support wire encodings.

The utf8 stuff hasn't been coming up much; the token-based stuff is just blocked on docs and also hasn't come up much; I touched the protocol enough that I'm pretty confident we can move forward without backward compatibility breaks.

Jan 5 2015, 11:46 PM · Restricted Project, Auth, Conduit
epriestley removed projects from T5955: Refactor Conduit auth to be stateless, token-based, and support wire encodings: Support Impact, High Support Impact.
Jan 5 2015, 11:45 PM · Restricted Project, Auth, Conduit
chad triaged T6870: Return to previous page after login via header as Normal priority.

Seems reasonable to support this.

Jan 5 2015, 11:18 PM · Auth
chad renamed T6870: Return to previous page after login via header from Return to previous page after login to Return to previous page after login via header.
Jan 5 2015, 11:12 PM · Auth

Jan 2 2015

epriestley added a revision to T6484: Conduit API for SSH key retreival: Restricted Differential Revision.
Jan 2 2015, 11:40 PM · Conduit, Auth

Dec 29 2014

chad removed a project from T1205: Allow grey users in some form or other: Maniphest.
Dec 29 2014, 5:34 AM · Restricted Project, People, Auth, Wikimedia

Dec 26 2014

calfzhou added a comment to T6707: 'Attaching' two external accounts of the same type results in AphrontCountQueryException.

Facing the similar problem, but with two Google accounts.

Dec 26 2014, 3:01 AM · Wikimedia, Auth

Dec 17 2014

beeerd added a comment to T4806: Give installs more control over session expiry / prompt for reauth before taking security actions.

Our security team is driving the request, we have some strict compliance requirements around session expirations for internal services, so I'll be monkey patching this so long as it's unresolved.

Dec 17 2014, 10:31 PM · Restricted Project, Auth
epriestley added a comment to T4806: Give installs more control over session expiry / prompt for reauth before taking security actions.

@beeerd, do you have a specific problem / use case driving your interest?

Dec 17 2014, 6:25 PM · Restricted Project, Auth
beeerd added a comment to T4806: Give installs more control over session expiry / prompt for reauth before taking security actions.

Would love to have this feature!

Dec 17 2014, 5:52 PM · Restricted Project, Auth

Dec 16 2014

qgil moved T6707: 'Attaching' two external accounts of the same type results in AphrontCountQueryException from Backlog to Details on the Wikimedia board.
Dec 16 2014, 12:44 PM · Wikimedia, Auth
qgil added a project to T6707: 'Attaching' two external accounts of the same type results in AphrontCountQueryException: Wikimedia.
Dec 16 2014, 12:43 PM · Wikimedia, Auth
qgil added a comment to T6707: 'Attaching' two external accounts of the same type results in AphrontCountQueryException.

FWIW today we got a second report.

Dec 16 2014, 8:55 AM · Wikimedia, Auth

Dec 13 2014

epriestley added a comment to T5955: Refactor Conduit auth to be stateless, token-based, and support wire encodings.

At least for now, this no longer blocks Phacility or T2783.

Dec 13 2014, 6:28 PM · Restricted Project, Auth, Conduit
epriestley placed T5955: Refactor Conduit auth to be stateless, token-based, and support wire encodings up for grabs.
Dec 13 2014, 6:27 PM · Restricted Project, Auth, Conduit
epriestley added a revision to T5955: Refactor Conduit auth to be stateless, token-based, and support wire encodings: Restricted Differential Revision.
Dec 13 2014, 6:23 PM · Restricted Project, Auth, Conduit
epriestley added a revision to T5955: Refactor Conduit auth to be stateless, token-based, and support wire encodings: Restricted Differential Revision.
Dec 13 2014, 12:45 AM · Restricted Project, Auth, Conduit
epriestley added a revision to T5955: Refactor Conduit auth to be stateless, token-based, and support wire encodings: Restricted Differential Revision.
Dec 13 2014, 12:31 AM · Restricted Project, Auth, Conduit

Dec 12 2014

epriestley added a revision to T5955: Refactor Conduit auth to be stateless, token-based, and support wire encodings: Restricted Differential Revision.
Dec 12 2014, 11:29 PM · Restricted Project, Auth, Conduit
epriestley added a revision to T5955: Refactor Conduit auth to be stateless, token-based, and support wire encodings: Restricted Differential Revision.
Dec 12 2014, 11:15 PM · Restricted Project, Auth, Conduit
epriestley added a revision to T5955: Refactor Conduit auth to be stateless, token-based, and support wire encodings: Restricted Differential Revision.
Dec 12 2014, 9:41 PM · Restricted Project, Auth, Conduit
epriestley moved T5955: Refactor Conduit auth to be stateless, token-based, and support wire encodings from Backlog to v0 Closed Beta on the Phacility board.
Dec 12 2014, 6:05 PM · Restricted Project, Auth, Conduit
epriestley added a comment to T5955: Refactor Conduit auth to be stateless, token-based, and support wire encodings.

T2783 is at least partially blocked on this, since I need to make intra-cluster calls as the viewing user and don't want to do handshakes. I also don't really want to ship session keys across in the raw, although that wouldn't be the end of the world. I'll probably push this about halfway forward and then drop it again once T2783 is unblocked.

Dec 12 2014, 6:04 PM · Restricted Project, Auth, Conduit
epriestley claimed T5955: Refactor Conduit auth to be stateless, token-based, and support wire encodings.
Dec 12 2014, 6:00 PM · Restricted Project, Auth, Conduit

Dec 9 2014

epriestley added a comment to T6707: 'Attaching' two external accounts of the same type results in AphrontCountQueryException.

Aaaaah, okay. I'm guessing that will probably repro across other providers. Thanks!

Dec 9 2014, 10:40 PM · Wikimedia, Auth
FallingBullets added a comment to T6707: 'Attaching' two external accounts of the same type results in AphrontCountQueryException.

Hey, im the one who reported/suffers from this.

Dec 9 2014, 10:31 PM · Wikimedia, Auth
Krenair added a comment to T6707: 'Attaching' two external accounts of the same type results in AphrontCountQueryException.

I don't see any option to add a second MediaWiki account... In Wikimedia Phabricator I see MediaWiki and LDAP linked to one account each, then "Your account is linked with all available providers." under "Add External Account".

Dec 9 2014, 9:17 PM · Wikimedia, Auth
epriestley removed a project from T6707: 'Attaching' two external accounts of the same type results in AphrontCountQueryException: Support Impact.
Dec 9 2014, 5:35 PM · Wikimedia, Auth
epriestley added a comment to T6707: 'Attaching' two external accounts of the same type results in AphrontCountQueryException.

Well, maybe I'm wrong. I can't reproduce this with other account types. For example, I have no option to attach a second Facebook account:

Dec 9 2014, 5:34 PM · Wikimedia, Auth
epriestley renamed T6707: 'Attaching' two external accounts of the same type results in AphrontCountQueryException from 'Attaching' two external accounts from the same provider results in AphrontCountQueryException to 'Attaching' two Mediawiki accounts results in AphrontCountQueryException.
Dec 9 2014, 5:33 PM · Wikimedia, Auth
epriestley renamed T6707: 'Attaching' two external accounts of the same type results in AphrontCountQueryException from 'Attaching' two mediawiki accounts results in AphrontCountQueryException to 'Attaching' two external accounts from the same provider results in AphrontCountQueryException.
Dec 9 2014, 5:32 PM · Wikimedia, Auth
epriestley added a comment to T6707: 'Attaching' two external accounts of the same type results in AphrontCountQueryException.

In particular, this is very unlikely to be particular to Mediawiki, and should reproduce with other providers (like GitHub). (If it doesn't, there's just an issue we need to fix with the MW adapter.)

Dec 9 2014, 5:32 PM · Wikimedia, Auth

Dec 8 2014

epriestley added a project to T6707: 'Attaching' two external accounts of the same type results in AphrontCountQueryException: Support Impact.

Expected behavior is that this is prevented until T2549, which will eventually make this work properly.

Dec 8 2014, 11:39 AM · Wikimedia, Auth
epriestley edited projects for T6707: 'Attaching' two external accounts of the same type results in AphrontCountQueryException, added: Auth; removed OAuthServer.
Dec 8 2014, 11:38 AM · Wikimedia, Auth

Dec 6 2014

epriestley added a comment to T6703: Allow multiple copies of the same auth provider type.

@qgil That's T2549, but the exception is not expected behavior. If you want to file that, we can fix it -- it's supposed to prevent attaching a second account until we support multiple accounts properly.

Dec 6 2014, 3:49 PM · Auth
qgil added a comment to T6703: Allow multiple copies of the same auth provider type.

For what is worth, we have the case of users with two Wikimedia accounts, and some of them do try to attach them to the same Phabricator account. The result is 'Attaching' two mediawiki accounts results in AphrontCountQueryException.

Dec 6 2014, 3:39 PM · Auth
epriestley created T6703: Allow multiple copies of the same auth provider type.
Dec 6 2014, 3:30 PM · Auth

Dec 5 2014

qgil moved T6354: Register a new user redirect to a png file instead of email confirmation page from Important to Details on the Wikimedia board.
Dec 5 2014, 1:27 PM · Support Impact, Wikimedia, Auth

Dec 4 2014

qgil moved T6354: Register a new user redirect to a png file instead of email confirmation page from Backlog to Important on the Wikimedia board.
Dec 4 2014, 12:53 PM · Support Impact, Wikimedia, Auth

Nov 15 2014

dereckson added a comment to T6561: YubiKey support for authentication.

I concur, the interest is limited, and it clearly belongs to the category of narrow use cases.

Nov 15 2014, 12:45 AM · Auth

Nov 14 2014

chad closed T6561: YubiKey support for authentication as Wontfix.

Please see https://secure.phabricator.com/book/phabcontrib/article/contributing_code/. The main concern here is we're not likely to take this into the upstream (as in, support indefinitely) without a lot of interest. I've only seen 1-2 requests for this over the past few years.

Nov 14 2014, 9:48 PM · Auth
dereckson added a comment to T6561: YubiKey support for authentication.

@limanoit has added some days later a love token to Q36.

Nov 14 2014, 8:51 PM · Auth
dereckson created T6561: YubiKey support for authentication.
Nov 14 2014, 8:51 PM · Auth
anton.vladimirov added a comment to T6479: Manual account creation without Username+Password auth unclear to users and administrators.

Unless you specifically instructed the users who did click the links to immediately go to Settings > External Accounts and link their LDAP accounts, they're not really in better shape: they'll get locked out of their accounts as soon as they log out, too. Their LDAP credentials are not tied to their accounts. We never tie credentials together based on email addresses or usernames, because these aren't trustworthy across authentication sources.

Nov 14 2014, 12:35 AM · LDAP, Auth
epriestley added a comment to T6479: Manual account creation without Username+Password auth unclear to users and administrators.

Unless you specifically instructed the users who did click the links to immediately go to Settings > External Accounts and link their LDAP accounts, they're not really in better shape: they'll get locked out of their accounts as soon as they log out, too. Their LDAP credentials are not tied to their accounts. We never tie credentials together based on email addresses or usernames, because these aren't trustworthy across authentication sources.

Nov 14 2014, 12:23 AM · LDAP, Auth
anton.vladimirov added a comment to T6479: Manual account creation without Username+Password auth unclear to users and administrators.

This seems to have gotten a little bit too deep, but I might be misunderstanding something.

Nov 14 2014, 12:04 AM · LDAP, Auth

Nov 13 2014

epriestley added a comment to T6479: Manual account creation without Username+Password auth unclear to users and administrators.

In the long run, I think the ideal workflow here is to make import/create more of a function of providers -- so you enable LDAP and get "Import from LDAP" (users would recover by logging in with their LDAP credentials), enable Google and get "Import from Google" (users would recover by logging in with their Google credentials), enable Facebook/GitHub/Whatever and get "Import from list of accounts in a CSV file" or something (users would recover by logging in with their service credentials), and enable username/password and get "Create New Account" (users would recover via password reset -- I think they already get cued about this, maybe?). We could let you "Create New Account" anyway, maybe, but since you can do that via the CLI if you really intend to do advanced stuff it might be more straightforward to simply remove it from the web UI.

Nov 13 2014, 11:54 PM · LDAP, Auth
epriestley added a comment to T6479: Manual account creation without Username+Password auth unclear to users and administrators.

(That title might be a little clearer?)

Nov 13 2014, 11:50 PM · LDAP, Auth
epriestley renamed T6479: Manual account creation without Username+Password auth unclear to users and administrators from Recovering from expired LDAP account registration unclear to users and administrators to Manual account creation without Username+Password auth unclear to users and administrators.
Nov 13 2014, 11:50 PM · LDAP, Auth
epriestley added a comment to T6479: Manual account creation without Username+Password auth unclear to users and administrators.

On SAAS, I believe the current plan is zero LDAP support, so it's moot.

Nov 13 2014, 11:49 PM · LDAP, Auth
epriestley added a comment to T6479: Manual account creation without Username+Password auth unclear to users and administrators.

Possibly we should have some kind of warning on People > Create New User if password auth is not enabled, since it's a confusing/advanced feature in that case (users must follow the links promptly, must link an external account, can not recover credentials, and don't get UI prompting).

Nov 13 2014, 11:48 PM · LDAP, Auth
epriestley added a comment to T6479: Manual account creation without Username+Password auth unclear to users and administrators.

I don't have everything set up locally, but two general pieces of information:

Nov 13 2014, 11:47 PM · LDAP, Auth
btrahan updated subscribers of T6479: Manual account creation without Username+Password auth unclear to users and administrators.

I'm actually not familiar with the LDAP stuff as I don't have a working LDAP server. Adding in my cohorts though.

Nov 13 2014, 11:32 PM · LDAP, Auth
btrahan renamed T6479: Manual account creation without Username+Password auth unclear to users and administrators from The page for the expired "Welcome to Phabricator" email has a dead link to Recovering from expired LDAP account registration unclear to users and administrators.
Nov 13 2014, 11:30 PM · LDAP, Auth
anton.vladimirov added a comment to T6479: Manual account creation without Username+Password auth unclear to users and administrators.

We don't have username/password authentication enabled - just LDAP. Is that a bad idea?

Nov 13 2014, 11:18 PM · LDAP, Auth
chad triaged T6549: Backup codes for multi-factor authentication as Wishlist priority.
Nov 13 2014, 8:51 PM · Restricted Project, Auth
joshuaspence created T6549: Backup codes for multi-factor authentication.
Nov 13 2014, 8:16 PM · Restricted Project, Auth

Nov 12 2014

epriestley added a comment to T6354: Register a new user redirect to a png file instead of email confirmation page.

Er, not a 404, a permissions dialog.

Nov 12 2014, 2:51 AM · Support Impact, Wikimedia, Auth
epriestley added a comment to T6354: Register a new user redirect to a png file instead of email confirmation page.

Theory here is that the image isn't being created with the right permissions when we pull it from the authentication service. Then the user tries to load it, hits a 404 in the background, and gets a cookie set.

Nov 12 2014, 2:51 AM · Support Impact, Wikimedia, Auth

Nov 11 2014

chad added a project to T6354: Register a new user redirect to a png file instead of email confirmation page: Support Impact.

This has Support Impact

Nov 11 2014, 11:49 PM · Support Impact, Wikimedia, Auth

Nov 8 2014

btrahan added a comment to T6479: Manual account creation without Username+Password auth unclear to users and administrators.

Note it is possble to get a 400 error if you do not have the Username / Password auth enabled. If its a 400, see http://<domain/auth/ for your settings. If you don't have the Username / Password auth type enabled, turning it on should fix this workflow. That said, please let us know what the settings you do have here are as I don't think you should have fallen into this hole with any combination of settings.

Nov 8 2014, 7:10 AM · LDAP, Auth
btrahan added a comment to T6479: Manual account creation without Username+Password auth unclear to users and administrators.

Looks like we’re getting a 404 error for http://<domain>/login/email/

Nov 8 2014, 7:03 AM · LDAP, Auth

Nov 6 2014

qgil added a project to T6354: Register a new user redirect to a png file instead of email confirmation page: Wikimedia.

As we approach Wikimedia 's Bugzilla to Phabricator migration (planned to start on Nov 21), more users are registering to our instance and we hear more about "suitcase icon" cases. Today we got a user bothered enough to file a bug: https://phabricator.wikimedia.org/T1138

Nov 6 2014, 11:32 PM · Support Impact, Wikimedia, Auth
qgil updated the task description for T6354: Register a new user redirect to a png file instead of email confirmation page.
Nov 6 2014, 11:30 PM · Support Impact, Wikimedia, Auth
epriestley triaged T6484: Conduit API for SSH key retreival as Normal priority.
Nov 6 2014, 9:52 PM · Conduit, Auth
epriestley added a comment to T6484: Conduit API for SSH key retreival.

Context from IRC:

Nov 6 2014, 9:52 PM · Conduit, Auth
lfaraone updated subscribers of T6484: Conduit API for SSH key retreival.
Nov 6 2014, 9:49 PM · Conduit, Auth
lfaraone created T6484: Conduit API for SSH key retreival.
Nov 6 2014, 9:48 PM · Conduit, Auth
anton.vladimirov added a comment to T6479: Manual account creation without Username+Password auth unclear to users and administrators.

Looks like we’re getting a 404 error for http://<domain>/login/email/

Nov 6 2014, 9:24 PM · LDAP, Auth
btrahan raised the priority of T6479: Manual account creation without Username+Password auth unclear to users and administrators from Normal to Needs Triage.

Let us know what the URI is on the 404 page.

Nov 6 2014, 4:36 PM · LDAP, Auth
anton.vladimirov added a comment to T6479: Manual account creation without Username+Password auth unclear to users and administrators.

Hmm, nope - I was definitely seeing a 404 after clicking the button to send another email.
I'll try again tomorrow and see if I can get any more specific info.

Nov 6 2014, 3:35 AM · LDAP, Auth
btrahan added a comment to T6479: Manual account creation without Username+Password auth unclear to users and administrators.

fwiw I do not repro. my steps were

Nov 6 2014, 1:17 AM · LDAP, Auth

Nov 5 2014

chad triaged T6479: Manual account creation without Username+Password auth unclear to users and administrators as Normal priority.
Nov 5 2014, 8:39 PM · LDAP, Auth

Nov 3 2014

chad edited projects for T5310: Create oauth providers (and initial user) from CLI?, added: Auth; removed Phabricator.
Nov 3 2014, 2:55 AM · Auth
chad removed a project from T4842: Log when a user's SSH keys are used and what IP accessed them: Phabricator.
Nov 3 2014, 2:41 AM · Security, Auth, Diffusion
chad edited projects for T1205: Allow grey users in some form or other, added: Auth, People; removed Phabricator.
Nov 3 2014, 1:38 AM · Restricted Project, People, Auth, Wikimedia

Nov 2 2014

chad triaged T4137: user_externalaccount table goes out of sync when user try to login before adding it using "import from LDAP" as Low priority.
Nov 2 2014, 11:02 PM · Support Impact, LDAP, Auth