Allow null for token expiration date
ClosedPublic
Actions

Authored by avivey on Oct 31 2013, 12:18 AM.

Tags

None

Referenced Files

	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)
	Sat, Aug 17, 12:28 AM

	Unknown Object (File)
	Wed, Aug 14, 4:36 AM

	Unknown Object (File)
	Wed, Aug 14, 4:34 AM

	Unknown Object (File)
	Wed, Aug 14, 4:33 AM

Subscribers

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

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

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

Revision Contents
Changeset List

Path

Size

src/

applications/

auth/

provider/

PhabricatorAuthProviderOAuth.php

4 lines

Diff 16824

View Options

src/applications/auth/provider/PhabricatorAuthProviderOAuth.php

Allow null for token expiration date ClosedPublicActions