← Back to issue list

fix(inspectors/git): allow packets containing only sideband

View original Github issue

Metadata

Project
fetch-service
Number
#467
Type
pull request
State
merged
Author
cmatsuoka
Labels
—
Created
Updated
Closed

Current evaluation

Merged into main as bb36f1e, allowing Git sideband-only packets and adding a regression test with a hand-crafted packet fixture.

Resolution: closed resolved

Reason: PR canonical/fetch-service#467 was approved with all CI checks passing and merged as commit bb36f1e.

Evaluated by: mr-cal-bot

No scores available.

Issue body

Each git packet starts with four bytes containing the the hex-encoded size and one sideband byte. A packet containing only the sideband byte is valid and contains 0005\01. - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? -----

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged into main as bb36f1e, allowing Git sideband-only packets and adding a regression test with a hand-crafted packet fixture.
stealth/space-bunny-alpha — closed resolved Merged PR #467 fixed Git inspector handling of valid sideband-only packets and added regression coverage. The fix shipped in v0.10.1 via commit bb36f1e.
stealth/space-bunny-alpha — closed resolved Merged PR #467 as bb36f1e, updating the Git inspector to accept sideband-only packets and adding a binary-crafted regression test.
stealth/space-bunny-alpha — closed resolved Merged #467 as commit bb36f1e, allowing the Git inspector to accept sideband-only packets and adding a regression test fixture.
qwen/qwen3.6-35b-a3b — — Merged a fix for the git inspector to accept packets containing only a sideband byte. Updated parsing logic and added a test case created via binary editing. Approved by three reviewers and passed all CI checks.

Update history

Date Change
created

Related issues

No related issues found above the similarity threshold.