Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Почистить мидвари #53

Merged
merged 10 commits into from
Dec 17, 2023

Conversation

thefrol
Copy link
Owner

@thefrol thefrol commented Dec 17, 2023

  • реализовано похищение тела запроса по рекомендации Влада
  • добавить тесты для подписывания
  • удалить лишние хелпер-функции
  • убрать баг в агенте, где лишний раз сбрасыавлся PollCount
  • получше отформатировать и подписать код и переменные

@thefrol thefrol merged commit 8ba9e16 into fix/rework-before-sprint7 Dec 17, 2023
1 of 3 checks passed
@thefrol thefrol deleted the fix/replace-request-body branch December 17, 2023 16:08
thefrol added a commit that referenced this pull request Dec 23, 2023
* Удалить ненужные пакеты (#51)

* fix: lib удалить старый sheduler

* fix: lib удалить пакет slices

* fix: collector перенести compress в collector

---------

Co-authored-by: Dima Frolenko <[email protected]>

* Почистить мидвари (#53)

* fix: middleware/singing по-новому перехватиывать тело запроса

* fix: middleware Подписать буферы

* fix: api/middleware возврашать 400 при ошибке подписи

* test: middleware Добавить  тесты подписывающей мидлвари

* fix: middleware разделить подписывание на две мидлвари

* fix: report исправить баг с дополнительным сбросом полкаунт при отправке

* fix: report переименовать переменную с клиентом

* test: report убрать вспомогательные функции

* fix: переместить пакет compress

* test: middleware Добавить упушенную мидлварь

---------

Co-authored-by: Dima Frolenko <[email protected]>

* Выделить прикладной уровень + хендлеры (#54)

* feat: server/app добавый слой приложения метрик

* feat: domain добавить новую ошибку

* fix: manager исправить сигнатуру интерфейса

* feat: storage Добавить адаптер для нового приложения

* feat: server Добавить новое апи

* fix: router заменить один маршрут на новое апи

* fix: router /update/counter на новом хендлере

* fix: manager исправить сигнатуру юзкейса, обновляющего гауж

* fix: api добавить хендлеры через новое приложение manager

* fix: router отслеживать невалидные запросы

* fix: handlers исправить названия структур

* fix: app добавить приложение для получение всех названий метрик

* feat: storage Добавить методы получения списка метрик

* fix: router добавить хендлеры для дэшборда

* fix: router получше оформить код

* fix: handlers добавить новый JSON хендлер

* feat: server добавить новый юзкейс - работа с моделью Metrics

* fix: handlers Использовать новый юзкейс metrics

* feat: app Обновлять метрикой на уровне приложения

* fix: router обновить хендлер /update

* feat: metrica Добавить массив метрик с easyjson

* fix: manager Обновлять пачкой на новых юзкейсах

* feat: server новый юзкейс по пингованию БД

* fix: handlers возвращать правильный статус при ошибке джейсона

* fix: удалить лишние хендлеры

* fix: server образовать пакет httpio

* fix: hanlders убрать лишний хелпер для контент-тайпа

* fix: httoio удалить неиспользуемые retry функции

* fix: handlers новые хелперы ошибок

* fix: api удалить старые комменты, отметить тудушку

* fix: middleware закрывать тела ответов в тестах

---------

Co-authored-by: Dima Frolenko <[email protected]>

* Расхламить сервер (#55)

* fix: router новый способ инициализации

* fix: router исправить тесты под новую сигнатуру

* fix: реворк нежной остановки сервера

* fix: server перенести мидлвари и хендлеры в роутер

* fix: перенести storage  в server

* fix: domain перенести ошибки в app

* fix: httpio отметить файл с интерфейсом как депрекейтед

* doc: hanlders обнаружить баг

---------

Co-authored-by: Dima Frolenko <[email protected]>

* fix: httpio Возвращать text/plain при ошибке

* fix: перенести названия заголовков в константы

Squashed commit of the following:

commit 9cd9a6e252367866e5621e91ce9ab81e56a41956
Author: Dima Frolenko <[email protected]>
Date:   Wed Dec 20 13:08:35 2023 +0300

    fix: report заголовки в константы
    Content-Encoding

commit 09328b1ddc30e3eacde925891f461ebd55e69cc3
Author: Dima Frolenko <[email protected]>
Date:   Wed Dec 20 13:08:09 2023 +0300

    fix: httpi еще заголовки в константы

    Content-Encoding
    Content-Length

commit 201adc3cf94ab7433b2909b33728065203aa75c6
Author: Dima Frolenko <[email protected]>
Date:   Wed Dec 20 13:03:36 2023 +0300

    fix: sign переименовать константу для заголовка

commit 2e005353c54745742086534c8ef59ffe7da81842
Author: Dima Frolenko <[email protected]>
Date:   Wed Dec 20 12:59:52 2023 +0300

    fix: использовать константу для контент-тайп

* Переделать мемстораж (#56)

* feat: storagev2 реализовать мемстор для счетчиков

* feat: app поменять интерфейсы репозиториев

* fix: storagev2 добавить тесты

* fix: storagev2 разбить класс по файлам

* feat: storagev2 реализровать гепозиторий гаужей

* fix: storagev2 новый инкерфейс получения всех меток

* feat: storagev2 добавить новый способ выдавать метки

* fix: storagev2 переимновать файл для тестов

* test: storagev2 тестировать получение названий метрик

* fix: storagev2 Почистить оъявление структуры

* fix: storagev2 переименовать файл

* test: storagev2 добавить тесты для гаужей

* test: storage v2 добавить тест на чтение из файла

* fix: storagev2 исправить сохранение в файл

* fix: cmd/server теперь использует storagev2/mem

* feat: storagev2 поменять логику Restore

* feat: storagev2 поменять сигнатуру Dump()

* fix: server Завершать хранилище через defer

* fix: storagev2 Избавиться от гонки при выводе

* fix: server сохраняться перед выходом при интервальном ханении

* feat: storage интервальный сохранятель вывести в отдельный класс

* fix: server избавиться от старого контекста БД

* fix: memstorev2 по другому читать из файла

* fix: storagev2 задокументировать интервальную запись

* docs: storagev2 задокументировать MemStore

* test: memstore Расширить тест

* fix: memstore2 - убрать лишнюю проверку на nil

---------

Co-authored-by: Dima Frolenko <[email protected]>

* Новый sql-репозиторий на sqlc (#57)

* feat: добавить миграции

* fix: добавить sqlc спеку

* feat: storagev2 сгенерировать код хранилища

* sql: обновить схему

* fix: app перенести ошибки в прикладной слой

* fix: storagev2 добавить соединение с постгрес

* fix: storagev2  реазировать crud по метрикам postgres

* fix: sqlrepo перегенерировать с целевым пакетом database/sql

* fix: server использовать только новое хранилище

* fix: sqlc исправить запрос IncrementCounter()

* fix: memstore исправить загрузку из файла в пустые мапы

* fix: cmd Документировать запуск сервера

* fix: cmd продолжать работу, если файла хранилища нет

---------

Co-authored-by: Dima Frolenko <[email protected]>

* Удалить старое хранилище (#58)

* fix: storage удалить

* fix: sqlrepo добавить логгер

* fix: cmd структурировать лучше запуск

* fix: sqlrepo реализовать вывод списка метрик

* fix: app переименовать юзкейс в dashboard

* docs: server/app - задокументировать юз-кейсы

* fix: memstore потерянное изменение

---------

Co-authored-by: Dima Frolenko <[email protected]>

* cicd: обновить воркфлоу на гитхабе (#59)

Co-authored-by: Dima Frolenko <[email protected]>

---------

Co-authored-by: Dima Frolenko <[email protected]>
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.

1 participant