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.

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

Fix body_limit middleware unit test #2145

Conversation

wsx864321
Copy link
Contributor

Hello, the four test items in Testbodylimit are actually tests on content length, not content read, and there is no difference between the first two test items and the latter two. After modification, the latter two test items can test content read.

@wsx864321
Copy link
Contributor Author

@aldas please review,If can, please help merge it,tks.

@wsx864321
Copy link
Contributor Author

@aldas Can this PR be merged?if not,I will close the PR,tks.

@aldas aldas merged commit 572466d into labstack:master Apr 5, 2022
15 checks passed
@aldas
Copy link
Contributor

aldas commented Apr 5, 2022

sorry for the delay

@wsx864321
Copy link
Contributor Author

sorry for the delay

That's OK!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants