← Back to issue list

fix(i/snap): fix snap declaration assertion parsing

View original Github issue

Metadata

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

Current evaluation

Merged in #414 as commit a47e7d2, fixing snap declaration assertion parsing without requiring authority-id to be the second field; fixes #413.

Resolution: closed resolved

Reason: PR #414 was merged with approval and resolves #413. Commit a47e7d2 updated assertion detection and added tests covering valid field ordering and invalid authority-id cases.

Evaluated by: mr-cal-bot

No scores available.

Issue body

Don't require authority-id to be the second assertion field. Fixes #413 - [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 in #414 as commit a47e7d2, fixing snap declaration assertion parsing without requiring authority-id to be the second field; fixes #413.
stealth/space-bunny-alpha — closed resolved Merged fetch-service#414 as a47e7d2, fixing snap declaration assertion parsing so authority-id need not be the second field; addresses #413.
qwen/qwen3.6-35b-a3b — — Merged after resolving snap declaration assertion parsing by allowing authority-id to appear in any field position. Added validation test cases. Approved by two reviewers and passed CI.

Update history

Date Change
created

Related work

  • Caused By: fetch-service#413 (confidence 100%)

    PR #414 explicitly states that it fixes the assertion-parsing error reported in #413.

Related issues

Issue Project State Summary Similarity
#413 Error parsing snap declaration assertion fetch-service closed Fixed snap declaration assertion parsing in merged PR #414 (commit a47e7d2), allowing LXD snap declarations to be inspected.
75%
#250 feat(i/snap): add snap-declaration assertion filters fetch-service merged Merged in canonical/fetch-service#250 (commit 49d6a0d), adding snap-declaration header filters, including publisher-id allowlists, and fixing #249.
71%