From f67481feeb4789eb5d84bdffb0064693eaa5c479 Mon Sep 17 00:00:00 2001
From: Giuseppe Lo Presti <giuseppe.lopresti@cern.ch>
Date: Mon, 29 Nov 2021 17:00:37 +0100
Subject: [PATCH] Ensure webconflict files can be written also when user has no
 access to the folder

---
 src/core/wopi.py      | 9 ++++++++-
 src/core/wopiutils.py | 7 ++++++-
 2 files changed, 14 insertions(+), 2 deletions(-)

diff --git a/src/core/wopi.py b/src/core/wopi.py
index 69197635..ee5d3ff6 100644
--- a/src/core/wopi.py
+++ b/src/core/wopi.py
@@ -414,7 +414,14 @@ def putFile(fileid):
             newname, ext = os.path.splitext(acctok['filename'])
             # !!! typical EFSS formats are like '<filename>_conflict-<date>-<time>', but they're not synchronized back !!!
             newname = '%s-webconflict-%s%s' % (newname, time.strftime('%Y%m%d-%H'), ext.strip())
-            utils.storeWopiFile(flask.request, retrievedLock, acctok, utils.LASTSAVETIMEKEY, newname)
+            try:
+                utils.storeWopiFile(flask.request, retrievedLock, acctok, utils.LASTSAVETIMEKEY, newname)
+            except IOError as e:
+                if utils.ACCESS_ERROR in str(e):
+                    # let's try the user's home instead of the current folder
+                    newname = utils.getuserhome(acctok['username']) + os.path.sep + os.path.basename(newname)
+                    utils.storeWopiFile(flask.request, retrievedLock, acctok, utils.LASTSAVETIMEKEY, newname)
+
             # keep track of this action in the original file's xattr, to avoid looping (see below)
             st.setxattr(acctok['endpoint'], acctok['filename'], acctok['userid'], utils.LASTSAVETIMEKEY, 0)
             log.info('msg="Conflicting copy created" user="%s" savetime="%s" lastmtime="%s" newfilename="%s" token="%s"' %
diff --git a/src/core/wopiutils.py b/src/core/wopiutils.py
index a9784dfb..8d3fa31f 100644
--- a/src/core/wopiutils.py
+++ b/src/core/wopiutils.py
@@ -380,4 +380,9 @@ def storeWopiFile(request, retrievedlock, acctok, xakey, targetname=''):
     st.setxattr(acctok['endpoint'], targetname, acctok['userid'], xakey, int(time.time()))
     # and reinstate the lock if existing
     if retrievedlock:
-        st.setlock(acctok['endpoint'], targetname, acctok['userid'], _makeLock(retrievedlock))
\ No newline at end of file
+        st.setlock(acctok['endpoint'], targetname, acctok['userid'], _makeLock(retrievedlock))
+
+
+def getuserhome(username):
+    '''Returns the path to the "home" directory for a given user. CERN/EOS-specific, to be removed once locking is fully implemented.'''
+    return '/eos/user/%s/%s' % (username[0], username)