<#381 Init customTokenSource.refreshTime> Pull req...
# flyte-github
a
#381 Init customTokenSource.refreshTime Pull request opened by andrewwdye TL;DR Follow up to #377. This change sets
refreshTime
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 checks