fix: умная проверка UFW правил без принудительного reset#3
Merged
Conversation
Исправлена ошибка проверки SSH правила и добавлена интеллектуальная система управления правилами UFW. Изменения: - Добавлена функция check_rule_exists для проверки существующих правил (активный и неактивный UFW) - Модифицирована verify_ssh_rule_exists: комбинированная проверка через ufw status и ufw show added - Модифицированы add_ssh_rule и add_https_rule: проверка перед добавлением, нет дублирования - Модифицирована main: опциональный reset только при подтверждении пользователя - Сохранение существующих правил UFW без необходимости пересоздания Проблемы решены: - ufw status не показывает правила когда UFW неактивен - Автоматический reset удалял все существующие правила - Дублирование правил при повторном запуске скрипта 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
ikeniborn
pushed a commit
that referenced
this pull request
Nov 19, 2025
…lidation) Исправлены 4 критические ошибки деплоя: 1. Ошибка #1: cp не может найти nginx config file - Проблема: stdout contamination в generate_nginx_config() - Решение: Перенаправить info() в stderr через >&2 - Файл: scripts/nginx-setup.sh:91 2. Ошибка #2: Certbot не может получить SSL (port 80 занят) - Проблема: certbot --standalone требует свободный порт 80 - Решение: Автоматически останавливать nginx перед certbot, запускать обратно после - Файл: scripts/ssl-setup.sh:97-137 3. Ошибка #3: NETWORK_NAME variable is not set - Проблема: Недостаточная валидация .env переменных - Решение: Добавить NETWORK_NAME и NETWORK_MODE в required_vars - Файл: deploy.sh:94 4. Ошибка #4: UFW is not active - Проблема: Недостаточно информативный error message - Решение: Улучшить error message с инструкциями - Файл: deploy.sh:104-112 Дополнительно: - Обновлена документация в docs/troubleshooting.md - Добавлена секция "Проблемы с деплоем" с описанием всех 4 ошибок - Добавлен Pre-deployment Checklist Тестирование: - Все скрипты прошли bash syntax check 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Исправлена ошибка проверки SSH правила и добавлена интеллектуальная система управления правилами UFW.
Изменения:
Проблемы решены:
🤖 Generated with Claude Code
Co-Authored-By: Claude noreply@anthropic.com