Skip to content

Prevent bad ssl_multicert.config load from being swapped in#8515

Merged
randall merged 1 commit intoapache:masterfrom
randall:complete_multicert_load_error_fix
Nov 16, 2021
Merged

Prevent bad ssl_multicert.config load from being swapped in#8515
randall merged 1 commit intoapache:masterfrom
randall:complete_multicert_load_error_fix

Conversation

@randall
Copy link
Contributor

@randall randall commented Nov 12, 2021

retStatus contains the load status of the config file, but wasn't used
to check if it successful before being swapped in

This is a follow up to #8256

ret_status contains the load status of the config file, but wasn't used
to check if it successful before being swapped in

This is a follow up to apache#8256
@randall randall added this to the 10.0.0 milestone Nov 12, 2021
@randall randall self-assigned this Nov 12, 2021
@randall randall requested a review from bryancall as a code owner November 12, 2021 23:24
@bneradt
Copy link
Contributor

bneradt commented Nov 13, 2021

[approve ci]

@randall randall merged commit a329358 into apache:master Nov 16, 2021
@randall randall deleted the complete_multicert_load_error_fix branch November 16, 2021 00:24
zwoop pushed a commit that referenced this pull request Nov 17, 2021
retStatus contains the load status of the config file, but wasn't used
to check if it successful before being swapped in

This is a follow up to #8256

(cherry picked from commit a329358)
zwoop pushed a commit that referenced this pull request Nov 17, 2021
retStatus contains the load status of the config file, but wasn't used
to check if it successful before being swapped in

This is a follow up to #8256

(cherry picked from commit a329358)
@zwoop
Copy link
Contributor

zwoop commented Nov 17, 2021

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

@zwoop zwoop modified the milestones: 10.0.0, 9.1.2 Nov 17, 2021
@zwoop zwoop added the 9.2.0 label Nov 17, 2021
randall added a commit to randall/trafficserver that referenced this pull request Nov 19, 2021
With this test in place, we might've caught apache#8515 and apache#8256 earlier
randall added a commit to randall/trafficserver that referenced this pull request Nov 22, 2021
With this test in place, we might've caught apache#8515 and apache#8256 earlier
randall added a commit to randall/trafficserver that referenced this pull request Nov 23, 2021
With this test in place, we might've caught apache#8515 and apache#8256 earlier
randall added a commit that referenced this pull request Nov 23, 2021
With this test in place, we might've caught #8515 and #8256 earlier
zwoop pushed a commit that referenced this pull request Dec 1, 2021
With this test in place, we might've caught #8515 and #8256 earlier

(cherry picked from commit 7bc025c)
moonchen pushed a commit to moonchen/trafficserver that referenced this pull request Mar 17, 2022
* asf/9.2.x:
  Updated ChangeLog
  Marks tls_engine test as OpenSSL-only (apache#8509)
  Add skipped log entries to troubleshooting FAQ in docs. (apache#8507)
  Remove commented out EventProcessor::eventthread - it's time (apache#8517)
  Makes proxy_serve_stale version independent (apache#8508)
  Add proxy.config.http2.stream_error_sampling_threshold (apache#8506)
  url_sig: change cleanup msg from TSError to TSDebug (apache#8523)
  Adds the set-http-cntl operator for header_rewrite (apache#8375)
  Prevent bad ssl_multicert.config load from being swapped in (apache#8515)
moonchen pushed a commit to moonchen/trafficserver that referenced this pull request Jul 26, 2022
…8532)

With this test in place, we might've caught apache#8515 and apache#8256 earlier

(cherry picked from commit 7bc025c)
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