Инкремент №23

[ex0rcist]

По шифрованию: в пачке писал вопрос, четкого ответа не дали. В итоге сделал шифрование chunk-ами целиком под RSA. Это медленнее, чем если бы мы шифровали AES тело, а RSA только ключи AES. Но быстрее в реализации, т.к. проект учебный, то я не вижу смысла тут особо заморачиваться, особенно в контексте подготовки к собесам и переворачиваниям деревьев.

По json-конфигу: там есть некоторые различия и нестыковки в задании, я сделал чтение из файла, если передан аргумент. При этом аргумент я парсил вручную через os.Args и перед парсингом pflags, потому что согласно доке это можно сделать один раз. А нам надо сначала выяснить, есть ли в аргументах вообще файл конфига. Вообщем как по мне абсолютно дурацкое требование, я в реальной жизни за 15 лет никогда не сталкивался с такими трёхуровневыми костылями конфигурации, надеюсь что не столкнусь.

По сигналам: плавная остановка сервера уже была реализована раньше, добавил плавную остановку агента

Покрытие:

$ make unit-test
...
total: (statements) 71.6%

[Perederey]

Привет! Отлично справился понимаю твою боль, но шифрование оно такое :)

[Perederey]

Выдели константы для сигналов в отдельную группу для лучшей читаемости:

var shutdownSignals = []os.Signal{
    syscall.SIGINT,
    syscall.SIGTERM,
    syscall.SIGQUIT,
}

[Perederey]

Отлично реализован graceful shutdown

[Perederey]

Тут для всех интервалов, используй time.Duration вместо int для интервалов в конфигурации

[Perederey]

Конфигурацию лучше выносить в отдельный пакет. Это упростит тестирование и переиспользование. Например, можно сделать так:

package config

type Config struct {
    Host string
    Port int
}

func New() *Config {
    return &Config{
        Host: "localhost",
        Port: 8080,
    }
}

// В основном коде
cfg := config.New()

Кстати, можно применить паттерн Options для гибкой настройки конфига. Это сделает код более поддерживаемым.

[Perederey]

Хорошо структурированный код с четким разделением ответственности получился у тебя