Skip to content
New issue

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

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

Already on GitHub? # to your account

[compress] Fix compressible-status-code config #12116

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

masaori335
Copy link
Contributor

The compressible-status-code config of compress plugin doesn't work as expected. Because the set of status code has default value but it's not cleared before insert. So setting like compressible-status-code 200,304 doesn't update the set.

std::set<TSHttpStatus> compressible_status_codes_ = {TS_HTTP_STATUS_OK, TS_HTTP_STATUS_PARTIAL_CONTENT,
TS_HTTP_STATUS_NOT_MODIFIED};

@masaori335 masaori335 added the compress compress plugin label Mar 21, 2025
@masaori335 masaori335 added this to the 10.2.0 milestone Mar 21, 2025
@masaori335 masaori335 self-assigned this Mar 21, 2025
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
compress compress plugin
Projects
Status: In progress
Status: No status
Development

Successfully merging this pull request may close these issues.

1 participant