Page MenuHomePhabricator

Allow null for token expiration date
ClosedPublic

Authored by avivey on Oct 31 2013, 12:18 AM.
Tags
None
Referenced Files
F13808685: D7466.diff
Tue, Sep 17, 1:42 AM
Unknown Object (File)
Wed, Sep 4, 10:13 AM
Unknown Object (File)
Tue, Sep 3, 6:28 PM
Unknown Object (File)
Tue, Aug 27, 11:11 PM
Unknown Object (File)
Wed, Aug 21, 12:37 PM
Unknown Object (File)
Aug 17 2024, 12:28 AM
Unknown Object (File)
Aug 14 2024, 4:36 AM
Unknown Object (File)
Aug 14 2024, 4:34 AM

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Commits
Restricted Diffusion Commit
rP2250ee6aa6b8: Allow null for token expiration date
Summary

At least under GitHub, the token value is stored as "null", and not missing. And null > anything is false, so Phabricator thinks the token is expired or not there.

Test Plan

http://ph.vm/settings/panel/external/ before shows "No OAuth Access Token," and after it says "Active OAuth Token".

Diff Detail

Branch
local
Lint
Lint Passed
Unit
No Test Coverage

Event Timeline

epriestley closed this revision.

Closed by commit rP2250ee6aa6b8 (authored by Aviv Eyal <aeyal@groupon.com>, committed by @epriestley).