Skip to content

Omit Trusted for deb822 sources when allow_unsigned is false - #1330

Open
bwitt wants to merge 1 commit into
puppetlabs:mainfrom
bwitt:deb822-trusted-false
Open

bwitt wants to merge 1 commit into
puppetlabs:mainfrom
bwitt:deb822-trusted-false

Conversation

@bwitt

@bwitt bwitt commented Oct 22, 2026

Copy link
Copy Markdown

Summary

Omit Trusted in the source file for deb822 sources when allow_unsigned is false

Add a spec test to cover this.

Related Issues (if any)

Fixes #1329

Checklist

  • 🟢 Spec tests.
  • 🟢 Acceptance tests.
  • Manually verified. (For example puppet apply)

Signed-off-by: Brian Witt <brianwitt@gmail.com>
@jst-cyr
jst-cyr force-pushed the deb822-trusted-false branch from 92b9acd to a7aaa4e Compare October 27, 2026 21:18
@jst-cyr

jst-cyr commented Oct 27, 2026

Copy link
Copy Markdown
Member

Hey @bwitt , looks good at first glance, I've rebased and will kick off the acceptance testing so we can get some platform feedback.

@jst-cyr

jst-cyr commented Oct 27, 2026

Copy link
Copy Markdown
Member

Running CI tests for this PR via #1335

@jst-cyr

jst-cyr commented Oct 28, 2026

Copy link
Copy Markdown
Member

✅ Acceptance Test Results: https://github.com/puppetlabs/puppetlabs-apt/actions/runs/40586353135?pr=1335

Green across the board! Good on testing, needs code review from engineering.

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.

deb822 does the wrong thing for allow_unsigned => false

2 participants