Skip to content

Check and warn if CA certificate isn't marked as critical#804

Open
marv7000 wants to merge 1 commit into
uyuni-project:main-leap16from
marv7000:warn-if-not-critical
Open

Check and warn if CA certificate isn't marked as critical#804
marv7000 wants to merge 1 commit into
uyuni-project:main-leap16from
marv7000:warn-if-not-critical

Conversation

@marv7000
Copy link
Copy Markdown

@marv7000 marv7000 commented Jun 1, 2026

What does this PR change?

Issue a warning if a CA is not marked as critical. See https://datatracker.ietf.org/doc/html/rfc5280 for details.

Codespace

Check if you already have a running container clicking on Running CodeSpace

Create CodeSpace About billing for Github Codespaces CodeSpace Billing Summary CodeSpace Limit

Test coverage

  • No tests: already covered

  • DONE

Links

Issue(s): #

  • DONE

Changelogs

Make sure the changelogs entries you are adding are compliant with https://github.com/uyuni-project/uyuni/wiki/Contributing#changelogs and https://github.com/uyuni-project/uyuni/wiki/Contributing#uyuni-projectuyuni-repository

If you don't need a changelog check, please mark this checkbox:

  • No changelog needed

If you uncheck the checkbox after the PR is created, you will need to re-run changelog_test (see below)

Before you merge

Check How to branch and merge properly!

@mcalmer
Copy link
Copy Markdown
Contributor

mcalmer commented Jun 1, 2026

Double work, see: #803

@marv7000 marv7000 closed this Jun 2, 2026
@marv7000 marv7000 reopened this Jun 2, 2026
Comment thread shared/ssl/ssl.go Outdated
@marv7000 marv7000 force-pushed the warn-if-not-critical branch from f4d5b9e to 49c872b Compare June 2, 2026 08:38
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Jun 2, 2026

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