Skip to content

Conversation

@masaori335
Copy link
Contributor

2188810 (#6781) changed the default value of proxy.config.ssl.handshake_timeout_in from 0 to 30 since 9.0.0. This follows the change on the docs.

@masaori335 masaori335 added this to the 10.0.0 milestone Jan 5, 2022
@masaori335 masaori335 self-assigned this Jan 5, 2022
@masaori335 masaori335 requested a review from bryancall as a code owner January 5, 2022 00:09
.. ts:cv:: CONFIG proxy.config.ssl.handshake_timeout_in INT 30
When enabled this limits the total duration for the server side SSL
When enabled this limits the total duration for the incoming side SSL
Copy link
Contributor

Choose a reason for hiding this comment

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

I'd prefer "incoming (requestor-facing)".

@zwoop zwoop merged commit f4d26aa into apache:master Jan 5, 2022
zwoop pushed a commit that referenced this pull request Jan 5, 2022
zwoop pushed a commit that referenced this pull request Jan 5, 2022
@zwoop
Copy link
Contributor

zwoop commented Jan 5, 2022

Cherry-picked to v9.1.x branch.
Cherry-picked to v9.2.x

@zwoop zwoop modified the milestones: 10.0.0, 9.1.2 Jan 5, 2022
@zwoop zwoop added the 9.2.0 label Jan 5, 2022
moonchen pushed a commit to moonchen/trafficserver that referenced this pull request Mar 17, 2022
* asf/9.2.x:
  Updated ChangeLog
  docs: fix fedora install notes and spelling issues (apache#8537)
  Docs: Fix default value of proxy.config.ssl.handshake_timeout_in (apache#8574)
  Partial of revert "Cleanup generated LDFLAGS for jemalloc (apache#8285)" (apache#8533)
  TSUserArg: add value type checking (apache#8550)
  Relax key validation of sni.yaml (apache#8549)
  Clear random header value by AIO read error (apache#8559)
  Fixes macOS arm64 builds (again) (apache#8556)
  Traffic Dump: Use the correct transaction user index (apache#8548)
  combo_handler: Initialize User Arg Index in TSRemapInit (apache#8551)
  backout down parent retry limiting in parent selection and nexthop (apache#8546)
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.

3 participants