Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

config: update minimum TLS version for server #1945

Merged
merged 1 commit into from
Jan 5, 2024

Conversation

crozzy
Copy link
Collaborator

@crozzy crozzy commented Jan 4, 2024

The config is used both when setting up the HTTP API and for the notifier's Deliverer.

@crozzy crozzy requested a review from a team as a code owner January 4, 2024 23:31
@crozzy crozzy requested review from hdonnay and removed request for a team January 4, 2024 23:31
Copy link
Member

@hdonnay hdonnay left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

The config is used both when setting up the HTTP API and
for the notifier's Deliverer.

Signed-off-by: crozzy <[email protected]>
@crozzy crozzy merged commit 33a7743 into quay:main Jan 5, 2024
13 checks passed
hdonnay pushed a commit to hdonnay/clair that referenced this pull request Feb 14, 2024
The config is used both when setting up the HTTP API and
for the notifier's Deliverer.

Backports: quay#1945
Signed-off-by: Hank Donnay <[email protected]>
Signed-off-by: crozzy <[email protected]>
(cherry picked from commit 18aa1b3)
hdonnay pushed a commit to hdonnay/clair that referenced this pull request Feb 26, 2024
The config is used both when setting up the HTTP API and
for the notifier's Deliverer.

Backports: quay#1945
Signed-off-by: Hank Donnay <[email protected]>
Signed-off-by: crozzy <[email protected]>
(cherry picked from commit 18aa1b3)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

2 participants