kali834x opened a new pull request, #1320:
URL: https://github.com/apache/axis-axis2-java-core/pull/1320

   sessionmanager.getSessionContext now creates transport session keys with 
UIDGenerator.generateURNString, the SecureRandom-backed UUID the soap session 
path in ConfigurationContext already uses for its service group context id, 
instead of generateUID. generateUID is a per-thread constant (start time and 
thread id) followed by a per-thread counter XORed with one per-JVM value, and 
on the standalone server (SimpleHTTPServer / AxisHttpService) that key is all a 
caller presents, in an axis_session or JSESSIONID request header read by 
RequestSessionCookie, to be bound to a SessionContext and its 
transportsession-scoped service instances. since ResponseSessionCookie hands 
every caller their own key, one anonymous request tells you the fixed 32 hex 
characters for that worker thread, and the next key issued on it differs only 
in the counter's low bits: one of 64 candidates when nothing else drew a UID on 
the thread in between, a small search otherwise (CWE-330, predictable session 
identifi
 er). the servlet path uses the container's HttpSession and is unaffected. 
SessionManagerTest gets a regression that derives the 64 candidates from each 
key and fails before the change (the next key is among them) and passes after; 
the transport-http module's verify is green (112 tests).
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to