Skip to content

Endurece descargas de GitHub y shares de Samba - #73

Merged
Xalcker merged 1 commit into
mainfrom
fix/descargas-github
Oct 3, 2026
Merged

Xalcker merged 1 commit into
mainfrom
fix/descargas-github

Conversation

@Xalcker

@Xalcker Xalcker commented Oct 3, 2026

Copy link
Copy Markdown
Owner

Cambios

Descargas (#14)

  • `github_api_get`: `GITHUB_TOKEN` opcional y mensaje claro ante HTTP 403/429 (límite de 60 peticiones/hora por IP).
  • Verificación opcional de sha256: `YARG_SHA256` / `CLONEHERO_SHA256`, o el digest que publica la API de GitHub en los canales latest/nightly. Aplica al instalador y a `update-yarg` / `update-clonehero`.
  • Los updaters usan `mktemp` en `/var/tmp` en lugar de rutas fijas en `/tmp`.
  • `update-yarg` avisa cuando el canal `stable` usa URL fija.

Samba (#13)

  • `write_samba_share` unifica los shares de YARG, Clone Hero y RPCS3.
  • `SAMBA_GUEST=false` exige el usuario Samba; `SAMBA_HOSTS_ALLOW` limita las redes.
  • `server min protocol = SMB2` siempre (los clientes solo SMB1 dejan de conectar).
  • `SAMBA_HOSTS_ALLOW` solo se escribe en un `smb.conf` nuevo.
  • SECURITY.md y README documentan los tres shares y las variables.

Pruebas

Suite completa en WSL: 386 pruebas, 0 fallos.

Closes #13
Refs #14 (quedan fuera checksums obligatorios y detección real de versión desactualizada)

🤖 Generated with Claude Code

Descargas:
- github_api_get: usa GITHUB_TOKEN opcional y explica el limite 403/429
- verificacion opcional de sha256 (YARG_SHA256, CLONEHERO_SHA256) o digest de
  la API de GitHub en instaladores y updaters
- updaters usan mktemp en /var/tmp en lugar de rutas fijas en /tmp
- aviso en update-yarg cuando el canal stable usa URL fija

Samba:
- write_samba_share unifica los shares de YARG, Clone Hero y RPCS3
- SAMBA_GUEST=false exige el usuario Samba; SAMBA_HOSTS_ALLOW limita redes
- server min protocol = SMB2 siempre

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@Xalcker
Xalcker merged commit d21ce24 into main Oct 3, 2026
2 checks passed
@Xalcker
Xalcker deleted the fix/descargas-github branch October 3, 2026 19:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Samba: endurecer el share de canciones (guest opcional, hosts allow)

1 participant