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 : Can't handle CJK multibyte request in AJP parameter value #275
Conversation
Linux Build 1341 is now running using a merge of c9c994c |
Windows Build 905 outcome was FAILURE using a merge of c9c994c Build problems:Failed tests detected Failed tests
|
Linux Build 1341 outcome was FAILURE using a merge of c9c994c Build problems:Failed tests detected Failed tests
|
Linux Build 1342 is now running using a merge of e2c0429 |
Windows Build 906 outcome was FAILURE using a merge of e2c0429 Build problems:Failed tests detected Failed tests
|
Linux Build 1342 outcome was FAILURE using a merge of e2c0429 Build problems:Failed tests detected Failed tests
|
Linux Build 1345 is now running using a merge of e2c0429 |
Windows Build 909 outcome was FAILURE using a merge of e2c0429 Build problems:Failed tests detected Failed tests
|
Linux Build 1345 outcome was FAILURE using a merge of e2c0429 Build problems:Failed tests detected Failed tests
|
Linux Build 1350 is now running using a merge of e2c0429 |
Windows Build 914 outcome was FAILURE using a merge of e2c0429 Build problems:Failed tests detected Failed tests
|
Linux Build 1350 outcome was FAILURE using a merge of e2c0429 Build problems:Failed tests detected Failed tests
|
Thanks for the PR, I have created https://issues.jboss.org/browse/UNDERTOW-362 to track this. I will probably fix this a slightly different way however, as List is very inefficient (it creates a lot of unnecessary objects) |
Thanks for the PR, however I have fixed it a different way that should give better performance: Can you test this out and make sure it works for you? |
It works very well. |
Problem :
In HTTP protocol hander can handle multibyte character set. My case use EUC-KR character set. Tested in Wildfly 8.2 GA.
http://localhost:8080/test/test.jsp?param=한글
But, When I use AJP protocol can't handle EUC-KR character set.
http://localhost/test/test.jsp?param=한글
Query param's value are not '한글', it prints '??'.
Similar cases are in https://developer.jboss.org/thread/242677?start=0&tstart=0
Reason :
I found that it cause of handle byte as a char in AbstractAjpParser.
Multibyte(CJK ; Chinese, Japanese, Korean) character set are use most significant bit. So, when byte are casted to char, multibytes informations are eliminated.
Thanks for your great 'undertow'.