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 request body line parsing: Issue #2581 #3076
Conversation
Hi @cyburgee, Thank you for your contribution! We really value the time you've taken to put this together. Before we proceed with reviewing this pull request, please sign the Lightbend Contributors License Agreement: |
Thank you for your pull request! After a quick sanity check one of the team will reply with 'OK TO TEST' to kick off our automated validation on Jenkins. This compiles the project, runs the tests, and checks for things like binary compatibility and source code formatting. When two team members have also manually reviewed and (perhaps after asking for some amendments) accepted your contribution, it should be good to be merged. For more details about our contributing process, check out CONTRIBUTING.md - and feel free to ask! |
Test FAILed. !!! Couldn't read commit file !!! |
8ceb32c
to
00e5641
Compare
Rebased and fixed the test compilation error. |
akka-http-core/src/test/scala/akka/http/impl/engine/parsing/RequestParserSpec.scala
Outdated
Show resolved
Hide resolved
Test PASSed. |
...-http-tests/src/test/scala/akka/http/scaladsl/unmarshalling/MultipartUnmarshallersSpec.scala
Show resolved
Hide resolved
...-http-tests/src/test/scala/akka/http/scaladsl/unmarshalling/MultipartUnmarshallersSpec.scala
Show resolved
Hide resolved
Test PASSed. |
PLS BUILD |
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.
LGTM, thanks again, @cyburgee.
Test FAILed. Pull request validation reportFailed Test SuitesTest result for 'akka-http-core / Pr-validation / ./ executeTests'
|
Refs akka#2581 * Improved the heuristic for detecting end of line character(s) * Added tests for UndefinedEndOfLineConfiguration Co-authored-by: Johannes Rudolph <johannes.rudolph@gmail.com>
ccabcc1
to
428877e
Compare
Test FAILed. Pull request validation reportFailed Test SuitesTest result for 'akka-http-core / Pr-validation / ./ executeTests'
|
I try to reproduce the failures, these tests don't seem to fail in the backport (#3144). |
10.1 Backport of #3076 Refs #2581 * Improved the heuristic for detecting end of line character(s) * Added tests for UndefinedEndOfLineConfiguration Co-authored-by: Johannes Rudolph <johannes.rudolph@gmail.com> (cherry picked from commit 428877e) Co-authored-by: Collin Burger <collin.e.burger@gmail.com>
Should be rebuilt after #3145 is merged. |
PLS BUILD |
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.
LGTM.
PLS BUILD |
Test PASSed. |
Thanks a lot, @cyburgee. |
References #2581