Skip to content

Bump vite in UI packages to patched releases#66798

Closed
arpitjain099 wants to merge 1 commit into
apache:mainfrom
arpitjain099:security/bump-vite-ui-packages
Closed

Bump vite in UI packages to patched releases#66798
arpitjain099 wants to merge 1 commit into
apache:mainfrom
arpitjain099:security/bump-vite-ui-packages

Conversation

@arpitjain099
Copy link
Copy Markdown
Contributor

Summary

  • Bump vite to patched ranges in both UI package manifests:
    • airflow-core/src/airflow/ui/package.json: ^5.4.17 -> ^5.4.18 (resolves to 5.4.21 in lockfile)
    • airflow-core/src/airflow/api_fastapi/auth/managers/simple/ui/package.json: ^6.2.6 -> ^6.4.2
  • Refresh related lockfiles:
    • both pnpm-lock.yaml files
    • package-lock.json in the simple auth UI package

Why

Dependabot flags multiple vulnerabilities in vulnerable Vite ranges used by these UI packages (including advisories patched in 5.4.18+ and 6.4.2+). This updates the direct dependency constraints and lockfiles to patched versions.

Validation

  • pnpm install --frozen-lockfile --ignore-scripts in airflow-core/src/airflow/ui
  • pnpm install --frozen-lockfile --ignore-scripts in airflow-core/src/airflow/api_fastapi/auth/managers/simple/ui
  • npm ci --ignore-scripts --legacy-peer-deps in airflow-core/src/airflow/api_fastapi/auth/managers/simple/ui

@boring-cyborg boring-cyborg Bot added area:API Airflow's REST/HTTP API area:UI Related to UI/UX. For Frontend Developers. labels May 12, 2026
@bbovenzi
Copy link
Copy Markdown
Contributor

Merge conflicts

@arpitjain099
Copy link
Copy Markdown
Contributor Author

@bbovenzi rebased and same story as #66797 - moot. Main has moved vite to ^8.0.8 and ^8.0.10 in the two UIs, while this PR was bumping to ^5.4.18 / ^6.4.2. Both bumps have been superseded. Closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:API Airflow's REST/HTTP API area:UI Related to UI/UX. For Frontend Developers.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants