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.

Sign up for GitHub

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

Jump to bottom

TempestaFW blocks GET/HEAD requests with Content-Type or Content-Length headers #2322

Open
2 tasks done
RomanBelozerov opened this issue Jan 13, 2025 · 1 comment · May be fixed by #2341
Open
2 tasks done

TempestaFW blocks GET/HEAD requests with Content-Type or Content-Length headers #2322

RomanBelozerov opened this issue Jan 13, 2025 · 1 comment · May be fixed by #2341
Assignees
@const-t
Labels
crucial enhancement good to start Start form this tasks if you're new in Tempesta FW
Milestone
0.8 - Beta

Comments

@RomanBelozerov
Copy link
Contributor

RomanBelozerov commented Jan 13, 2025 •
edited by const-t
Loading

Scope

I receive Warning in dmesg using TempestaFW as proxy for Jenkins with Blue Ocean plugin:

Warning: Content-Length or Content-Type not allowed to be used with such (overridden) method

A lot of apps can use these headers by default and TemepstaFW will be block them. This can be a problem.

My proposal:

  • remove the check or;
  • add a new directive for disabling the check (global/vhost/location);

Testing

  • add new tests or create a new issue in tempesta-test (probably we need to update DeproxyAutoParser)

Documentation

  • update wiki
@RomanBelozerov RomanBelozerov added enhancement good to start Start form this tasks if you're new in Tempesta FW labels Jan 13, 2025
@RomanBelozerov RomanBelozerov added this to the 0.9 - LA milestone Jan 13, 2025
@krizhanovsky
Copy link
Contributor

Required for S3 compatibility. We need a configuration option to switch on/off this feature

@krizhanovsky krizhanovsky modified the milestones: 1.0 - GA, 0.8 - Beta Feb 10, 2025
@const-t const-t linked a pull request Feb 11, 2025 that will close this issue
Content-Lenth and Content-Type in bodyless message #2341
Open
@krizhanovsky krizhanovsky linked a pull request Feb 17, 2025 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@const-t const-t

Labels
crucial enhancement good to start Start form this tasks if you're new in Tempesta FW
Projects
None yet
Milestone
0.8 - Beta
Development

Successfully merging a pull request may close this issue.

Content-Lenth and Content-Type in bodyless message tempesta-tech/tempesta
3 participants
@krizhanovsky @const-t @RomanBelozerov