Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

header_rewrite rm-destination support #8025

Merged
merged 1 commit into from
Jul 1, 2021

Conversation

ezelkow1
Copy link
Member

@ezelkow1 ezelkow1 commented Jul 1, 2021

Adds a new rm-destination, this lets you specify either QUERY or PATH, and be able to drop them from the incoming request

closes #8024
closes #4670

@ezelkow1 ezelkow1 requested review from zwoop and randall July 1, 2021 16:34
@ezelkow1 ezelkow1 self-assigned this Jul 1, 2021
@ezelkow1 ezelkow1 requested a review from bryancall as a code owner July 1, 2021 16:34
@ezelkow1 ezelkow1 changed the title Hdrrwr drop query Hdrrwr rm-destination support Jul 1, 2021
…, and be able to drop them from the incoming request

closes apache#8024
closes apache#4670
@ezelkow1 ezelkow1 force-pushed the hdrrwr-drop-query branch from 1004f9e to 717c9e3 Compare July 1, 2021 16:58
@randall randall changed the title Hdrrwr rm-destination support header_rewrite rm-destination support Jul 1, 2021
Copy link
Contributor

@zwoop zwoop left a comment

Choose a reason for hiding this comment

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

Love it!

@zwoop zwoop added this to the 10.0.0 milestone Jul 1, 2021
@ezelkow1 ezelkow1 merged commit 3ca5eba into apache:master Jul 1, 2021
@ezelkow1 ezelkow1 deleted the hdrrwr-drop-query branch July 1, 2021 18:20
@zwoop zwoop modified the milestones: 10.0.0, 9.2.0 Sep 23, 2021
moonchen pushed a commit to moonchen/trafficserver that referenced this pull request Mar 7, 2022
* asf/master: (763 commits)
  rate_limit: Add a global hook to rate limit concurrent connections based on SNI (apache#8021)
  Fix uri_signing unit test for out of source builds (apache#8040)
  tests: Add conditions for BoringSSL and OpenSSL (apache#8045)
  change debug tags and make sure sni is printed on certain logs (apache#7673)
  Doc build in CI: build English docs with -W (apache#8039)
  When loading async SSL configuration file fails, log SSL error (apache#8036)
  Doc build: treat warnings as errors only by default (apache#8038)
  For test async_engine, export all symbols (apache#8037)
  Fix the server cert reload (apache#8030)
  Treat Sphinx doc build warnings as errors. (apache#8033)
  Stablize trace curl test in good_request_after_bad (apache#8032)
  Doc: Update documentation to build cleanly in Sphinx 3. Require Sphinx 3 or better. (apache#7978)
  Docs: Fix pre-formatting for ratelimit plugin (apache#7986)
  Make it slightly harder to dump private keys to logs (apache#8029)
  tls_bad_alpn: Add an openssl version skip check (apache#8026)
  per thread jemalloc arena for MADV_DONTDUMP (apache#7501)
  Adds a new rm-destination, this lets you specify either QUERY or PATH, and be able to drop them from the incoming request (apache#8025)
  Fix HPACK eviction iterator manipulation (apache#8004)
  Do not invalidate cached resources upon error responses to unsafe methods (apache#7999)
  Cleanup SSLUtils (apache#8007)
  ...
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.

Add rm-destinations for various parts in header rewrite Allow header_rewrite to remove query parameters
2 participants