acoustic-carpenter-78188
03/09/2023, 5:49 AMrefreshTime based on the cached token expiry when instantiating customTokenSource. Otherwise refreshTime defaults to 0, effectively nullifying the cache we just added since it's always viewed as refresh window exceeded on first use.
Type
☑︎ Bug Fix
☐ Feature
☐ Plugin
Are all requirements met?
☑︎ Code completed
☑︎ Smoke tested
☑︎ Unit tests added
☐ Code documentation added
☐ Any pending items have an associated Issue
Complete description
This change
• Chooses a new refreshTime if a cached token is available, otherwise default to empty time
• Moves token source provider tests to dedicated file
• Adds new test TestCustomTokenSource_GetTokenSource
• Removes overly verbose logging
Tracking Issue
NA
Follow-up issue
NA
flyteorg/flyteidl
✅ All checks have passed
13/13 successful checksacoustic-carpenter-78188
03/10/2023, 1:39 AM