martin-nohava / kryzbu

Kryptograficky zabezpečené úložiště 🔒
MIT License
1 stars 0 forks source link

Zajistit kompatibilitu s Unix-like systémy #7

Closed martin-nohava closed 2 years ago

martin-nohava commented 2 years ago

V současné podobě kód podporuje pouze platformu Windows.

martin-nohava commented 2 years ago

Podpora pro upload přidána v 8f3cf6ffb35fa89fb6976a89842339700f10bbb8

martin-nohava commented 2 years ago

Byla navržena i nová struktura složek na straně serveru pro ukládání souborů a servisních dat. (Viz obrázek.) Po uploadu se soubor uloží do adresáře server/_data/files/<jmeno_souboru.koncovka>.

Navrhuji úpravu obsahu proměnné file_name v client.py. V současné době neobsahuje pouze název souboru, ale (obsahuje/může obsahovat) i cestu k němu. Je to zmatečné a na straně serveru zbytečná informace. Lze ořezat pomocí os.path.basename(filename). Názory? @Bloc3k

Snímek obrazovky 2022-02-28 v 21 24 56

Bloc3k commented 2 years ago

změna nazvu proměné file_name je dobrý navrh a změnu provedeme. Teď se to ořezává na serveru. Jak jsi říkal serveru stačí vědět jméno pod kterým má soubor uložit. Navrhuji aby se ořezávání provádělo již na straně klienta (podobně jak to udělal kašpy v jeho commitu).

Bloc3k commented 2 years ago

Pullnul jsem si commit 8f3cf6 spustil a vyhodilo tento error.

image

Zjistil jsem že jako referenční adresář mi to používá kořenový adresář projektu \kryzbu\ tudiž nevidí přímo adresář \server\.

Úprava 16 řádku v server.py:

SERVER_FOLDER = Path("src/server/_data/files/")

pomohla a pak to funguje. Ale tipuji že pak se to rozbije tobě. Co s tím nevím. proto píšu

martin-nohava commented 2 years ago

Úprava 16 řádku v server.py:

SERVER_FOLDER = Path("src/server/_data/files/")

pomohla a pak to funguje. Ale tipuji že pak se to rozbije tobě. Co s tím nevím. proto píšu

Možná bude záležet z jakého adresáře to pouštíš. Zapínáš to jako python3 server.py a nebo python3 src/server.py a nebo něco takového.

Bloc3k commented 2 years ago

Ano spouštěl jsem to z VSC a ten to bral z root adresáře. už to funguje

Jinak návrh na změnu názvu proměnné byl přijat a proměnná změněna v 460ff0122b040ea46af994165f4cf9566218e18c, proto tuto issue timto uzavírám.