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

Allow Helidon server to properly implement Expect: 100-continue protocol #4912

Closed
3 tasks done
janotav opened this issue Sep 19, 2022 · 1 comment · Fixed by #5714, #5912 or #5965
Closed
3 tasks done

Allow Helidon server to properly implement Expect: 100-continue protocol #4912

janotav opened this issue Sep 19, 2022 · 1 comment · Fixed by #5714, #5912 or #5965
Assignees
@danielkec
Labels
bug Something isn't working P2 webserver

Comments

@janotav
Copy link

janotav commented Sep 19, 2022 •
edited by danielkec
Loading

Environment Details

  • Helidon Version: 2.5.2
  • Helidon SE
  • JDK version: 11
  • OS: Linux

Problem Description

When request with header "Expect: 100-continue" is received by Helidon server, it automatically responds with "100 Continue".

The webserver implementation must have means to respond with "417 Expectation Failed" if that is appropriate in given context.

This ticket partially responds to the question I originally raised as #4881

PRs:
@janotav janotav mentioned this issue Oct 14, 2022
Closed
@m0mus m0mus added P2 bug Something isn't working labels Nov 10, 2022
@danielkec danielkec mentioned this issue Dec 16, 2022
Merged
@danielkec danielkec mentioned this issue Jan 19, 2023
Merged
@danielkec danielkec reopened this Jan 19, 2023
@danielkec danielkec linked a pull request Jan 19, 2023 that will close this issue
[2.x] 100 continue triggered by content request #5912
Merged
@danielkec danielkec mentioned this issue Jan 25, 2023
Merged
@danielkec danielkec linked a pull request Jan 25, 2023 that will close this issue
[4.x] 100 continue triggered by content request #5965
Merged
@danielkec danielkec mentioned this issue Feb 1, 2023
Closed
3 tasks
@danielkec
Copy link
Contributor

Fixed

@m0mus m0mus added this to Backlog Aug 12, 2024
@m0mus m0mus moved this to Closed in Backlog Aug 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@danielkec danielkec

Labels
bug Something isn't working P2 webserver
Projects
Backlog
Archived in project
Milestone
No milestone
3 participants
@danielkec @janotav @m0mus