-
Notifications
You must be signed in to change notification settings - Fork 81
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
RawPostDataException with 0.6.8 #67
Comments
Is this happening on all requests or just some? It feels like this may be a setup issue. |
I got this error too, the moment I used https://github.com/miki725/django-url-filter Based on an old post I added : Could you please investigate this Thanks. |
Can one of you provide a test case that causes the issue? Guessing something in #65 is causing the body to be read and we don't have a test covering the problem. |
@kennethjiang can you guys take a look at this to see if the last PR caused an error? |
I'm currently trying to do a test reproducing the error
…________________________________
De : Tom Clancy <[email protected]>
Envoyé : jeudi 2 mai 2019 23:11
À : Rhumbix/django-request-logging
Cc : dmod1; Author
Objet : Re: [Rhumbix/django-request-logging] RawPostDataException with 0.6.8 (#67)
@kennethjiang<https://github.com/kennethjiang> can you guys take a look at this to see if the last PR caused an error?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub<#67 (comment)>, or mute the thread<https://github.com/notifications/unsubscribe-auth/AEK6UDEGVZMTFLKOSUKBRLDPTNKHNANCNFSM4HJ3PWPA>.
|
Can't reproduce it easily in a clean project but I think the issue is when posting data including uploaded file |
I've managed to reproduce the bug - see example project here: https://github.com/therefromhere/django_request_logging_bug I think the django-request-logging tests need a bit of refactoring so they can accommodate an integration test like this? Using the example project: with
|
thank you. It seems that the version was upgraded because no version was specified in the "requirements.txt". But, I did not understand the cause that would occur when 0.6.9. sorry. |
Same exact issue. Attached trace stack below.
In my case, the request is a POST as multipart/form with a file upload. |
Sorry for missing your question @tclancy . We can't reproduce this problem here. |
@kennethjiang did you try using the example project I linked above? I think the bug is triggered by a POST that contains no form fields (eg of file only). |
Working on this now and figuring out how to fix it |
Still in 0.7.1 |
From my tests : I had the bug in 0.6.9, upgrading to 0.7.2 fixed it. |
Still in 0.7.3, problem doesn't exists in 0.7.2 |
We see this error when doing a post after upgrading from 0.6.7 to 0.6.8:
using django 2.2.1
The text was updated successfully, but these errors were encountered: