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

fixing KieServerHttpRequestTest.getWithResponseCharset test #1891

Closed

Conversation

mbiarnes
Copy link
Contributor

No description provided.

@mbiarnes mbiarnes requested a review from mareknovotny August 19, 2019 14:57
@mbiarnes
Copy link
Contributor Author

Copy link
Member

@mareknovotny mareknovotny left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks ok

mareknovotny and others added 2 commits August 20, 2019 11:05
jetty upgrade and fixing org.kie.server.common.rest.KieServerHttpRequestTest.numberHeader() failure
@mbiarnes
Copy link
Contributor Author

@mareknovotny
Copy link
Member

mareknovotny commented Aug 21, 2019

ok, I got it now. So null in numberHeader method is treated with invalid character. I am not sure why I can't see it locally running test. There is the warning in the log:

2019-08-20 05:45:16,793 [qtp2042495840-15] WARN  Illegal character 0xA in state=HEADER_IN_NAME for buffer HeapByteBuffer@181a1f75[p=27,l=166,c=8192,r=139]={GET / HTTP/1.1\r\nh1: 5\r\nh2\r\n<<<User-Agent: Java/... keep-alive\r\n\r\n>>> keep-alive\r\n\r\n, ...\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00}
2019-08-20 05:45:16,796 [qtp2042495840-15] WARN  bad HTTP parsed: 400 Illegal character 0xA for HttpChannelOverHttp@64d42b6a{r=46,c=false,a=IDLE,uri=null}

@mareknovotny mareknovotny self-requested a review August 21, 2019 15:40
@mareknovotny
Copy link
Member

I am not sure why it locally works, but I found this jetty/jetty.project#1116 which seems as jetty had some problems with null HTTP headers, but still it is a little bit hidden for me it is showing Illegal character 0xA which is LF line feed in empty HTTP header

@mareknovotny
Copy link
Member

closing as the parent PR was closed long ago

@mbiarnes mbiarnes deleted the org.eclipse.jetty_vulnerability branch August 13, 2020 07:31
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.

2 participants