FrankYang0529 commented on code in PR #71842:
URL: https://github.com/apache/airflow/pull/71842#discussion_r3842718186
##########
providers/microsoft/azure/src/airflow/providers/microsoft/azure/hooks/msgraph.py:
##########
@@ -620,6 +615,27 @@ async def run(
return response
+ async def assert_allowed_host(self, url: str | None) -> None:
Review Comment:
Thanks for the review.
The connection already carries an `allowed_hosts` extra.
`get_allowed_hosts()` reads it and hands it to
`AzureIdentityAuthenticationProvider`, where it decides which hosts Kiota
attaches the bearer token to. The pagination guard can reuse `allowed_hosts`
field:
- `allowed_hosts` configured: a pagination link may point at any host on
that list.
- `allowed_hosts` empty: Kiota reads that as "any host", so the guard pins
to the endpoint's own netloc. The default does not change.
--
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]
