Skip to content

chore(deps): bump tauri-plugin-shell to 2.2.1 (CVE-2025-31477) - #605

Closed
devin-ai-integration[bot] wants to merge 3 commits into
devfrom
devin/1789980942-tauri-shell-bump
Closed

devin-ai-integration[bot] wants to merge 3 commits into
devfrom
devin/1789980942-tauri-shell-bump

Conversation

@devin-ai-integration

@devin-ai-integration devin-ai-integration Bot commented Sep 21, 2026

Copy link
Copy Markdown

Summary

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Verification

  • Applied the exact Cargo.lock version/checksum change Dependabot computed for 2.2.1 (69d5eb33…); dependency list unchanged from 2.2.0.
  • Diff reviewed: only Cargo.toml pin + Cargo.lock version/checksum lines change; build-override profile preserved.

Checklist

  • Scope stays focused and avoids unrelated cleanup.
  • Docs or release notes were updated when needed.
  • Security-sensitive changes were reviewed for secrets, auth, and unsafe defaults.

Link to Devin session: https://app.devin.ai/sessions/18173a51bcf04824a2e66ca1d2e42ccf
Open in Devin Desktop: https://app.devin.ai/desktop/session/18173a51bcf04824a2e66ca1d2e42ccf?variant=devin
Requested by: @luvs01


Devin Review

Review readiness checklist

This PR stays in draft until every box below is ticked. Tick all four boxes once the requirements are met:

  • All CI tests are green on my local testing.

  • I pushed my PR to the latest dev commit.

  • I resolved all correct Codex and CodeRabbit findings.

  • My PR is ready for review.

Fixes CVE-2025-31477 (GHSA-c9pr-q8gx-3mgp): the shell plugin's `open`
endpoint accepted dangerous protocols (file:, smb:, nfs:) when scope
validation silently fell back to allow-all. Patch release; dependency
set identical to 2.2.0.

Supersedes Dependabot PR #570, which is unmergeable — it branched before
dev gained the [profile.release.build-override] block, so its Cargo.toml
diff deletes that profile on merge.

Co-Authored-By: Epinephrine <luvs01@hanmail.net>
@devin-ai-integration

Copy link
Copy Markdown
Author

I'll fix CI failures and address comments from users with write access. I'll skip comments containing "(aside)".

  • Disable automatic comment, CI, and merge conflict monitoring

@coderabbitai

coderabbitai Bot commented Sep 21, 2026

Copy link
Copy Markdown

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository: luvs01/opencodex/.coderabbit.yaml

Review profile: ASSERTIVE

Plan: Advanced

Run ID: 131cb52a-18f9-4866-bb52-fc1d9946b428

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Devin Review

@github-actions github-actions Bot added chore intake: hygiene-blocked Deterministic PR hygiene checks failed labels Sep 21, 2026
@github-actions

github-actions Bot commented Sep 21, 2026

Copy link
Copy Markdown

Deterministic PR hygiene checks passed.

@github-actions github-actions Bot removed the intake: hygiene-blocked Deterministic PR hygiene checks failed label Sep 21, 2026
@github-actions

github-actions Bot commented Sep 21, 2026

Copy link
Copy Markdown

✅ READY

  • all PR quality gates passed; the review readiness checklist is complete.

Review readiness checklist

  • ✅ All CI tests are green on my local testing.
  • ✅ I pushed my PR to the latest dev commit.
  • ✅ I resolved all correct Codex and CodeRabbit findings.
  • ✅ My PR is ready for review.

4/4 boxes ticked.

This pull request is already Ready for Review.
The review-ready label marks this PR as ready; review automation runs independently.
Maintainers: @lidge-jun @Ingwannu

@github-actions
github-actions Bot marked this pull request as draft September 21, 2026 09:24
@github-actions
github-actions Bot marked this pull request as ready for review September 21, 2026 12:33
@luvs01

luvs01 commented Sep 22, 2026

Copy link
Copy Markdown
Owner

Consolidated into lidge-jun#5525 as the existing upstream dependency update.

Source head: 184f8716f29fd87f593e7305ec9b05a974ea3f7a. Replacement head: d6dea8f246944677c8ce80264c66095b562e3deb.

The complete functional patch 43a43f58a9a0e4019ea5ad5af260b5191a4ffcdd has the same stable Git patch ID as upstream d6dea8f246944677c8ce80264c66095b562e3deb. Both Cargo.toml and Cargo.lock pin tauri-plugin-shell to 2.2.1 with checksum 69d5eb3368b959937ad2aeaf6ef9a8f5d11e01ffe03629d3530707bbcb27ff5d. Later source commits only synchronize dev or retrigger CI and have no additional contribution diff. The upstream tree also retains its newer macos-private-api feature; no functional fork change is omitted. Reusing the existing upstream PR avoids importing the same dependency update again. Build, platform CI and security review still belong to the upstream update; no merge or deployment is claimed.

Closing this duplicate standalone review entry as part of the requested consolidation after verifying coverage. This is not a merge or release claim; remaining integration checks and reviews are tracked on the replacement. Original branches are retained.

@luvs01 luvs01 closed this Sep 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant