Skip to content

Conversation

beberry
Copy link
Member

@beberry beberry commented Jul 15, 2025

Problem

The getPatreonUser function attempts to fetch user data as long
as access_token != ''. This check does not account for the access
token being null. Access token data can be set to null if a token
refresh failed. Because of this, the app might be attempting to refresh
user tokens even of no token is available.

Solution

Use empty which checks for both - null and ''.

@beberry beberry force-pushed the jstikans-fix-user-token-refresh branch 3 times, most recently from a705744 to 143a186 Compare July 15, 2025 01:15
@beberry beberry force-pushed the jstikans-fix-user-token-refresh branch from 143a186 to 35d0537 Compare July 15, 2025 01:17
@beberry beberry requested a review from nkouevda-patreon July 18, 2025 20:24
@beberry beberry marked this pull request as ready for review July 18, 2025 20:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants