1Ninad commented on code in PR #62664:
URL: https://github.com/apache/airflow/pull/62664#discussion_r2884373320
##########
providers/google/src/airflow/providers/google/common/auth_backend/google_openid.py:
##########
@@ -135,12 +135,12 @@ def decorated(*args, **kwargs):
access_token = _get_id_token_from_request(flask_request)
if not access_token:
log.debug("Missing ID Token")
- return Response("Forbidden", 403)
+ return Response("Unauthorized", 401)
userid = _verify_id_token(access_token)
if not userid:
log.debug("Invalid ID Token")
- return Response("Forbidden", 403)
+ return Response("Unauthorized", 401)
Review Comment:
Hi,
no FAB version bump is needed for this change. It only updates the Google
OpenID auth backend response for missing/invalid token from 403 to 401 and does
not use any new FAB API. So older supported FAB versions (>=2.0.0) should
continue to work, and providers/google/pyproject.toml does not need an update.
--
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]
