Add cache file versioning, distrust cache with different version #33
Conversation
TojikCZ
force-pushed
the
metadata-versioning
branch
5 times, most recently
from
2e37f6c to
fe1b5b1
Compare
TojikCZ
changed the title
TojikCZ
force-pushed
the
metadata-versioning
branch
3 times, most recently
from
f667943 to
c9ad956
Compare
TojikCZ
force-pushed
the
metadata-versioning
branch
from
c9ad956 to
6b98c50
Compare
| """Filters out disallowed characters, used with str.filter""" | ||
| return char not in "\",\n} " | ||
|
|
||
| # This expects the first item in the json file to be the |
There was a problem hiding this comment.
kdyz ocekáváš že to má být v první line je potřeba v případě nenajití toho stringu procházet celý soubor?
There was a problem hiding this comment.
Načítám soubor řádek po řádku, jakmile narazím na jakékoli písmenko abecedy, zjistím, jestli načtené znaky začínají slovem "version", pokud ano, pokračuju parsováním verze z momentálního načteného řádku, co začínal slovem version a další řádky nečtu. Pokud ne, vracím false. Jediný případ kdy bych přečetl celý soubor by nastal, pokud by v něm nebyla písmenka, což se v normálním případě nestane, jen někdo se zlým úmyslem by mi mohl soubor zaplnit pouze speciálnímy znaky a čísly a v takovém případě bych doopravdy četl celý soubor.
Pokud by se celý soubor nacházel na prvním řádku, přečtu ho taky celý, to není ideální, ale dokud s tím nebude problém, nechci špinit kód dalšími checky
TojikCZ
dismissed
ondratu’s stale review
Updated to reflect the requested changes, reviewer does not have time to re-review this time
No description provided.