SEPURI-SAI-KRISHNA commented on issue #71847:
URL: https://github.com/apache/airflow/issues/71847#issuecomment-5350989099
Verified the `microsoft.azure 15.0.0rc1` item for #71649 (Keep MSGraph
request configuration across paginated pages).
Setup: Airflow 3.3.1 on Python 3.12 with
`apache-airflow-providers-microsoft-azure==15.0.0rc1` installed from PyPI into
a clean venv, compared against `14.1.0` (the previous stable) in a second venv.
**Functional check.** Built an `MSGraphAsyncOperator` carrying a full
request configuration (`headers`, `data`, `scopes`), fed it a first page ending
in `@odata.nextLink`, and inspected the `MSGraphTrigger` the operator defers to
for page 2:
| field | 14.1.0 | 15.0.0rc1 |
|---|---|---|
| `headers` | `None` ❌ | `{'Prefer': 'outlook.timezone="Europe/Brussels"',
'ConsistencyLevel': 'eventual'}` ✅ |
| `data` | `None` ❌ | `{'filter': "displayName eq 'x'"}` ✅ |
| `scopes` | `None` ❌ | `['https://graph.microsoft.com/.default']` ✅ |
| `method` | `GET` ✅ | `GET` ✅ |
| `conn_id` | `msgraph_api` ✅ | `msgraph_api` ✅ |
So on 14.1.0 the second and later pages were issued without the caller's
headers, body, and scopes, which is what the fix addresses. On the RC all five
survive the pagination hand-off.
**Test suite.** Ran the provider's MSGraph unit tests (operators, triggers,
hooks) from `main` against the installed RC wheel: **92 passed, 1 xfailed**.
Also diffed the shipped `15.0.0rc1` wheel against `main`,
`operators/msgraph.py` matches, so the release artifact carries the merged
change.
Addressed ✅
--
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]
