break, again.

carl-adams-planet · carl-adams-planet · commit 5dee276ee6e5 · 2025-05-07T14:56:36.000-07:00
diff --git a/src/planet_auth/oidc/request_authenticator.py b/src/planet_auth/oidc/request_authenticator.py
@@ -124,6 +124,9 @@ def _refresh_if_needed(self):
                     msg="Error refreshing auth token. Continuing with old auth token. Refresh error: " + str(e)
                 )
 
+        # FIXME?
+        # Why is this breaking the SDK CICD? Why only for the pipeline and not a local nox?
+        # The pipeline should be using a legacy API key, so why is this even being called?
         if not (self._credential and self._credential.is_loaded()):
             # "refresh" may also be called to initialize in some cases, as in client credentials flow.
             # Continuing with what we have is not an option when we have nothing.

Original file line number	Diff line number	Diff line change
`@@ -124,6 +124,9 @@ def _refresh_if_needed(self):`
`124`	`124`	`msg="Error refreshing auth token. Continuing with old auth token. Refresh error: " + str(e)`
`125`	`125`	`)`
`126`	`126`
	`127`	`+ # FIXME?`
	`128`	`+ # Why is this breaking the SDK CICD? Why only for the pipeline and not a local nox?`
	`129`	`+ # The pipeline should be using a legacy API key, so why is this even being called?`
`127`	`130`	`if not (self._credential and self._credential.is_loaded()):`
`128`	`131`	`# "refresh" may also be called to initialize in some cases, as in client credentials flow.`
`129`	`132`	`# Continuing with what we have is not an option when we have nothing.`