Skip to content

Upgrade website dependencies#2414

Merged
Gold856 merged 2 commits intoPhotonVision:mainfrom
samfreund:upgrade-website
Mar 27, 2026
Merged

Upgrade website dependencies#2414
Gold856 merged 2 commits intoPhotonVision:mainfrom
samfreund:upgrade-website

Conversation

@samfreund
Copy link
Copy Markdown
Member

@samfreund samfreund commented Mar 27, 2026

Description

This PR updates the website deps to use Vite 8 and rolldown. This is partially because of a security advisory, as we host the website on our VPS. This PR also adjusts links so they go straight to the file, instead of the public directory, which is unnecessary.

Meta

Merge checklist:

  • Pull Request title is short, imperative summary of proposed changes
  • The description documents the what and why, including events that led to this PR
  • If this PR changes behavior or adds a feature, user documentation is updated
  • If this PR touches photon-serde, all messages have been regenerated and hashes have not changed unexpectedly
  • If this PR touches configuration, this is backwards compatible with all settings going back to the previous seasons's last release (seasons end after champs ends)
  • If this PR touches pipeline settings or anything related to data exchange, the frontend typing is updated
  • If this PR addresses a bug, a regression test for it is added
  • If this PR adds a dependency, the license has been checked for compatibility and steps taken to follow it

@samfreund samfreund requested a review from a team as a code owner March 27, 2026 16:45
@github-actions github-actions Bot added the website anything relating to https://photonvision.org label Mar 27, 2026
@mcm001
Copy link
Copy Markdown
Contributor

mcm001 commented Mar 27, 2026

To be clear -- do the CVEs apply to statically hosted websites? We just serve static files so I don't see the threat vector

@samfreund
Copy link
Copy Markdown
Member Author

To be clear -- do the CVEs apply to statically hosted websites? We just serve static files so I don't see the threat vector

I think it probably doesn't apply to us? I'm not 100% confident tho which is why I mentioned it. Regardless, we do want Vite 8

@GrahamSH-LLK
Copy link
Copy Markdown
Contributor

To be clear -- do the CVEs apply to statically hosted websites? We just serve static files so I don't see the threat vector

They absolutely do not :)

@Gold856 Gold856 changed the title Upgrade website Upgrade website dependencies Mar 27, 2026
@Gold856 Gold856 merged commit 03ffcb1 into PhotonVision:main Mar 27, 2026
164 of 168 checks passed
@samfreund samfreund deleted the upgrade-website branch March 27, 2026 18:32
SpeedSlicer pushed a commit to SpeedSlicer/photonvision that referenced this pull request Mar 30, 2026
cswilson252 pushed a commit to cswilson252/photonvision that referenced this pull request Apr 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

website anything relating to https://photonvision.org

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants