Skip to content

[PM-33760] Unpin libnss3-tools#472

Merged
jprusik merged 1 commit intomainfrom
pm-33760
Mar 16, 2026
Merged

[PM-33760] Unpin libnss3-tools#472
jprusik merged 1 commit intomainfrom
pm-33760

Conversation

@jprusik
Copy link
Copy Markdown
Contributor

@jprusik jprusik commented Mar 16, 2026

🎟️ Tracking

PM-33760

📔 Objective

Because the Github action runner uses a newer version of libnss3-tools natively, a downgrade is required to meet the pinned version. We should be able to forgo this by removing the pin (it's only used for certutil).

@jprusik jprusik self-assigned this Mar 16, 2026
@github-actions
Copy link
Copy Markdown

Logo
Checkmarx One – Scan Summary & Detailsfc8116c4-9588-41fc-b11b-25f32918a69d

Great job! No new security vulnerabilities introduced in this pull request

@jprusik jprusik added the ai-review Request a Claude code review label Mar 16, 2026
@jprusik jprusik marked this pull request as ready for review March 16, 2026 15:54
@jprusik jprusik requested a review from a team as a code owner March 16, 2026 15:54
@jprusik jprusik requested a review from bensbits91 March 16, 2026 15:54
@claude
Copy link
Copy Markdown

claude bot commented Mar 16, 2026

Claude finished @jprusik's task in 1m 38s —— View job


I'll analyze this and get back to you.

@claude
Copy link
Copy Markdown

claude bot commented Mar 16, 2026

Overall Assessment: APPROVE

Reviewed the unpinning of libnss3-tools across all four workflow files. The change removes the version pin (2:3.98-1build1) that was forcing a downgrade on newer GitHub Action runners, and adds the -y flag for non-interactive CI installs. All four workflow files are updated consistently, and no references to the old pinned version remain in the repository.

Code Review Details

No findings. The change is minimal, consistent, and correctly scoped. The package comes from the official Ubuntu apt repository and is only used for certutil in the SSL certificate setup step.

@jprusik jprusik merged commit 53ca3fa into main Mar 16, 2026
20 of 22 checks passed
@jprusik jprusik deleted the pm-33760 branch March 16, 2026 16:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ai-review Request a Claude code review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants