feat(gzip): add via control option in gzip plugin #9024
Conversation
|
The CI error may not be caused by me. |
| function _M.rewrite(conf, ctx) | ||
| if conf.disable_via == true then | ||
| -- According to nginx source code, via header will cause gzip not work | ||
| core.request.set_header(ctx, "via", nil) |
There was a problem hiding this comment.
If this can satisfy your requirement, why not just use proxy-rewrite plugin to do so?
There was a problem hiding this comment.
I just think that this is a control that the gzip plugin lacks, and there is no need to introduce additional plugins. (of course, proxy-rewrite can also be implemented)
|
This pull request has been marked as stale due to 60 days of inactivity. It will be closed in 4 weeks if no further activity occurs. If you think that's incorrect or this pull request should instead be reviewed, please simply write any comment. Even if closed, you can still revive the PR at any time or discuss it on the dev@apisix.apache.org list. Thank you for your contributions. |
|
This pull request/issue has been closed due to lack of activity. If you think that is incorrect, or the pull request requires review, you can revive the PR at any time. |
Description
Fixes #8995
Checklist