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

Execute tests in servlet-2.5-mvc-3 #4473

Merged
merged 4 commits into from
May 20, 2020

Conversation

jbescos
Copy link
Member

@jbescos jbescos commented May 19, 2020

Locally I'm having an error in one test that checks one ISO character encoding, but it seems it is running fine in Travis.

Signed-off-by: Jorge Bescos Gascon <[email protected]>
@jbescos
Copy link
Member Author

jbescos commented May 19, 2020

Same issue in Travis, as far as I can see. I will try to fix it.

Signed-off-by: Jorge Bescos Gascon <[email protected]>
@jbescos
Copy link
Member Author

jbescos commented May 19, 2020

I deployed the war file in a local Jetty 9 and I saw the charset was not specified in the header.

The test was reading the page. As the charset was not specified it was reading it as UTF-8 by default.

I had to add a filter to explicitly set the header with the required charset.

jbescos added 2 commits May 19, 2020 16:44
This reverts commit 8daf7b8.

Signed-off-by: Jorge Bescos Gascon <[email protected]>
Signed-off-by: Jorge Bescos Gascon <[email protected]>
@jbescos jbescos force-pushed the servlet-2.5-mvc-3 branch from ae9609d to db3bfa6 Compare May 20, 2020 07:18
@senivam senivam merged commit 5b6597a into eclipse-ee4j:master May 20, 2020
@senivam senivam added this to the 2.31 milestone May 20, 2020
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.

3 participants