Page MenuHomePhabricator
Differential D11357

People - add application policy on user creation
ClosedPublic
Actions

Authored by btrahan on Jan 12 2015, 10:44 PM.
Tags
None
Referenced Files
F14833690: D11357.id27289.diff
Thu, Jan 30, 11:56 AM
Unknown Object (File)
Mon, Jan 27, 10:59 PM
Unknown Object (File)
Sun, Jan 26, 9:22 PM
Unknown Object (File)
Thu, Jan 23, 9:32 PM
Unknown Object (File)
Thu, Jan 23, 9:32 PM
Unknown Object (File)
Thu, Jan 23, 9:32 PM
Unknown Object (File)
Thu, Jan 23, 9:31 PM
Unknown Object (File)
Thu, Jan 23, 8:26 AM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
Maniphest Tasks
T4137: user_externalaccount table goes out of sync when user try to login before adding it using "import from LDAP"
T6947: Add configuration to lock auth/user creation, or to lock application policies
Commits
Restricted Diffusion Commit
rPe85dfcbeeeb3: People - add application policy on user creation
Summary

Ref T6947.

Test Plan

made the setting say only admin user a and noted admin user b lost access

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

btrahan updated this revision to Diff 27286.Jan 12 2015, 10:44 PM
btrahan added a task: T6947: Add configuration to lock auth/user creation, or to lock application policies.
btrahan retitled this revision from to People - add application policy on user creation.
btrahan updated this object.
btrahan edited the test plan for this revision. (Show Details)
btrahan added a reviewer: epriestley.
Herald added subscribers: epriestley, Korvin. · View Herald TranscriptJan 12 2015, 10:44 PM
epriestley accepted this revision.Jan 12 2015, 10:52 PM
epriestley edited edge metadata.
epriestley added a task: T4137: user_externalaccount table goes out of sync when user try to login before adding it using "import from LDAP".

Might need this check in PhabricatorPeopleLdapController too.

Or we could just delete it and close T4137 until T5953.

This revision is now accepted and ready to land.Jan 12 2015, 10:52 PM
btrahan updated this revision to Diff 27288.Jan 12 2015, 11:15 PM
btrahan edited edge metadata.

add the check to ldap controller

Closed by commit rPe85dfcbeeeb3: People - add application policy on user creation (authored by btrahan, committed by btrahan). · Explain WhyJan 12 2015, 11:18 PM
This revision was automatically updated to reflect the committed changes.
• lokinell added a task: T5375: Tokenizers fail in Firefox on Android.Jan 13 2015, 5:20 AM
• lokinell removed a task: T5375: Tokenizers fail in Firefox on Android.

Revision Contents
Changeset List

Diff 27289

View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/people/application/PhabricatorPeopleApplication.php

Loading...
View Options

src/applications/people/capability/PeopleCreateUsersCapability.php

Loading...
View Options

src/applications/people/controller/PhabricatorPeopleController.php

Loading...
View Options

src/applications/people/controller/PhabricatorPeopleCreateController.php

Loading...
View Options

src/applications/people/controller/PhabricatorPeopleLdapController.php

Loading...
View Options

src/applications/people/controller/PhabricatorPeopleNewController.php

Loading...
Phabricator · Built by Phacility