Bump axios in UI packages to patched release#66770
Conversation
|
Congratulations on your first Pull Request and welcome to the Apache Airflow community! If you have any issues or are unsure about any anything please check our Contributors' Guide
|
|
Thanks for the update. Since this change bumps axios to a patched version, have you validated if there are any behavioral changes impacting existing API calls, especially around error handling or interceptors? It might also be helpful to confirm compatibility across different UI environments. |
|
Merge conflicts |
Rebases the original PR onto current main (was ~30k commits behind). On main today, the simple-auth UI already pins axios at ^1.16.0, so the only remaining change is in airflow-core/src/airflow/ui/ which is still at ^1.15.2. Bumps that to ^1.16.0 and regenerates pnpm-lock.yaml. Signed-off-by: Arpit Jain <arpitjain099@gmail.com>
|
@bbovenzi rebased onto current main. On main today the simple-auth UI already pins |
3554c6d to
b63f388
Compare
|
Awesome work, congrats on your first merged pull request! You are invited to check our Issue Tracker for additional contributions. |
|
Thanks for the contribution :) |
|
Thank you! I appreciate the review and support. Glad to contribute to the project. |
Summary
axiosin both UI packages from^1.8.4to^1.16.0:airflow-core/src/airflow/ui/package.jsonairflow-core/src/airflow/api_fastapi/auth/managers/simple/ui/package.jsonpnpm-lock.yamlfilespackage-lock.jsonin simple auth UIWhy
Dependabot reports multiple open advisories on vulnerable axios ranges in these UI dependency manifests. Updating to a patched axios line addresses the shared vulnerability bucket across these locks.
Validation
pnpm install --frozen-lockfile --ignore-scriptsinairflow-core/src/airflow/uipnpm install --frozen-lockfile --ignore-scriptsinairflow-core/src/airflow/api_fastapi/auth/managers/simple/uinpm ci --ignore-scripts --legacy-peer-depsinairflow-core/src/airflow/api_fastapi/auth/managers/simple/ui