Skip to content

Conversation

@moonchen
Copy link
Contributor

A recent change (#9090) broke parsing for the config key virtual_host which can be supplied without a value. This change fixes parsing for virtual_host, and adds it to autest.

A recent change broke parsing for the config key virtual_host which can
be supplied without a value.  This change fixes parsing for
virtual_host, and adds it to autest.
@ezelkow1
Copy link
Member

[approve ci]

@bryancall
Copy link
Contributor

[approve ci autest]

@bryancall bryancall requested a review from ywkaras September 26, 2022 23:15
@bryancall bryancall added this to the 10.0.0 milestone Sep 26, 2022
@ywkaras ywkaras merged commit d35a779 into apache:master Sep 30, 2022
zwoop pushed a commit that referenced this pull request Oct 4, 2022
A recent change broke parsing for the config key virtual_host which can
be supplied without a value.  This change fixes parsing for
virtual_host, and adds it to autest.

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

zwoop commented Oct 4, 2022

Cherry-picked to v9.2.x

@zwoop zwoop modified the milestones: 10.0.0, 9.2.0 Oct 4, 2022
Ftywan pushed a commit to Ftywan/trafficserver that referenced this pull request Dec 21, 2022
A recent change broke parsing for the config key virtual_host which can
be supplied without a value.  This change fixes parsing for
virtual_host, and adds it to autest.
masaori335 pushed a commit to masaori335/trafficserver that referenced this pull request Feb 21, 2023
* asf/9.2.x:
  Updated ChangeLog
  s3_auth: Fix parsing of virtual_host (apache#9103)
  s3_auth: accept longer config lines (apache#9090)
  Add a proxy.config.http.per_server.connection.max test (apache#9097)
  Move plugin_init for verify cmd (apache#9102)
  Fix unused-but-set-variable warnings by llvm-15 (apache#9106)
  Fix compile error with llvm-15 (apache#9105)
  Fix expected sha1sum for the Proxy Verifier binary (apache#9112)
  Updating AuTest to use Proxy Verifier v2.4.2 (apache#9110)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants