Merge lp:~anybox/openobject-addons/lp1040901 into lp:openobject-addons

[Georges Racinet]

Change the name in TMPDIR of the created/used OpenID session store to depend on the system user. Still allows multiple processes sharing one (WSGI cases), provided they run under the same system user.

[Fabien (Open ERP) (fp-tinyerp)]

I suggest to put this in a module.

[Olivier Dony (Odoo) (odo-openerp)]

The patch was merged some time ago in trunk[1], thanks a lot for the contribution :-)

[1] rev.7890 revid:gracinet@purity-20120914081101-4bx5yi8fxrlbav1r

=== modified file 'auth_openid/controllers/main.py'
--- auth_openid/controllers/main.py	2012-08-31 13:51:36 +0000
+++ auth_openid/controllers/main.py	2012-09-14 08:22:23 +0000
@@ -22,6 +22,7 @@
 import logging
 import os
 import tempfile
+import getpass
 import urllib
 from openerp import SUPERUSER_ID
 
@@ -46,7 +47,16 @@
 _logger = logging.getLogger(__name__)
 oidutil.log = _logger.debug
 
-_storedir = os.path.join(tempfile.gettempdir(), 'openerp-auth_openid-store')
+def get_system_user():
+    """Return system user info string, such as USERNAME-EUID"""
+    info = getpass.getuser()
+    euid = getattr(os, 'geteuid', None) # Non available on some platforms
+    if euid is not None:
+        info = '%s-%d' % (info, euid())
+    return info
+
+_storedir = os.path.join(tempfile.gettempdir(), 
+                         'openerp-auth_openid-%s-store' % get_system_user())
 
 class GoogleAppsAwareConsumer(consumer.GenericConsumer):
     def complete(self, message, endpoint, return_to):