Skip to content

feat(response-rewrite): support filters#6750

Merged
spacewander merged 23 commits into
apache:masterfrom
kwanhur:feat-resp-filter
Apr 11, 2022
Merged

feat(response-rewrite): support filters#6750
spacewander merged 23 commits into
apache:masterfrom
kwanhur:feat-resp-filter

Conversation

@kwanhur

@kwanhur kwanhur commented Mar 29, 2022

Copy link
Copy Markdown
Contributor

Description

Support filters to do substitute.

Change: add property filters.

  • match pattern can be regex or fixed string
  • match mode can be once or global, default once
  • ngx.re.sub(gsub) default options jo

Fixes #5451

Checklist

  • I have explained the need for this PR and the problem it solves
  • I have explained the changes or the new features added to this PR
  • I have added tests corresponding to this change
  • I have updated the documentation to reflect this change
  • I have verified that this change is backward compatible (If not, please discuss on the APISIX mailing list first)

@kwanhur
kwanhur force-pushed the feat-resp-filter branch 3 times, most recently from ae7b211 to d6ae58d Compare March 30, 2022 10:45
* match pattern can be regex or fixed string
* match mode can be once or global, default once
* ngx.re.sub(gsub) default options jo

Signed-off-by: kwanhur <[email protected]>
@kwanhur
kwanhur marked this pull request as ready for review March 31, 2022 13:40
Comment thread apisix/plugins/response-rewrite.lua Outdated
Comment thread apisix/plugins/response-rewrite.lua Outdated
Comment thread docs/en/latest/plugins/response-rewrite.md Outdated
Comment thread t/plugin/response-rewrite.t Outdated
Comment thread apisix/plugins/response-rewrite.lua Outdated
Comment thread apisix/plugins/response-rewrite.lua
Comment thread apisix/plugins/response-rewrite.lua Outdated
Comment thread apisix/plugins/response-rewrite.lua Outdated
Comment thread apisix/plugins/response-rewrite.lua Outdated
Comment thread apisix/plugins/response-rewrite.lua Outdated
Comment thread docs/en/latest/plugins/response-rewrite.md Outdated
Comment thread t/plugin/response-rewrite2.t
@kwanhur
kwanhur requested a review from spacewander April 6, 2022 09:58
Comment thread apisix/plugins/response-rewrite.lua Outdated
Comment thread apisix/plugins/response-rewrite.lua Outdated
Comment thread apisix/plugins/response-rewrite.lua Outdated
Comment thread apisix/plugins/response-rewrite.lua Outdated
Comment thread docs/en/latest/plugins/response-rewrite.md Outdated
Comment thread t/plugin/response-rewrite2.t
Comment thread docs/en/latest/plugins/response-rewrite.md Outdated
Comment thread apisix/plugins/response-rewrite.lua Outdated
@kwanhur
kwanhur requested a review from tzssangglass April 9, 2022 10:52
@spacewander
spacewander merged commit e89ab8d into apache:master Apr 11, 2022
@kwanhur
kwanhur deleted the feat-resp-filter branch April 11, 2022 12:07
Liu-Junlin pushed a commit to Liu-Junlin/apisix that referenced this pull request May 20, 2022
spacewander added a commit that referenced this pull request Jun 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

request help: Does response-rewrite plugin support partial or regular substitution

3 participants