Page MenuHomePhabricator
Differential D11298

OAuthServer - harden things up a bit
ClosedPublic
Actions

Authored by btrahan on Jan 9 2015, 6:57 PM.
Tags
None
Referenced Files
F13228557: D11298.diff
Mon, May 20, 10:24 AM
F13209855: D11298.id27146.diff
Fri, May 17, 2:34 AM
F13198814: D11298.id27146.diff
Mon, May 13, 10:05 AM
F13198390: D11298.id.diff
Mon, May 13, 6:07 AM
F13181939: D11298.id27147.diff
Thu, May 9, 5:10 PM
Unknown Object (File)
Tue, May 7, 7:22 AM
Unknown Object (File)
Fri, May 3, 6:01 AM
Unknown Object (File)
Thu, Apr 25, 6:00 AM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
Maniphest Tasks
T4593: Mitigate OAuth session theft
Commits
Restricted Diffusion Commit
rP152072fc974b: OAuthServer - harden things up a bit
Summary

This is the hardening work mentioned in T887#86529. Also take a documentation pass for accuracy about these changes and formatting. Ref T4593.

Test Plan

unit tests...! generated diviner docs and oauthserver doc looked good

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

btrahan updated this revision to Diff 27146.Jan 9 2015, 6:57 PM
btrahan retitled this revision from to OAuthServer - harden things up a bit.
btrahan updated this object.
btrahan edited the test plan for this revision. (Show Details)
btrahan added a reviewer: epriestley.
btrahan added a task: T4593: Mitigate OAuth session theft.
Herald added subscribers: epriestley, Korvin. · View Herald TranscriptJan 9 2015, 6:57 PM
epriestley accepted this revision.Jan 9 2015, 7:01 PM
epriestley edited edge metadata.

Awesome!

This revision is now accepted and ready to land.Jan 9 2015, 7:01 PM
Closed by commit rP152072fc974b: OAuthServer - harden things up a bit (authored by btrahan, committed by btrahan). · Explain WhyJan 9 2015, 7:04 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
oauthserver/
14 lines
__tests__/
18 lines
docs/
contributor/
48 lines

Diff 27147

View Options

src/applications/oauthserver/PhabricatorOAuthServer.php

Loading...
View Options

src/applications/oauthserver/__tests__/PhabricatorOAuthServerTestCase.php

Loading...
View Options

src/docs/contributor/using_oauthserver.diviner

Loading...
Phabricator · Built by Phacility