Skip to content

Conversation

@maskit
Copy link
Member

@maskit maskit commented Aug 29, 2023

This adds support of OCSP requests by GET method. Requests by GET method enables caching their responses.

If proxy.config.ssl.ocsp.request_mode is set to 1, ATS uses GET method for OCSP requests if possible. If the encoded URL is going to be too long, it fallbacks to POST method. The setting is 0 by default (no behavior change).

maskit and others added 2 commits August 29, 2023 14:30
This adds proxy.config.ssl.ocsp.request_mode.

Co-authored-by: Jeff Elsloo <elsloo@apache.org>
Co-authored-by: Jeff Elsloo <elsloo@apache.org>
@maskit maskit added the TLS label Aug 29, 2023
@maskit maskit added this to the 10.0.0 milestone Aug 29, 2023
@maskit maskit self-assigned this Aug 29, 2023
@maskit
Copy link
Member Author

maskit commented Aug 30, 2023

[approve ci autest]
[approve ci rocky]

@bneradt
Copy link
Contributor

bneradt commented Sep 4, 2023

Looks good. Thanks for working on the OCSP fetch.

@maskit maskit merged commit 12e492f into apache:master Sep 11, 2023
cmcfarlen pushed a commit to cmcfarlen/trafficserver that referenced this pull request Jun 3, 2024
This adds support for OCSP requests by GET method. Requests by GET method enables caching their responses.

If proxy.config.ssl.ocsp.request_mode is set to 1, ATS uses GET method for OCSP requests if possible. If the encoded URL is going to be too long, it fallbacks to POST method. The setting is 0 by default (no behavior change).

* Add a setting to make OCSP request by GET method

This adds proxy.config.ssl.ocsp.request_mode.

Co-authored-by: Jeff Elsloo <elsloo@apache.org>

* Implement OCSP request by GET method

Co-authored-by: Jeff Elsloo <elsloo@apache.org>

* Include cstring header file

* Use ink_string instead

* Remove unnecessary static

* Remove a repeat

---------

Co-authored-by: Jeff Elsloo <elsloo@apache.org>
cmcfarlen pushed a commit to cmcfarlen/trafficserver that referenced this pull request Jun 3, 2024
* commit '236b749b2b3cc746829ad534a7034ab7799d1b71':
  Allow origins to do TLS renegotiation (apache#10385)
  Remove deprecated debug output functions from 21 source files. (apache#9683)
  Fixes some make test build problems (apache#10402)
  Removes unused Errata functions from WCCP (apache#10380)
  Move InkAPI.cc into src/api (apache#10315)
  cmake: Generate files in rc, install the trafficserver script (apache#10367)
  Add support for OCSP requests by GET method (apache#10306)
  Preserve unmapped url regardless of need for remapping (apache#10304)
  Add TSVConnFdGet api (apache#10324)
  include/ts: comma on all last enum elements (apache#10400)
  cmake: Add remaining plugins without external deps (apache#10395)
  CID-1508974 (apache#10397)
  CID-1508987 (apache#10398)
  Coverity 1518564: fix off by one (apache#10401)
@maskit maskit mentioned this pull request Aug 16, 2024
91 tasks
maskit added a commit to maskit/trafficserver that referenced this pull request Aug 20, 2024
maskit added a commit that referenced this pull request Aug 22, 2024
* Update ATS 10 documenation

This is for #10627 and #10306.

* Fix syntax error
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants