Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0046567Openbravo ERPA. Platformpublic2021-05-05 16:382021-05-17 10:16
Reportershuehner 
Assigned Toalostale 
PrioritynormalSeverityminorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionPR21Q1.2Fixed in VersionPR21Q1.2 
Merge Request Statusapproved
Review Assigned To
OBNetwork customerNo
Web browser
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0046567: junit DataSourceSecurity.fetchShouldBeAllowedOnlyIfRoleIsGranted fails on Tomcat9 (only for Datasource: AccountTreeMovement)
DescriptionThis junit test fails when application is deployed on tomcat 9.x but only for the specific datasource AccountTreeMovement.
It shows 4 failures as four RoleTypes are tested for each dataSource.

Problem is the test triggering the following error on tomcat 9.x:
May 05 12:27:36 ip-172-31-9-130 tomcat9[2709]: Error parsing HTTP request header
May 05 12:27:36 ip-172-31-9-130 tomcat9[2709]: Note: further occurrences of HTTP request parsing errors will be logged at DEBUG level.
May 05 12:27:36 ip-172-31-9-130 tomcat9[2709]: java.lang.IllegalArgumentException: Invalid character found in the request target. The valid characters are defined in RFC 7230 and RFC 3986

Steps To ReproduceDeploy application with tomcat 9.x
ant run.test -Dtest=org.openbravo.test.datasource.DataSourceSecurity

Check result*.xml file in src/ folder for the result of the test run
Proposed SolutionFix the invalid request target (values in url) to be valid.

Additional Information
TagsNo tags attached.
Relationships
blocks defect 0046520PR21Q3 closed alostale junit DataSourceSecurity.fetchShouldBeAllowedOnlyIfRoleIsGranted fails on Tomcat9 (only for Datasource: AccountTreeMovement) 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2021-05-07 08:11alostaleTypedefect => backport
2021-05-07 08:11alostaleTarget VersionPR21Q3 => PR21Q1.2
2021-05-07 11:30alostaleIssue cloned0046579
2021-05-17 08:03hgbotMerge Request Status => open
2021-05-17 08:03hgbotNote Added: 0128168
2021-05-17 10:16hgbotMerge Request Statusopen => approved
2021-05-17 10:16hgbotResolutionopen => fixed
2021-05-17 10:16hgbotStatusscheduled => closed
2021-05-17 10:16hgbotNote Added: 0128204
2021-05-17 10:16hgbotFixed in Version => PR21Q1.2
2021-05-17 10:16hgbotNote Added: 0128205

Notes
(0128168)
hgbot   
2021-05-17 08:03   
Merge Request created: https://gitlab.com/openbravo/product/openbravo/-/merge_requests/362 [^]
(0128204)
hgbot   
2021-05-17 10:16   
Merge request merged: https://gitlab.com/openbravo/product/openbravo/-/merge_requests/362 [^]
(0128205)
hgbot   
2021-05-17 10:16   
Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/openbravo [^]
Changeset: 21f7703ff61f7527c36d97f4b147cdff8019e4d2
Author: Asier Lostalé <asier.lostale@openbravo.com>
Date: 2021-05-17T08:01:58+02:00
URL: https://gitlab.com/openbravo/product/openbravo/-/commit/21f7703ff61f7527c36d97f4b147cdff8019e4d2 [^]

fixes BUG-46567: DataSourceSecurity fails on Tomcat9

fetchShouldBeAllowedOnlyIfRoleIsGranted test was creating an invalid
request by adding some parameters with special characters and not
ecoding them. Tomcat 9 is stricter parsing URLs than previous version,
not allowing this case.

Fixed by removing those parameters that were causing problems as they
were not ncessary.

---
M src-test/src/org/openbravo/test/datasource/DataSourceSecurity.java
---