Skip to content

Document OCSP remap requirement#12952

Closed
bneradt wants to merge 2 commits intoapache:masterfrom
bneradt:ocsp-doc-update
Closed

Document OCSP remap requirement#12952
bneradt wants to merge 2 commits intoapache:masterfrom
bneradt:ocsp-doc-update

Conversation

@bneradt
Copy link
Copy Markdown
Contributor

@bneradt bneradt commented Mar 10, 2026

Document that ATS 10 and later fetch OCSP responses through
FetchSM instead of OpenSSL.

Add the remap.config requirement and a Digicert mapping example
to the OCSP stapling admin guide section so operators know they
must map the responder URL.

Document that ATS 10 and later fetch OCSP responses through
FetchSM instead of OpenSSL.

Add the remap.config requirement and a Digicert mapping example
to the OCSP stapling admin guide section so operators know they
must map the responder URL.
@bneradt bneradt added this to the 11.0.0 milestone Mar 10, 2026
@bneradt bneradt self-assigned this Mar 10, 2026
@bneradt bneradt added Documentation Security easy_review No new logic, easy to review. labels Mar 10, 2026
Add the ATS 10 migration note for OCSP stapling to the upgrade
notes as well as the admin guide.

This calls out the remap.config requirement for OCSP responder
URLs so operators upgrading from ATS 9 do not miss it.
@bneradt bneradt requested review from Copilot and maskit March 10, 2026 20:39

Before ATS 10, OCSP fetches were handled by OpenSSL. Starting in ATS 10,
|TS| performs the OCSP fetch itself through FetchSM, so the OCSP responder URL
must also be reachable through :file:`remap.config`. For example, if the
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This shouldn't be true.
#10146

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Ah, good point. OK, I'll close this PR. The remap.config rules shouldn't be needed for OCSP fetch.

Copy link
Copy Markdown
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.

Pull request overview

Documents the operational requirement for OCSP stapling in ATS 10+ where OCSP responses are fetched via FetchSM and therefore require an explicit remap.config mapping for the responder URL.

Changes:

  • Added an upgrade note calling out the new OCSP responder remap requirement for ATS 10+ with a DigiCert example.
  • Expanded the OCSP stapling admin guide to explain the OpenSSL → FetchSM behavior change and the required remap.config mapping.

Reviewed changes

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

File Description
doc/release-notes/upgrading.en.rst Adds an upgrade note describing remap requirements for OCSP responder access in ATS 10+.
doc/admin-guide/security/index.en.rst Documents FetchSM-based OCSP fetching in ATS 10+ and provides a remap.config example.

You can also share your feedback on Copilot code review. Take the survey.

Comment on lines +337 to +338
Before ATS 10, OCSP fetches were handled by OpenSSL. Starting in ATS 10,
|TS| performs the OCSP fetch itself through FetchSM, so the OCSP responder URL
Copy link

Copilot AI Mar 10, 2026

Choose a reason for hiding this comment

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

Line 338 has ||TS| (double pipe), which will render incorrectly in Sphinx and likely break the |TS| substitution. Replace it with |TS|.

Copilot uses AI. Check for mistakes.
Comment on lines 182 to +194
allocating iobuffers and cache volumes from hugepages if configured in the
system.
- The records.yaml entry ``proxy.config.plugin.compiler_path`` has been added to specify an optional compiler tool path for compiling plugins.
- If OCSP stapling is enabled, ATS 10 and later fetch OCSP responses through
FetchSM instead of OpenSSL. Add the OCSP responder URL from the certificate
to :file:`remap.config`. For example, a certificate that uses
``http://ocsp.digicert.com/`` needs the following mapping:

.. code-block:: none

map http://ocsp.digicert.com/ http://ocsp.digicert.com/

See :ref:`admin-ocsp-stapling` for more details.
Copy link

Copilot AI Mar 10, 2026

Choose a reason for hiding this comment

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

This new bullet is placed under 'The following records.yaml changes have been made', but it describes a remap.config requirement (not a records.yaml change). Consider moving this note to a more appropriate upgrade subsection (e.g., SSL/OCSP, or a general 'behavior changes' section) to avoid misleading readers.

Copilot uses AI. Check for mistakes.
@bneradt bneradt closed this Mar 10, 2026
@zwoop zwoop removed Security Documentation easy_review No new logic, easy to review. labels Mar 18, 2026
@zwoop zwoop removed this from the 11.0.0 milestone Mar 18, 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