Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
0028566Openbravo ERPA. Platformpublic2015-01-12 14:172022-02-01 08:09
shuehner 
Triage Platform Base 
normalminorhave not tried
acknowledgedopen 
5
 
 
Core
No
0028566: ad_session.websession column is too small
That 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.



-
Increase 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)
No tags attached.
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 
Issue History
2015-01-12 14:17shuehnerNew Issue
2015-01-12 14:17shuehnerAssigned To => AugustoMauch
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.