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

Setting limit_payload to disabled doesn't disable payload limit #1075

Closed
BWStearns opened this issue Dec 12, 2024 · 2 comments
Closed

Setting limit_payload to disabled doesn't disable payload limit #1075

BWStearns opened this issue Dec 12, 2024 · 2 comments
Assignees
@kaplanelad
Milestone
0.14.0

Comments

@BWStearns
Copy link
Contributor

Description

#1069

To Reproduce

Try this setting in env/development.yaml and try uploading a 4MB file.

  middlewares:
    # Limit payload size for incoming requests.
    limit_payload:
      enable: false

Expected Behavior

Should upload.

Environment:

Dev (though presumably any)

Additional Context

Seems to be failing back on the default from axum.
@BWStearns
Copy link
Contributor Author

Feel free to assign me.

@jondot jondot added this to the 0.14.0 milestone Dec 13, 2024
@kaplanelad kaplanelad assigned kaplanelad and unassigned BWStearns Dec 29, 2024
@BWStearns
Copy link
Contributor Author

I went to go work on this today and I couldn't reproduce it. Did either of you get it to repro?

@kaplanelad kaplanelad mentioned this issue Jan 2, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kaplanelad kaplanelad

Labels
None yet
Projects
None yet
Milestone
0.14.0
Development

No branches or pull requests
3 participants
@jondot @kaplanelad @BWStearns