From c594e635125bbc0eccc09802a4af457489861c30 Mon Sep 17 00:00:00 2001 From: canewsin Date: Fri, 7 Jan 2022 02:29:04 +0530 Subject: [PATCH] Bug Fixes --- plugins | 2 +- src/Content/ContentDb.py | 4 +-- src/Site/Site.py | 2 +- src/Site/SiteManager.py | 10 +++--- src/Test/conftest.py | 2 +- src/Test/testdata/chart.db-shm | Bin 0 -> 32768 bytes src/Test/testdata/chart.db-wal | Bin 0 -> 78312 bytes src/Test/testdata/content.db-shm | Bin 0 -> 32768 bytes src/Test/testdata/content.db-wal | Bin 0 -> 98912 bytes src/Test/testdata/filters.json | 1 + src/Test/testdata/openssl.cnf | 58 +++++++++++++++++++++++++++++++ src/Test/testdata/sites.json | 1 + src/Test/testdata/trackers.json | 1 + src/Test/testdata/users.json | 9 +++++ src/User/User.py | 12 +++---- 15 files changed, 86 insertions(+), 16 deletions(-) create mode 100644 src/Test/testdata/chart.db-shm create mode 100644 src/Test/testdata/chart.db-wal create mode 100644 src/Test/testdata/content.db-shm create mode 100644 src/Test/testdata/content.db-wal create mode 100644 src/Test/testdata/filters.json create mode 100644 src/Test/testdata/openssl.cnf create mode 100644 src/Test/testdata/sites.json create mode 100644 src/Test/testdata/trackers.json create mode 100644 src/Test/testdata/users.json diff --git a/plugins b/plugins index 9cadd0c6..a55d1b02 160000 --- a/plugins +++ b/plugins @@ -1 +1 @@ -Subproject commit 9cadd0c69f5af06739cc34cc79dc650053a9b4bd +Subproject commit a55d1b02488de996c32e515d3ecaf701dd515d46 diff --git a/src/Content/ContentDb.py b/src/Content/ContentDb.py index f0401652..f284581e 100644 --- a/src/Content/ContentDb.py +++ b/src/Content/ContentDb.py @@ -82,7 +82,7 @@ class ContentDb(Db): return schema def initSite(self, site): - self.getSites()[site.address] = site + self.sites[site.address] = site def needSite(self, site): if site.address not in self.site_ids: @@ -97,7 +97,7 @@ class ContentDb(Db): if site_id: self.execute("DELETE FROM site WHERE site_id = :site_id", {"site_id": site_id}) del self.site_ids[site.address] - del self.getSites()[site.address] + del self.sites[site.address] def setContent(self, site, inner_path, content, size=0): self.insertOrUpdate("content", { diff --git a/src/Site/Site.py b/src/Site/Site.py index ace2a65e..46e19169 100644 --- a/src/Site/Site.py +++ b/src/Site/Site.py @@ -290,7 +290,7 @@ class Site(object): if not SiteManager.site_manager.sites: SiteManager.site_manager.sites = {} if not SiteManager.site_manager.sites.get(self.address): - SiteManager.site_manager.getSites()[self.address] = self + SiteManager.site_manager.sites[self.address] = self SiteManager.site_manager.load(False) SiteManager.site_manager.saveDelayed() diff --git a/src/Site/SiteManager.py b/src/Site/SiteManager.py index da199ee8..8175a1f5 100644 --- a/src/Site/SiteManager.py +++ b/src/Site/SiteManager.py @@ -73,7 +73,7 @@ class SiteManager(object): except Exception as err: self.log.debug("Error loading site %s: %s" % (address, err)) continue - self.getSites()[address] = site + self.sites[address] = site self.log.debug("Loaded site %s in %.3fs" % (address, time.time() - s)) added += 1 elif startup: @@ -88,7 +88,7 @@ class SiteManager(object): if cleanup: for address in list(self.sites.keys()): if address not in address_found: - del(self.getSites()[address]) + del(self.sites[address]) self.log.debug("Removed site: %s" % address) # Remove orpan sites from contentdb @@ -106,7 +106,7 @@ class SiteManager(object): if address in content_db.site_ids: del content_db.site_ids[address] if address in content_db.sites: - del content_db.getSites()[address] + del content_db.sites[address] self.loaded = True for address, settings in sites_need: @@ -203,7 +203,7 @@ class SiteManager(object): self.log.debug("Added new site: %s" % address) config.loadTrackersFile() site = Site(address, settings=settings) - self.getSites()[address] = site + self.sites[address] = site if not site.settings["serving"]: # Maybe it was deleted before site.settings["serving"] = True site.saveSettings() @@ -226,7 +226,7 @@ class SiteManager(object): def delete(self, address): self.sites_changed = int(time.time()) self.log.debug("Deleted site: %s" % address) - del(self.getSites()[address]) + del(self.sites[address]) # Delete from sites.json self.save() diff --git a/src/Test/conftest.py b/src/Test/conftest.py index 1a7c68e7..c8739086 100644 --- a/src/Test/conftest.py +++ b/src/Test/conftest.py @@ -219,7 +219,7 @@ def site(request): site = Site("1TeSTvb4w2PWE81S2rEELgmX2GCCExQGT") # Create new Site object to load content.json files if not SiteManager.site_manager.sites: SiteManager.site_manager.sites = {} - SiteManager.site_manager.getSites()["1TeSTvb4w2PWE81S2rEELgmX2GCCExQGT"] = site + SiteManager.site_manager.sites["1TeSTvb4w2PWE81S2rEELgmX2GCCExQGT"] = site site.settings["serving"] = True return site diff --git a/src/Test/testdata/chart.db-shm b/src/Test/testdata/chart.db-shm new file mode 100644 index 0000000000000000000000000000000000000000..7c822be1e7e27ced30c9f50e2208298742b46757 GIT binary patch literal 32768 zcmeI*Jx&5)5C-566vcngh67lUPrFXG0@dAWorivQvYNHbQMA|E=q`@e(dqs0`)#>iJiUB;J-%-?KhMk0>BIbWa^UylQe6h| z`n#9y>D?iIyY-{n`)J4Ue*av4&TE`RPou-=S@b-55xtCFMf2W;009C72oNAZfB*pk z1PBlyK!5-N0t5&UAV7cs0RjXF5FkK+009C72oNAZfB*pk1PBlyK!5-N0t5&UAV7cs z0RjXF5FkL{zXg&jGZVYNADUdsIg+NUK+o5az@1{=>3XIB5Zg2u)1O&)vNoSUca{UiHRkmNOzaFoH=)( zW!e7M)~xRAFWTnD;gQYxqIr)v+T+n4GEK3sN6cP%OnH*W`_zVbb=q@n|JkZXYwX?` z6*V5?4$UZv1rq`YAby@7R_f9JAG{r=k^A zY9`cLUw^^u$mFbtt*p5!+tHQI_L?iLUNha3>+Z;igS)IuE+zIhbJn_C%4u62klHjl zJmhL1Dahsb{A=-o5$~5n?+%>ZCcm?nLkV(?2>}EUKmY**5I_I{1Q0*~0R*O|fV}tb zRe1!*!v~-HW{LhWae=9Mu{kRQ5I_I{1Q0*~0R#|0009Ih2~<2s&~|0y;+62RB{zAF z!2R5S%r|gv|D)#!ys^)n=Lp0L69NbzfB*srAbWI^3lUzpHLxK35dfj$7AIsE#9Dq@xi_WTzt`*JEnr!1 zCa#wymj(mzS@n8v>C8)ha;H#5BMSCf3+003cvrvy*}c;Nkw?n8t^!1sIB%N&E0FY> z*2-bx%*mH)GN-Cfsjihro|`nRoOtW9jM*kez{_=XS!SEnp6=<)nTgh(Y*q|O*C$KY zm%uGVo`H*GD*I`e;If1!57cI zKk@xXE9LkCe&boq_}VySd|o1PhKepP#*W`Fa@_(2Yx)_+vr<0R#|0009ILKmY**5I_KdDI*Zky$!kw5BSAt zInMy7xWI-*;ftT1O0SgT3;2yansL}TV|;7uF4ftKmtz`1Q0*~0R#|0009ILKmY**swCiq0ItOa#+Sag?9jr`hznFnn`RL} z009ILKmY**5I_I{1Q4h#fw1h3iVJ-5!pY!@>5VzY7pSd^L~js4009ILKmY**5I_I{ z1Q4i_Kt#5y;sVhXKOH#oj5R{OK$Wy<76AkhKmY**5I_I{1Q0*~f!Y#qQUg?6;I?RT zZTjfI5%L9U>mtz`1Q0*~0R#|0009ILKmY**sw5B-?Z#AG;EUj$^A7&-<^cHuRnn$e X1Q0*~0R#|0009ILKmY**YDnOBiETRB literal 0 HcmV?d00001 diff --git a/src/Test/testdata/content.db-shm b/src/Test/testdata/content.db-shm new file mode 100644 index 0000000000000000000000000000000000000000..dfcb5880ffa6e96abc2f854c2e00c0c65b9eecfe GIT binary patch literal 32768 zcmeI*Jx)Sl5CzZ=6cv9E|DPMMa|<^1Zh!@7=?-)+tigoZ-kw%BVW~4O#8BASnasJ# z3qux*gq#j-$KLz36`QAUcU2MyJuE=y5dX5ds7V z5FkK+009C72oNAZfB*pk1PBlyK!5-N0t5&UAV7cs0RjXF5FkK+009C72oNAZfB*pk z1PBlyK!5-N0t5&UAV7csf&VBlh&>azQ9=mR6BzcJRO-!bVgk8oD0c`6fr1x5(e6mb4g&HvrhDR7E`Kv{vavXdK}z&Qc}Wd$zEPHu1ld6KaZAV7cs0Rq1X Fd;>p`DLViF literal 0 HcmV?d00001 diff --git a/src/Test/testdata/content.db-wal b/src/Test/testdata/content.db-wal new file mode 100644 index 0000000000000000000000000000000000000000..a78922375e66cae01ad55a26b077217f6a50a63f GIT binary patch literal 98912 zcmeI*Z)_Xo9l&v)|K|Utse(H~(J^RaQ#T{6SXND&aMFWjq)wYUUAt-+6JHjCW4pG~ zE<+tkk#l`?lW!m$c?{R#o zE5-&kioO=9efK=~{CuD1@uB?u+>?=qg2z2uFKzI6dOh;-Q2XzDPM!Jk@|(Zt`oY{= zC;uRJY4!Va=l*%-Z?ezhT< z?e^-n8@=>;h9-}7ixRK?S&v>3|L{Nn0R#|0009ILKmY**5J2ElDX_;I3~$+@9jv79 z%UR{aIq~aqs%#xzu<|pjH+^GMW;|&c$@u7mX{@Rn$*1S7m^d=J)UJiX8#ii8mfhw9 z*5Z+LZoyK|URNt>uMttNQrV0#o=BQ^nN!A|sqx+Mse{Jd=0PJqotzv`h=X^ViDXRF zHj?InWXx_`9T2;}T*%io5EbO|_xxJEVEKs`Zhi9T*+0tP*_~8^e8mF+1Q0*~0R#|0 z009ILKmY**)~0~`?%${85yWODPhNNA%~RwB)@EmORtO+~00IagfB*srAbXHN>L!ZzD0tg_000IagfB*srAbmjCJ3mO}Pd3q{K+rNkEm@_Buu zFBVh<`PG7aZA8j)4Weu&W~4Kjl2tCpjAEfwi7u7bg~I*)+QZ}ax4}M$d<+<6cmCG2T6O*H{l2u%&q$}A%{_3ig?7Wo{ZCfj=8`YYz%VrA;`K!yK_w|Ot zgM-?V&(_SAQ<=iie6Eld4b5hAR;o~x@zS|<4ny_Z8dp7elCHWnyuS0}rQjEKE? zja8$Z@33mr&bz4f&qo;4~IqTWq0N z?K@RZj6PN5>XXLhMfnz12Fc2?4Mx|h!j)mupy}%2Gh)TBa--&>J9>iQ-GiFP9v{xa zU+INP!QM`}x+690+SKpu3Wc{0XiEXR?o1(HvGSGFd?Aw+ADoP{-R&B@t1HRDM!Sx?K^(AasuDYBXY13}>uf-TX>tN0%1cL*an|b>VU9I!_(0ky&fU zc~Lh!T#K1~lEcz94CRL1L1oqL_AssPs$D#4WeQd8h;Ho&hT{X*>UnjR!_JG&L|ZVt zsb4!>9VqfT?p#jg=dQg`N7x@obM%BLYbv`;63WmSff35SigAB`m+r96Uh^~d$1*x-Qx0tg_000IagfB*srAb0CsS;OWQdQ}^-XHLcv^xhoc`Q{J_KAj7narT5f(_ofh%yzU3Vb(c0nZK_>Tty2q1s}0tg_000IagfB*srtQmnW z&9_lg`GKG~E&Cn-l^1wn`Q^;dUwQWpxxPS9PkZ#I^hJGHe@jp6FRmE^P6Po25I_I{ z1Q0*~0R#|00D)!+g#B&9+SL;Wf`-3s%-P>?+CboXf7^&tVb!Doe_fmPa|V2M`|GC+ zctuk?syDz5sf2)H0;hlb*W+J5z2$5CJ%X0@3BeIS009ILKmY**5I_I{1Q2MJKu88s^95cV zx#yN29txdjzCg41iGu(F2q1s}0tg_000IagfIv$H?7vo_@&d!d&-_w5cWgWJ1zOrC z1V;b?1Q0*~0R#|0009ILK%iLy?J|(c3%vIBjW^5>{^%L<0?pzl4gv@ufB*srAb_{=g12*i=Q|MAb#jS9yUC79V@(Cnx^#2I~v7v`+|*00IagfB*srAbl>dSFVHN0;vj$k0tg_0 z00IagfB*srAkb2QJ{eu*1@5@@mhatk_To9#7iejp5F7yn5I_I{1Q0*~0R#|00D)!+ ttQUdStGqz*#Y3A