Skip to content

Conversation

@bneradt
Copy link
Contributor

@bneradt bneradt commented Apr 11, 2025

Delay OCSP fetch until FetchSM is initialized. This avoids noisy OCSP error messages on ATS initialization that result when the FetchSM calls fail each attempted OCSP cert fetch.

Fixes: #9819

@bneradt bneradt added the TLS label Apr 11, 2025
@bneradt bneradt added this to the 10.2.0 milestone Apr 11, 2025
@bneradt bneradt requested a review from Copilot April 11, 2025 20:21
@bneradt bneradt self-assigned this Apr 11, 2025
@bneradt bneradt requested a review from maskit April 11, 2025 20:21
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

@bneradt bneradt force-pushed the quiet_ocsp_startup_errors_os branch 2 times, most recently from 55580a0 to 055cd8c Compare April 11, 2025 21:24
Delay OCSP fetch until FetchSM is initialized. This avoids noisy OCSP
error messages on ATS initialization that result when the FetchSM calls
fail each attempted OCSP cert fetch.

Fixes: apache#9819
@bneradt bneradt force-pushed the quiet_ocsp_startup_errors_os branch from 055cd8c to cbe7fe5 Compare April 11, 2025 21:26
@bneradt bneradt merged commit 9b571e4 into apache:master Apr 11, 2025
15 checks passed
@github-project-automation github-project-automation bot moved this to For v10.1.0 in ATS v10.1.x Apr 11, 2025
@bneradt bneradt deleted the quiet_ocsp_startup_errors_os branch April 11, 2025 22:33
@cmcfarlen cmcfarlen moved this from For v10.1.0 to picked v10.1.0 in ATS v10.1.x Apr 14, 2025
@cmcfarlen cmcfarlen modified the milestones: 10.2.0, 10.1.0 Apr 14, 2025
@cmcfarlen
Copy link
Contributor

Cherry-picked to 10.1.x branch

cmcfarlen pushed a commit that referenced this pull request Apr 14, 2025
Delay OCSP fetch until FetchSM is initialized. This avoids noisy OCSP
error messages on ATS initialization that result when the FetchSM calls
fail each attempted OCSP cert fetch.

Fixes: #9819
(cherry picked from commit 9b571e4)
bneradt pushed a commit to bneradt/trafficserver that referenced this pull request Apr 15, 2025
Delay OCSP fetch until FetchSM is initialized. This avoids noisy OCSP
error messages on ATS initialization that result when the FetchSM calls
fail each attempted OCSP cert fetch.

Fixes: apache#9819

Cherry picking this from:
apache#12185
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: picked v10.1.0

Development

Successfully merging this pull request may close these issues.

10.0.x: OCSP errors on startup

3 participants