Skip to content
This repository has been archived by the owner on Dec 9, 2024. It is now read-only.

fix: Check for valid entries in token cache remove and find #244

Merged
merged 1 commit into from
Aug 16, 2019

Conversation

tbarlow12
Copy link
Contributor

Fails if user previously started interactive login process without finishing because it writes an invalid JSON file. This checks to make sure the parsed JSON is valid and that entries exists

@coveralls
Copy link

Coverage Status

Coverage increased (+0.3%) to 84.68% when pulling 41bf829 on tabarlow/fix-more-token-cache into 9e3fcac on dev.

@pjlittle
Copy link
Contributor

looks good 👍

Copy link
Contributor

@PIC123 PIC123 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, nice catch!

@tbarlow12 tbarlow12 merged commit 660f8b7 into dev Aug 16, 2019
@tbarlow12 tbarlow12 deleted the tabarlow/fix-more-token-cache branch August 16, 2019 23:12
tbarlow12 added a commit that referenced this pull request Sep 13, 2019
Fails if user previously started interactive login process without finishing because it writes an invalid JSON file. This checks to make sure the parsed JSON is valid and that entries exists
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants