Skip to content

Update gix to 0.82 (with security fixes and hardened parsers)#16941

Merged
weihanglo merged 1 commit intorust-lang:masterfrom
Byron:update-gix
Apr 25, 2026
Merged

Update gix to 0.82 (with security fixes and hardened parsers)#16941
weihanglo merged 1 commit intorust-lang:masterfrom
Byron:update-gix

Conversation

@Byron
Copy link
Copy Markdown
Member

@Byron Byron commented Apr 25, 2026

This seems to have been an eventless update locally.

Tasks

  • pass CI

@Byron Byron marked this pull request as ready for review April 25, 2026 03:03
@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Apr 25, 2026
@rustbot
Copy link
Copy Markdown
Collaborator

rustbot commented Apr 25, 2026

r? @weihanglo

rustbot has assigned @weihanglo.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

Why was this reviewer chosen?

The reviewer was selected based on:

  • Owners of files modified in this PR: @ehuss, @epage, @weihanglo
  • @ehuss, @epage, @weihanglo expanded to ehuss, epage, weihanglo
  • Random selection from ehuss, epage, weihanglo

Comment thread Cargo.toml
Copy link
Copy Markdown
Member

@weihanglo weihanglo Apr 25, 2026

Choose a reason for hiding this comment

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

Thanks! Do we have any changelog that I can follow?

View changes since the review

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

That's difficult for the lack of one global changelog.

There is one per crate effectively, and this would be a way to surface all of them.

GitoxideLabs/gitoxide@gix-v0.81.0...gix-v0.82.0

Short version of this release is hardening and security fixes. The respective advisories are still something I have to publish.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

And I realize that the main gix release (v0.82) is quite usable for an overview:

https://github.com/GitoxideLabs/gitoxide/releases/tag/gix-v0.82.0

Now with a list of advisories.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Yeah I saw the info/exclude as was wonder if there is anything related to cargo package. Turns out that we dont even have test around this (and hope people dont publish under worktrees).

I'll need to look closer to those advisories over the weekend. Seems like some related to stable features Cargo relies on, like file traversing/listing?

Regardless, thank you for fixing all of them!!

@weihanglo weihanglo added this pull request to the merge queue Apr 25, 2026
Merged via the queue into rust-lang:master with commit 3c4678d Apr 25, 2026
31 checks passed
@rustbot rustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Apr 25, 2026
@epage
Copy link
Copy Markdown
Contributor

epage commented Apr 25, 2026

Is the underlying problem in the previous version fixed and the processes that led to it?

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants