Function to test for account directories

blocking.py

@@ -9,6 +9,7 @@ __status__ = "Production"
 import os
 import json
 from datetime import datetime
+from utils import isAccountDir
 from utils import getCachedPostFilename
 from utils import loadJson
 from utils import saveJson
@@ -660,9 +661,7 @@ def setBrochMode(baseDir: str, domainFull: str, enabled: bool) -> None:
         followFiles = ('following.txt', 'followers.txt')
         for subdir, dirs, files in os.walk(baseDir + '/accounts'):
             for acct in dirs:
-                if '@' not in acct:
+                if not isAccountDir(acct):
-                    continue
-                if 'inbox@' in acct or 'news@' in acct:
                     continue
                 accountDir = os.path.join(baseDir + '/accounts', acct)
                 for followFileType in followFiles:

daemon.py

@@ -203,6 +203,7 @@ from shares import addShare
 from shares import removeShare
 from shares import expireShares
 from categories import setHashtagCategory
+from utils import isAccountDir
 from utils import getOccupationSkills
 from utils import getOccupationName
 from utils import setOccupationName
@@ -5700,9 +5701,7 @@ class PubServer(BaseHTTPRequestHandler):
         msg = ''
         for subdir, dirs, files in os.walk(baseDir + '/accounts'):
             for acct in dirs:
-                if '@' not in acct:
+                if not isAccountDir(acct):
-                    continue
-                if 'inbox@' in acct or 'news@' in acct:
                     continue
                 nickname = acct.split('@')[0]
                 domain = acct.split('@')[1]

follow.py

@@ -22,6 +22,7 @@ from posts import sendSignedJson
 from posts import getPersonBox
 from utils import loadJson
 from utils import saveJson
+from utils import isAccountDir
 from acceptreject import createAccept
 from acceptreject import createReject
 from webfinger import webfingerHandle
@@ -37,9 +38,7 @@ def createInitialLastSeen(baseDir: str, httpPrefix: str) -> None:
     """
     for subdir, dirs, files in os.walk(baseDir + '/accounts'):
         for acct in dirs:
-            if '@' not in acct:
+            if not isAccountDir(acct):
-                continue
-            if 'inbox@' in acct or 'news@' in acct:
                 continue
             accountDir = os.path.join(baseDir + '/accounts', acct)
             followingFilename = accountDir + '/following.txt'

metadata.py

@@ -7,6 +7,7 @@ __email__ = "bob@freedombone.net"
 __status__ = "Production"
 
 import os
+from utils import isAccountDir
 from utils import loadJson
 from utils import noOfAccounts
 from utils import noOfActiveAccountsMonthly
@@ -19,9 +20,7 @@ def _getStatusCount(baseDir: str) -> int:
     accountsDir = baseDir + '/accounts'
     for subdir, dirs, files in os.walk(accountsDir):
         for acct in dirs:
-            if '@' not in acct:
+            if not isAccountDir(acct):
-                continue
-            if 'inbox@' in acct or 'news@' in acct:
                 continue
             acctDir = os.path.join(accountsDir, acct + '/outbox')
             for subdir2, dirs2, files2 in os.walk(acctDir):

utils.py

@@ -2400,3 +2400,13 @@ def setOccupationSkillsList(actorJson: {}, skillsList: []) -> bool:
         occupationItem['skills'] = skillsList
         return True
     return False
+
+
+def isAccountDir(dirName: str) -> bool:
+    """Is the given directory an account within /accounts ?
+    """
+    if '@' not in dirName:
+        return False
+    if 'inbox@' in dirName or 'news@' in dirName:
+        return False
+    return True

webapp_accesskeys.py

@@ -7,6 +7,7 @@ __email__ = "bob@freedombone.net"
 __status__ = "Production"
 
 import os
+from utils import isAccountDir
 from utils import loadJson
 from utils import getConfigParam
 from webapp_utils import htmlHeaderWithExternalStyle
@@ -19,9 +20,7 @@ def loadAccessKeysForAccounts(baseDir: str, keyShortcuts: {},
     """
     for subdir, dirs, files in os.walk(baseDir + '/accounts'):
         for acct in dirs:
-            if '@' not in acct:
+            if not isAccountDir(acct):
-                continue
-            if 'inbox@' in acct or 'news@' in acct:
                 continue
             accountDir = os.path.join(baseDir + '/accounts', acct)
             accessKeysFilename = accountDir + '/accessKeys.json'