Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0028566Openbravo ERPA. Platformpublic2015-01-12 14:172022-02-01 08:09
Reportershuehner 
Assigned ToTriage Platform Base 
PrioritynormalSeverityminorReproducibilityhave not tried
StatusacknowledgedResolutionopen 
PlatformOS5OS Version
Product Version 
Target VersionFixed in Version 
Merge Request Status
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
Summary0028566: ad_session.websession column is too small
DescriptionThat column is defined as varchar(40) and standard tomcat session id seems to be 32chars (32 hex-chars).
Which fits nicely.

However when using jvmRoute parameter of tomcat this adds some suffix to the session-id values.
So they look like
aa<32chars>.suffix

With the '.' separator that only leaves 7 chars available for the suffix which can easily overflow depending on the admin chosen suffix.



Steps To Reproduce-
Proposed SolutionIncrease length of the websession column to i.e. varchar(64).

Note that qualifies as a api-change. Albeit a probably harmless one as not even inside the ERP anyone uses that column (used for helping debugging)
Additional Information
TagsNo tags attached.
Relationships
depends on feature request 00346543.0PR17Q2 closed alostale some performance improvements in DBSM tasks 
related to design defect 0034490 acknowledged Triage Platform Base [clustering] Support clustering 
related to defect 0033621 closed alostale Increase the size of the ad_alert.record_id from 100 to 2000 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2015-01-12 14:17shuehnerNew Issue
2015-01-12 14:17shuehnerAssigned To => AugustoMauch
2015-01-12 14:17shuehnerOBNetwork customer => No
2015-01-12 14:17shuehnerModules => Core
2015-01-12 14:17shuehnerTriggers an Emergency Pack => No
2015-01-12 14:23shuehnerNote Added: 0073426
2015-01-12 14:23shuehnerTypedefect => feature request
2015-01-12 15:37alostaleTarget Version => 3.0PR15Q2
2015-03-31 14:24alostaleTarget Version3.0PR15Q2 => 3.0PR15Q3
2015-03-31 14:24alostaleAssigned ToAugustoMauch => platform
2015-03-31 14:24alostaleStatusnew => acknowledged
2015-06-29 13:32alostaleTarget Version3.0PR15Q3 =>
2016-12-13 07:51alostaleRelationship addedrelated to 0034490
2017-01-11 13:02alostaleRelationship addeddepends on 0034654
2017-01-11 13:04alostaleRelationship addedrelated to 0033621
2022-02-01 08:09alostaleAssigned Toplatform => Triage Platform Base

Notes
(0073426)
shuehner   
2015-01-12 14:23   
Note: length of getId is undefined in spec (implementation dependent) but as we only target tomcat 60 compared to observed 32 should be more then enough for now.