-
-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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 getContent so body content stream can be accessed on POST method #17425
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
L424 still only mentions put
in the exception message
Signed-off-by: Pavel Krasikov <klonishe@gmail.com>
exception message Signed-off-by: Pavel Krasikov <klonishe@gmail.com>
Signed-off-by: Pavel Krasikov <klonishe@gmail.com>
Method __get get called before controller execution to get parameters for controller method. Therefore, I think that in switch cases params and parameters, only an parsed array is needed. |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
@xklonx can you rebase and squash all commits into one please? |
Closing for inactivity. Code needs to be adjusted to current code base, but it should be pretty easy. If someone is interesting in it, open a new PR with the modifications. |
I need to get file from body of POST request, but there is no way to get Request->inputStream so i think this fix is correct.