feat: add support for root domain www.abyaya.la #84
No Assignees
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: escuela-comun/abyayala#84
Reference in New Issue
Block a user
No description provided.
Delete Branch "root-domain"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Add capability to route root domain (abyaya.la and www.abyaya.la)
to sutty.comun while maintaining all existing subdomain routes.
Changes:
Technical details:
www.abyaya.la (no *.abyaya.la wildcard)
Generated with Claude Code
Aviso esto es pura IA hay que checkar
Más IA de cómo se mergeará esto con el PR #76
(Me llama la atención que no se haga tanto problema con los domains ..?)
📊 Análisis de Merge: issue42 → root-domain
✅ Buenas Noticias
El merge se realizará automáticamente sin conflictos. Git puede fusionar ambas ramas exitosamente.
🔍 Cambios que Traerá issue42
Cambio en issue42:
Integración con root-domain:
Resultado del merge:
Tarea 1: with wildcard (línea 32)
command: "--non-interactive --agree-tos --expand --email ..." # ✓ Incluye --expand
Tarea 2: without wildcard (línea 50)
command: "--non-interactive --agree-tos --email ..." # ✗ NO tiene --expand
Recomendación: Deberías agregar --expand también a la tarea "without wildcard" para mantener consistencia.
Cambio en issue42:
Integración con root-domain:
Cambio en issue42:
Integración con root-domain:
🎯 Orden de Merge Recomendado
Opción 1: issue42 primero (Recomendado)
1. Merge issue42 a master
git checkout master
git merge issue42
2. Merge root-domain a master
git merge root-domain
Ventaja:
Opción 2: root-domain primero
1. Merge root-domain a master
git checkout master
git merge root-domain
2. Merge issue42 a master
git merge issue42
Desventaja:
⚠️ Acción Necesaria Después del Merge
Independientemente del orden, deberás actualizar la tarea "without wildcard" para incluir --expand:
docker_container:
...
command: "--non-interactive --agree-tos --expand --email {{ webmaster_email }} certonly ..."^^^^^^^^ Agregar esto
📋 Resumen
Conclusión: El merge será limpio y funcional. Solo necesitas recordar agregar --expand a la segunda tarea de certbot después del merge.
● ✅ root-domain: Merge exitoso sin conflictos
● Bash(git merge issue42)
⎿ Error: Exit code 1
Auto-fusionando roles/proxy/tasks/main.yml
CONFLICTO (contenido): Conflicto de fusión en roles/proxy/tasks/main.yml
Fusión automática falló; arregle los conflictos y luego realice un commit con el resultado.
● ⚠️ issue42: CONFLICTO DETECTADO en roles/proxy/tasks/main.yml
Déjame mostrarte el conflicto:
● Read(roles/proxy/tasks/main.yml)
⎿ Read 108 lines
● Conflicto en roles/proxy/tasks/main.yml (líneas 48-67):
Hay dos versiones diferentes de cómo manejar los dominios por defecto:
Versión HEAD (testing):
Versión issue42:
✅ root-domain - Merge exitoso sin conflictos
✅ issue42 - Conflicto resuelto en roles/proxy/tasks/main.yml (mantenida versión issue42)
✅ feat-cert-auto-renewal - Merge exitoso sin conflictos
✅ refactor-simplify-node-naming - Merge exitoso sin conflictos
✅ feat/parametrizar_redes - Ya estaba actualizado
Todos los merges completados. La rama testing ahora está 15 commits adelante de origin/testing.
Rename the flag from `no_wildcard` to `root` throughout the codebase for better semantics, and fix nginx configuration generation for root domains. Changes: 1. Renamed `no_wildcard` → `root` flag in: - abyayala.yml (abyaya_root service) - roles/certbot/tasks/certbot.yml (uses is_root_domain internally) - README-root-domain.md (documentation) 2. Fixed nginx vhost generation in roles/proxy/templates/: - vhost.conf: Handle root domains without leading dot in server_name * root: yes → `server_name abyaya.la www.abyaya.la;` (exact match) * root: no → `server_name .comun.abyaya.la;` (wildcard match) - stream.conf: Same logic for SSH proxy streams Problem fixed: - Previous: `.abyaya.la` matched all subdomains, conflicting with other vhosts (comun.abyaya.la, sutty.abyaya.la, etc.) - Now: `abyaya.la www.abyaya.la` matches only root domain exactly 🤖 Generated with Claude Code Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>Funcional con
acdf35e5a7(falló exitosamente)A qué huerta hay que apuntarlo @f?
Notar que hubo que usar webroot para el root
Y que la primera vez hubo que ejecutar con --skip-tags=certbot y sin force_https para habilitar el webroot justamente, y a la segunda obtener el cert (en Numerica se levanta ahi un server temporal que es un bardo)
Pull request closed