Skip to content

Fix security 260128 #212

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
ldechamps wants to merge 14 commits into from
Closed

Fix security 260128 #212

ldechamps wants to merge 14 commits into from

Conversation

@ldechamps
Copy link
Contributor

@ldechamps ldechamps commented Jan 29, 2026 •
edited
Loading

Change description

Update package.json in differents packages and remove overrides in package.json root
Fix usage of pnpm 10
Update vite test
Fix type

Type of change

  • Bug fix (fixes an issue)
  • New feature (adds functionality)

Related issues

Fix various issues like
preact json injection
glob cli command injection
tar file overwrite
tar race condition
tar file creation

Checklists

Development

  • Lint rules pass locally
  • Application changes have been tested thoroughly
  • Automated tests covering modified code pass

Security

  • Security impact of change has been considered
  • Code follows company security practices and guidelines

Network

  • Changes to network configurations have been reviewed
  • Any newly exposed public endpoints or data have gone through security review

Code review

  • Pull request has a descriptive title and context useful to a reviewer. Screenshots or screencasts are attached as
    necessary
  • reviewers assigned
  • Pull request linked to task tracker where applicable

@ldechamps ldechamps self-assigned this Jan 29, 2026
@ldechamps ldechamps added the p1 High Priority label Jan 29, 2026
@ldechamps
Copy link
Contributor Author

ldechamps commented Feb 3, 2026

Outdated

@ldechamps ldechamps closed this Feb 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@louisdussarps louisdussarps Awaiting requested review from louisdussarps

1 more reviewer

@tmarcuzz tmarcuzz tmarcuzz approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@ldechamps ldechamps

Labels

p1 High Priority

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@ldechamps @tmarcuzz