Skip to content

feat(response-rewrite): support brotli when use filters.regex option#10733

Merged
juststillthinking merged 4 commits into
apache:masterfrom
yuweizzz:feat/resp-rewrite-brotli
Jan 4, 2024
Merged

feat(response-rewrite): support brotli when use filters.regex option#10733
juststillthinking merged 4 commits into
apache:masterfrom
yuweizzz:feat/resp-rewrite-brotli

Conversation

@yuweizzz

Copy link
Copy Markdown
Contributor

Description

Related: #10637 and #10562

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)

@yuweizzz yuweizzz changed the title feature(response-rewrite): support brotli when use filters.regex option feat(response-rewrite): support brotli when use filters.regex option Dec 29, 2023
Comment thread apisix/plugins/response-rewrite.lua
Comment thread apisix/plugins/response-rewrite.lua Outdated
Comment thread t/plugin/response-rewrite3.t
@juststillthinking

Copy link
Copy Markdown
Contributor

please make the ci pass

@juststillthinking juststillthinking added the wait for update wait for the author's response in this issue/PR label Jan 2, 2024
@yuweizzz

yuweizzz commented Jan 3, 2024

Copy link
Copy Markdown
Contributor Author

I've reviewed the ci logs and the failure was due to a network error, could you please rerun the ci?

@juststillthinking
juststillthinking merged commit ab67b09 into apache:master Jan 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

user responded wait for update wait for the author's response in this issue/PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants