Repository navigation
Port uyuşmazlığı, kimlik doğrulamasız silme API'si ve SHA-256 doğrulaması / Port mismatch, unauthenticated delete API, no SHA-256 #8
Description
Merhaba @woloop, Docker sürümü için eline sağlık. Kodu incelerken birkaç nokta gözüme çarptı:
- Port uyuşmuyor. README ve install.sh adres olarak :7572 yazıyor, ama docker-compose.yml yalnızca 127.0.0.1:7573:7572 portunu açıyor. Yazılan adres çalışmıyor; NAS'tan ya da telefondan erişim de mümkün değil.
- Silme API'sinde kimlik doğrulaması yok. DELETE /api/packages/{prefix} verilen adı glob deseni olarak kullanıyor; prefix olarak * gönderilirse tüm ZIM'ler silinir. Servis yalnızca 127.0.0.1'e açıkken sorun değil, ama NAS senaryosu için port ağa açılırsa ağdaki herkes kütüphaneyi silebilir. prefix'i katalogdaki adlarla sınırlamak ve yönetim uçlarına basit bir parola eklemek iyi olur.
- SHA-256 doğrulaması yok. Kiwix her ZIM'in yanında .sha256 yayımlıyor; ana repodaki PR Fix retired ZIM names, add Turkish content, catalog-based versions + SHA-256 / Ölü ZIM adları, Türkçe içerik, katalog tabanlı sürüm + SHA-256 #3 bunu nasıl kullandığını gösteriyor.
- Eski içerik. content.conf'ta WikEM kapatılmış ama ana sayfada hala görünüyor; fas-military-medicine_en'in yeni adı irp.fas.org_en_military-medicine (ana repoda PR Fix retired ZIM names, add Turkish content, catalog-based versions + SHA-256 / Ölü ZIM adları, Türkçe içerik, katalog tabanlı sürüm + SHA-256 #3).
- Çeviri uyarısı. Çeviri modülü Argos modellerini kullanıyor; ana repodaki issue [Öneri] Yabancı kaynaklar için çevrimdışı Türkçe çeviri ve yabancılarla konuşma (Pi 5 ölçümleriyle) #6'daki Pi 5 ölçümleri Argos'u tıbbi metinde kullanılamaz buldu. Çeviri sayfasına en azından "makine çevirisidir, tıbbi kararlarda güvenmeyin" uyarısı eklemek iyi olur.
Yardımcı olabileceğim bir yer olursa yazabilirsin.
Reacted by Yağız
Activity
woloop commentedon Nov 1, 2026
Merhaba @volthdl,
Detaylı incelemen ve bu son derece yerinde, yapıcı tespitlerin için gerçekten çok teşekkür ederim. Belirttiğin tüm maddeleri tek tek ele aldım, ve sanırsam çözdüm ve güncellemeleri ilgili repoya yükledim.
Geri bildirimlerin ve projeye katkın için tekrar teşekkürler!
schematrix commentedon Nov 2, 2026
@volthdl, bu kadar detaylı bir inceleme için teşekkürler. Özellikle silme API'sindeki açık ve çeviri modülüne tıbbi uyarı eklenmesi önerisi çok yerinde. @woloop, hızlı dönüşün ve düzeltmelerin için sana da teşekkürler.
Yalnız bu issue aslında @woloop'un Docker fork'uyla ilgili, bu repoyla değil. Takibi kolay olsun diye bu tür incelemeleri fork'un kendi reposunda açmak daha doğru olur. #7'deki "Topluluk sürümleri" bağlantısını eklemeden önce bu düzeltmelere ben de bakacağım. O yüzden bu issue'yu burada kapatıyorum, konuşmaya #7 altında devam edebiliriz.
@volthdl, thanks for such a detailed review. The unauthenticated delete API and the suggestion to add a medical warning to the translation module are especially spot on. @woloop, thanks to you too for the quick response and the fixes.
That said, this issue is really about @woloop's Docker fork rather than this repo. To keep things easy to track, it's better to open reviews like this in the fork's own repo. I'll check these fixes myself before adding the "Community editions" link from #7. So I'm closing this issue here, and we can continue the conversation under #7.
volthdl commentedon Nov 2, 2026
Haklısınız, yanlış repoya açmışım, kusura bakmayın. Devamını woloop'un reposunda açıyorum ve kısa bir özeti #7'ye yazıyorum.
You're right, I opened this in the wrong repo, sorry. I'm moving the follow-up to woloop's repo and posting a short summary under #7.