details:   /erp/devel/pi/rev/94fcf9a433af
changeset: 7899:94fcf9a433af
user:      Stefan Hühner <stefan.huehner <at> openbravo.com>
date:      Wed Jul 21 11:28:33 2010 +0200
summary:   Fixed 14012: Do commit immediately after insert into ad_session..
.. to prevent a race condition with other requests using i.e. xsql-queries
which otherwise don't see the inserted record early enough.
This happens when using DeepLinking together with any custom AuthManager.

diffstat:

 src/org/openbravo/erpCommon/security/SessionLogin.java |  2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diffs (12 lines):

diff -r e5ef80a147fb -r 94fcf9a433af 
src/org/openbravo/erpCommon/security/SessionLogin.java
--- a/src/org/openbravo/erpCommon/security/SessionLogin.java    Tue Jul 20 
18:25:35 2010 +0200
+++ b/src/org/openbravo/erpCommon/security/SessionLogin.java    Wed Jul 21 
11:28:33 2010 +0200
@@ -135,7 +135,7 @@
       session.setLoginStatus(status);
       session.setUsername(username);
       OBDal.getInstance().save(session);
-      OBDal.getInstance().flush();
+      OBDal.getInstance().commitAndClose();
       setSessionID(session.getId());
       return 1;
     } catch (Exception e) {

------------------------------------------------------------------------------
This SF.net email is sponsored by Sprint
What will you do first with EVO, the first 4G phone?
Visit sprint.com/first -- http://p.sf.net/sfu/sprint-com-first
_______________________________________________
Openbravo-commits mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openbravo-commits

Reply via email to