Pedrinhonitz commented on code in PR #67059:
URL: https://github.com/apache/airflow/pull/67059#discussion_r3255423815
##########
airflow-core/newsfragments/67059.bugfix.rst:
##########
Review Comment:
Please revert or remove the fragment
##########
airflow-core/src/airflow/api_fastapi/execution_api/security.py:
##########
@@ -124,7 +124,7 @@ async def __call__( # type: ignore[override]
try:
claims = await validator.avalidated_claims(creds.credentials,
dict(self.required_claims))
except Exception as err:
- log.warning("Failed to validate JWT", exc_info=True,
token=creds.credentials)
+ log.warning("Failed to validate JWT", exc_info=True)
raise HTTPException(status_code=status.HTTP_403_FORBIDDEN,
detail=f"Invalid auth token: {err}")
Review Comment:
Would it make sense for these details to return something more generic?
Without the `err` variable?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
