From c0f2d3de900e2234e9c7f928e4774c6c7d7ca481 Mon Sep 17 00:00:00 2001 From: Bob Mottram Date: Tue, 1 Sep 2020 14:35:05 +0100 Subject: [PATCH] Move blogs timeline to its own method --- daemon.py | 210 ++++++++++++++++++++++++++++++------------------------ 1 file changed, 117 insertions(+), 93 deletions(-) diff --git a/daemon.py b/daemon.py index f70544aa4..68fae1671 100644 --- a/daemon.py +++ b/daemon.py @@ -5757,6 +5757,107 @@ class PubServer(BaseHTTPRequestHandler): return True return False + def _showBlogsTimeline(self, authorized: bool, + callingDomain: str, path: str, + baseDir: str, httpPrefix: str, + domain: str, domainFull: str, port: int, + onionDomain: str, i2pDomain: str, + GETstartTime, GETtimings: {}, + proxyType: str, cookie: str, + debug: str) -> bool: + """Shows the blogs timeline + """ + if '/users/' in path: + if authorized: + inboxBlogsFeed = \ + personBoxJson(self.server.recentPostsCache, + self.server.session, + baseDir, + domain, + port, + path, + httpPrefix, + maxPostsInBlogsFeed, 'tlblogs', + True, self.server.ocapAlways) + if not inboxBlogsFeed: + inboxBlogsFeed = [] + if self._requestHTTP(): + nickname = path.replace('/users/', '') + nickname = nickname.replace('/tlblogs', '') + pageNumber = 1 + if '?page=' in nickname: + pageNumber = nickname.split('?page=')[1] + nickname = nickname.split('?page=')[0] + if pageNumber.isdigit(): + pageNumber = int(pageNumber) + else: + pageNumber = 1 + if 'page=' not in path: + # if no page was specified then show the first + inboxBlogsFeed = \ + personBoxJson(self.server.recentPostsCache, + self.server.session, + baseDir, + domain, + port, + path + '?page=1', + httpPrefix, + maxPostsInBlogsFeed, 'tlblogs', + True, self.server.ocapAlways) + msg = \ + htmlInboxBlogs(self.server.defaultTimeline, + self.server.recentPostsCache, + self.server.maxRecentPosts, + self.server.translate, + pageNumber, maxPostsInBlogsFeed, + self.server.session, + baseDir, + self.server.cachedWebfingers, + self.server.personCache, + nickname, + domain, + port, + inboxBlogsFeed, + self.server.allowDeletion, + httpPrefix, + self.server.projectVersion, + self._isMinimal(nickname), + self.server.YTReplacementDomain) + msg = msg.encode('utf-8') + self._set_headers('text/html', len(msg), + cookie, callingDomain) + self._write(msg) + self._benchmarkGETtimings(GETstartTime, GETtimings, + 'show media 2 done', + 'show blogs 2') + else: + # don't need authenticated fetch here because there is + # already the authorization check + msg = json.dumps(inboxBlogsFeed, + ensure_ascii=False) + msg = msg.encode('utf-8') + self._set_headers('application/json', + len(msg), + None, callingDomain) + self._write(msg) + self.server.GETbusy = False + return True + else: + if debug: + nickname = self.path.replace('/users/', '') + nickname = nickname.replace('/tlblogs', '') + print('DEBUG: ' + nickname + + ' was not authorized to access ' + path) + if path != '/tlblogs': + # not the blogs inbox + if debug: + print('DEBUG: GET access to blogs is unauthorized') + self.send_response(405) + self.end_headers() + self.server.GETbusy = False + return True + return False + def do_GET(self): callingDomain = self.server.domainFull if self.headers.get('Host'): @@ -7534,7 +7635,7 @@ class PubServer(BaseHTTPRequestHandler): 'show skills done', 'show status done') - # get the inbox for a given person + # get the inbox timeline for a given person if self.path.endswith('/inbox') or '/inbox?page=' in self.path: if self._showInbox(authorized, callingDomain, self.path, @@ -7554,7 +7655,7 @@ class PubServer(BaseHTTPRequestHandler): 'show status done', 'show inbox done') - # get the direct messages for a given person + # get the direct messages timeline for a given person if self.path.endswith('/dm') or '/dm?page=' in self.path: if self._showDMs(authorized, callingDomain, self.path, @@ -7574,7 +7675,7 @@ class PubServer(BaseHTTPRequestHandler): 'show inbox done', 'show dms done') - # get the replies for a given person + # get the replies timeline for a given person if self.path.endswith('/tlreplies') or '/tlreplies?page=' in self.path: if self._showReplies(authorized, callingDomain, self.path, @@ -7594,7 +7695,7 @@ class PubServer(BaseHTTPRequestHandler): 'show dms done', 'show replies 2 done') - # get the media for a given person + # get the media timeline for a given person if self.path.endswith('/tlmedia') or '/tlmedia?page=' in self.path: if self._showMediaTimeline(authorized, callingDomain, self.path, @@ -7616,95 +7717,18 @@ class PubServer(BaseHTTPRequestHandler): # get the blogs for a given person if self.path.endswith('/tlblogs') or '/tlblogs?page=' in self.path: - if '/users/' in self.path: - if authorized: - inboxBlogsFeed = \ - personBoxJson(self.server.recentPostsCache, - self.server.session, - self.server.baseDir, - self.server.domain, - self.server.port, - self.path, - self.server.httpPrefix, - maxPostsInBlogsFeed, 'tlblogs', - True, self.server.ocapAlways) - if not inboxBlogsFeed: - inboxBlogsFeed = [] - if self._requestHTTP(): - nickname = self.path.replace('/users/', '') - nickname = nickname.replace('/tlblogs', '') - pageNumber = 1 - if '?page=' in nickname: - pageNumber = nickname.split('?page=')[1] - nickname = nickname.split('?page=')[0] - if pageNumber.isdigit(): - pageNumber = int(pageNumber) - else: - pageNumber = 1 - if 'page=' not in self.path: - # if no page was specified then show the first - inboxBlogsFeed = \ - personBoxJson(self.server.recentPostsCache, - self.server.session, - self.server.baseDir, - self.server.domain, - self.server.port, - self.path + '?page=1', - self.server.httpPrefix, - maxPostsInBlogsFeed, 'tlblogs', - True, self.server.ocapAlways) - msg = \ - htmlInboxBlogs(self.server.defaultTimeline, - self.server.recentPostsCache, - self.server.maxRecentPosts, - self.server.translate, - pageNumber, maxPostsInBlogsFeed, - self.server.session, - self.server.baseDir, - self.server.cachedWebfingers, - self.server.personCache, - nickname, - self.server.domain, - self.server.port, - inboxBlogsFeed, - self.server.allowDeletion, - self.server.httpPrefix, - self.server.projectVersion, - self._isMinimal(nickname), - self.server.YTReplacementDomain) - msg = msg.encode('utf-8') - self._set_headers('text/html', - len(msg), - cookie, callingDomain) - self._write(msg) - self._benchmarkGETtimings(GETstartTime, GETtimings, - 'show media 2 done', - 'show blogs 2') - else: - # don't need authenticated fetch here because there is - # already the authorization check - msg = json.dumps(inboxBlogsFeed, - ensure_ascii=False) - msg = msg.encode('utf-8') - self._set_headers('application/json', - len(msg), - None, callingDomain) - self._write(msg) - self.server.GETbusy = False - return - else: - if self.server.debug: - nickname = self.path.replace('/users/', '') - nickname = nickname.replace('/tlblogs', '') - print('DEBUG: ' + nickname + - ' was not authorized to access ' + self.path) - if self.path != '/tlblogs': - # not the blogs inbox - if self.server.debug: - print('DEBUG: GET access to blogs is unauthorized') - self.send_response(405) - self.end_headers() - self.server.GETbusy = False + if self._showBlogsTimeline(authorized, + callingDomain, self.path, + self.server.baseDir, + self.server.httpPrefix, + self.server.domain, + self.server.domainFull, + self.server.port, + self.server.onionDomain, + self.server.i2pDomain, + GETstartTime, GETtimings, + self.server.proxyType, + cookie, self.server.debug): return self._benchmarkGETtimings(GETstartTime, GETtimings,