hot fix for 2.3.5

Author: jul-den

README.md

@@ -1,9 +1,9 @@
 # Simple Python FTP Server for Scanning Devices
 
-A lightweight, configurable FTP server designed specifically for receiving scanned documents from MFU devices. Built with Python and pyftpdlib, it supports both background daemon mode and interactive console operation with real-time configuration.
+A lightweight, configurable FTP server designed specifically for receiving scanned documents from MFP devices. Built with Python and pyftpdlib, it supports both background daemon mode and interactive console operation with real-time configuration.
 
 ## ⚠️ Current Status: Stable Beta
-Core functionality is production-ready and tested with MFU devices. The server is stable for everyday use but has some technical debt that needs addressing.
+Core functionality is production-ready and tested with MFP devices. The server is stable for everyday use but has known technical issues that require resolution.
 
 ## ✨ Features
 
@@ -12,7 +12,7 @@ Core functionality is production-ready and tested with MFU devices. The server i
 - **User Management**: Multiple user accounts with customizable permissions and access control
 - **Transfer Modes**: Configurable passive and active data transfer modes
 - **Anonymous Access**: Optional anonymous FTP access with restricted permissions
-- **Cross-Platform**: Works on Windows, Linux, and macOS (tested primarily on Windows)
+- **Cross-Platform**: Works on Windows, Linux, and macOS (tested primarily on Windows 10, additional testing required for other OSes)
 
 ## 🚀 Getting Started
 
@@ -97,14 +97,13 @@ Interactive console provides real-time management for:
 - Directory listing and file operations
 - Configurable data port ranges
 
-## 🐛 Known Technical Debt and Limitations
+## 🐛 Known Technical Issues and Limitations
 
 ### Architectural Issues
-- **Circular imports** between modules causing maintenance challenges
 - **SOLID principles violations** - classes have too many responsibilities
 - **Mixed concerns** - business logic mixed with configuration management
 
-### Security Limitations
+### Security Issues
 - **Password hashing incomplete** - implementation needs proper salting and storage
 - **Plaintext passwords** stored even when hashing is configured
 - **Input validation missing** for interactive mode user inputs
@@ -130,10 +129,9 @@ Interactive console provides real-time management for:
 ## 🗺️ Development Roadmap
 
 ### High Priority Fixes
-- [ ] Fix circular imports and improve module structure
+- [ ] Fix critical issues identified in the current version
 - [ ] Complete password hashing implementation with proper salting
 - [ ] Add input validation for interactive mode
-- [ ] Implement configuration caching to avoid repeated parsing
 - [ ] Fix PID file race conditions and Windows process detection
 
 ### Medium Priority Improvements
@@ -144,11 +142,10 @@ Interactive console provides real-time management for:
 - [ ] Fix threading issues and add proper timeouts
 
 ### Code Quality Enhancements
-- [ ] Eliminate code duplication
-- [ ] Replace magic numbers with constants
-- [ ] Improve resource management and cleanup
 - [ ] Add proper signal handling for different modes
 - [ ] Fix encoding issues for cross-platform compatibility
+- [ ] **No log rotation** - Log files can grow indefinitely. Need to add rotation and corresponding settings via configuration file
+- [ ] **No health-check mechanism** - Need to implement server health monitoring
 
 ## 🤝 Contributing
 
@@ -164,7 +161,7 @@ We welcome contributions! Please feel free to submit pull requests, report bugs,
 
 ### Common Issues
 - **Port already in use**: Change default port in configuration
-- **Permission denied**: Check directory permissions for anonymous access
+- **Permission denied**: Check directory permissions
 - **Connection timeout**: Verify firewall settings and passive port ranges
 
 ### Getting Help
@@ -178,9 +175,11 @@ This project is licensed under the MIT License - see the [LICENSE.md](LICENSE.md
 
 ## 🙏 Acknowledgments
 
-- Built on [pyftpdlib](https://github.com/giampaolo/pyftpdlib) - Extremely fast and scalable Python FTP server library
+- Built on [pyftpdlib](https://github.com/giampaolo/pyftpdlib) - extremely fast and scalable Python FTP server library
 - Inspired by real-world document management processes
 
 ---
 
-**Note**: This software is stable for production use with MFU devices but has technical debt that will be addressed in future versions.
+**Note**: This software is stable for production use with MFP devices but has known technical issues planned for resolution in future versions.
+
+Passwords are stored in plain text - **secure the configuration file**.

README_ru.md

@@ -3,7 +3,7 @@
 Легкий и настраиваемый FTP сервер, предназначенный для получения отсканированных документов с МФУ устройств. Поддерживает работу в фоновом режиме и интерактивном консольном режиме с интерактивной конфигурацией.
 
 ## ⚠️ Текущий статус: Стабильная бета-версия
-Основная функциональность готова к промышленному использованию и протестирована с МФУ устройствами. Сервер стабилен для повседневного использования, но имеет технический долг, который требует устранения.
+Основная функциональность готова к промышленному использованию и протестирована с МФУ устройствами. Сервер стабилен для повседневного использования, но имеет технические проблемы, которые требуют устранения.
 
 ## ✨ Возможности
 
@@ -12,7 +12,7 @@
 - **Управление пользователями**: Множественные учетные записи с настраиваемыми правами доступа
 - **Режимы передачи**: Настраиваемые пассивный и активный режимы передачи данных
 - **Анонимный доступ**: Опциональный анонимный доступ с ограниченными правами
-- **Кроссплатформенность**: Работает на Windows, Linux и macOS (в основном тестируется на Windows)
+- **Кроссплатформенность**: Работает на Windows, Linux и macOS. Основное тестирование проводилось на Windows 10 (для других ОС требуется дополнительное тестирование)
 
 ## 🚀 Начало работы
 
@@ -100,7 +100,6 @@ python src/ftp_server.py --service-stop
 ## 🐛 Известные технические проблемы и ограничения
 
 ### Архитектурные проблемы
-- **Циркулярные импорты** между модулями усложняют поддержку
 - **Нарушение принципов SOLID** - классы берут на себя слишком много ответственности
 - **Смешение ответственности** - бизнес-логика перемешана с управлением конфигурацией
 
@@ -130,10 +129,9 @@ python src/ftp_server.py --service-stop
 ## 🗺️ План разработки
 
 ### Высокоприоритетные исправления
-- [ ] Исправить циркулярные импорты и улучшить структуру модулей
+- [ ] Исправить выявленные критические проблемы тикужей версии
 - [ ] Завершить реализацию хеширования паролей с добавлением соли
 - [ ] Добавить проверку ввода для интерактивного режима
-- [ ] Реализовать кэширование конфигурации для избежания повторного парсинга
 - [ ] Исправить состояния гонки PID файлов и определение процессов Windows
 
 ### Среднеприоритетные улучшения
@@ -144,11 +142,10 @@ python src/ftp_server.py --service-stop
 - [ ] Исправить проблемы с потоками и добавить правильные таймауты
 
 ### Улучшения качества кода
-- [ ] Устранить дублирование кода
-- [ ] Заменить магические числа на константы
-- [ ] Улучшить управление ресурсами и очистку
 - [ ] Добавить правильную обработку сигналов для разных режимов
 - [ ] Исправить проблемы с кодировками для кроссплатформенной совместимости
+- [ ] Нет ротации логов - Файлы логов могут расти бесконечно. Нужно добавить ротацию и соотвествующие настройки через файл конфигурации
+- [ ] Нет механизма "health-check"
 
 ## 🤝 Участие в разработке
 
@@ -164,7 +161,7 @@ python src/ftp_server.py --service-stop
 
 ### Частые проблемы
 - **Порт уже используется**: Измените порт по умолчанию в конфигурации
-- **Доступ запрещен**: Проверьте права доступа к директориям для анонимного доступа
+- **Доступ запрещен**: Проверьте права доступа к директориям
 - **Таймаут соединения**: Проверьте настройки фаервола и диапазоны пассивных портов
 
 ### Получение помощи
@@ -183,4 +180,6 @@ python src/ftp_server.py --service-stop
 
 ---
 
-**Примечание**: Данное программное обеспечение стабильно для промышленного использования с МФУ устройствами, но имеет технический долг, который будет устранен в будущих версиях.
+**Примечание**: Данное программное обеспечение стабильно для промышленного использования с МФУ устройствами, но имеет технические проблемы, которые планируется устранить в будущих версиях.
+
+Пароли хранятся в открытом виде - **защитите конфигурационный файл**.

src/ftp_server_users.py

@@ -179,7 +179,7 @@ def toggle_user(config_manager: ConfigManager):
 def delete_user(config_manager: ConfigManager):
     """Удаление пользователя"""
     logger = config_manager.get_logger()
-    print_users(config_manager, logger)
+    print_users(config_manager)
 
     user_id = get_user_id_from_input(config_manager, "\nВведите номер пользователя для удаления: ")
     if not user_id:

src/version_info.txt

@@ -1,8 +1,8 @@
 # UTF-8
 VSVersionInfo(
   ffi=FixedFileInfo(
-    filevers=(2, 3, 5, 0),
-    prodvers=(2, 3, 5, 0),
+    filevers=(2, 3, 6, 1),
+    prodvers=(2, 3, 6, 1),
     mask=0x3f,
     flags=0x0,
     OS=0x4,
@@ -17,12 +17,12 @@ VSVersionInfo(
         [
           StringStruct(u'CompanyName', u'ФГБУ Российская государственная библиотека'),
           StringStruct(u'FileDescription', u'Простой FTP сервер для сохранения файлов со сканера сетевого МФУ'),
-          StringStruct(u'FileVersion', u'2.3.5'),
+          StringStruct(u'FileVersion', u'2.3.6'),
           StringStruct(u'InternalName', u'FTPServer'),
           StringStruct(u'LegalCopyright', u'Copyright © 2025 Баранчук Ю. Н. ФГБУ "РГБ"'),
           StringStruct(u'OriginalFilename', u'ftp_server.exe'),
           StringStruct(u'ProductName', u'FTP Server for Scanner'),
-          StringStruct(u'ProductVersion', u'2.3.5'),
+          StringStruct(u'ProductVersion', u'2.3.6'),
           StringStruct(u'ProgrammingLanguage', u'Python'),
           StringStruct(u'Author', u'Баранчук Юлий Николаевич'),
           StringStruct(u'Comments', u'Создано с помощью Python и PyInstaller')