fix: address PR review comments for OAuth2 implementation #32

Merged
skettiks merged 25 commits from feat/oauth2-user-service-26 into develop 2026-05-16 19:30:55 +00:00
25 Commits
Author SHA1 Message Date
skettiks 56f1f0a46b . 2026-05-15 22:46:38 +03:00
skettiks d2821a8439 Revert "- exclude("**/security.disabled/**") - убрано"
This reverts commit bc06654c84.
2026-05-15 22:40:50 +03:00
skettiks bc06654c84 - exclude("**/security.disabled/**") - убрано
-Добавлен плагин org.graalvm.buildtools.native в build.gradle.kts, чтобы появились задачи processAot и nativeCompile.
- Отключен configuration-cache в gradle.properties, чтобы убрать конфликт при nativeCompile на текущем toolchain.
2026-05-15 22:20:03 +03:00
skettiks a4f99bbe8a * Синхронизировал UserEntity и UserRepository с актуальной моделью User и контрактом UserRepositoryPort: убрал устаревшую логику с password и исправил невалидные override.
* Убрал конфликтующие gRPC-зависимости, которые тянули несовместимый runtime/test stack.
* Сделал Flyway-миграцию `V2` с OAuth2-индексом совместимой с H2.
* Отключил security-фильтры в controller-тестах через `MockMvc`, чтобы убрать редиректы на авторизацию и нестабильные падения тестов.
* Восстановил локальный запуск и стабилизировал тестовый контур.
2026-05-15 21:18:46 +03:00
skettiks 432a803b8f Merge branch 'feat/oauth2-user-service-26' of https://github.com/devitq/movienight-backend into feat/oauth2-user-service-26
# Conflicts:
#	src/main/kotlin/com/project/movienight/adapters/persistence/jdbc/UserRepository.kt
2026-05-15 20:47:13 +03:00
skettiksandGitHub 071fa53858 Merge branch 'develop' into feat/oauth2-user-service-26 2026-05-14 21:12:38 +03:00
d4dbc108c8 fix: resolve CI compile failure and complete OAuth2 review fixes
Agent-Logs-Url: https://github.com/devitq/movienight-backend/sessions/62eaa8e4-560b-4737-be4b-478f1a4c484a

Co-authored-by: devitq <118541411+devitq@users.noreply.github.com>
2026-05-14 21:01:25 +03:00
Elena Ponomarevaandskettiks aecf19aa11 изменила название папки и убрала лишние импорты 2026-05-14 21:00:46 +03:00
Elena Ponomarevaandskettiks 82a0cec1cc дописала security 2026-05-14 21:00:46 +03:00
skettiks a10737a7f7 Исправлена сборка проекта после слияния с develop
Проблема:
- После слияния с develop возникли конфликты в реализации OAuth2
- Две ветки независимо реализовали OAuth2 функциональность по-разному
- Сборка проекта падала из-за отсутствия зависимостей OAuth2

Изменения:
- Временно отключена OAuth2 зависимость в build.gradle.kts
- Перенесен OAuth2 код в папку security.disabled для сохранения
- Добавлены исключения security.disabled из компиляции, ktlint и detekt
- Удалена миграция V2__add_oauth2_fields.sql (OAuth2 поля теперь в V1)
- Удалено поле password из User domain модели
- Обновлены репозитории и сервисы для работы с новой схемой БД

Результат:
- Проект успешно собирается (./gradlew build)
- Все 35 тестов проходят
- OAuth2 код сохранен для будущего использования
2026-05-14 21:00:06 +03:00
skettiks cf7e3411da - Перенесён OAuth2UserInfo.kt из adapters/security/ в application/ports/input/security/; обновлены импорты во всех зависимых файла
- В UserPrincipal.kt заменён star import java.util.* на явный java.util.UUID.

- В YandexOAuth2UserInfo.kt и VkOAuth2UserInfo.kt убрана аннотация @Suppress("UNCHECKED_CAST")

- В build.gradle.kts хардкод версии заменён на version catalog

- Из CustomOAuth2UserService.kt удалены комментарии на русском языке
2026-05-14 20:57:58 +03:00
skettiks 166c5be863 Исправлена V1 миграция - добавлена запятая после email
Реализован saveWithOAuth2() в UserRepository
Реализован findByProviderAndProviderId() в UserRepository
Обновлен CustomOAuth2UserService для использования OAuth2 методов
Исправил package declaration во всех 7 OAuth2 файлах
2026-05-14 20:57:58 +03:00
skettiks 88794e6a96 Добавил V2__add_oauth2_fields.sql
Все UserRepositoryIntegrationTest тесты прошли
2026-05-14 20:54:30 +03:00
Elena Ponomarevaandskettiks 06e2b50ba5 OAuth2 (без тестов и ошибка с java) 2026-05-14 20:50:22 +03:00
0b57c31611 fix: resolve CI compile failure and complete OAuth2 review fixes
Agent-Logs-Url: https://github.com/devitq/movienight-backend/sessions/62eaa8e4-560b-4737-be4b-478f1a4c484a

Co-authored-by: devitq <118541411+devitq@users.noreply.github.com>
2026-05-08 17:37:35 +00:00
Elena Ponomareva 0aa7dcaf98 изменила название папки и убрала лишние импорты 2026-05-07 08:01:24 +03:00
Elena Ponomareva 6158d3543c дописала security 2026-05-07 07:58:11 +03:00
skettiks 27317c00e4 Исправлена сборка проекта после слияния с develop
Проблема:
- После слияния с develop возникли конфликты в реализации OAuth2
- Две ветки независимо реализовали OAuth2 функциональность по-разному
- Сборка проекта падала из-за отсутствия зависимостей OAuth2

Изменения:
- Временно отключена OAuth2 зависимость в build.gradle.kts
- Перенесен OAuth2 код в папку security.disabled для сохранения
- Добавлены исключения security.disabled из компиляции, ktlint и detekt
- Удалена миграция V2__add_oauth2_fields.sql (OAuth2 поля теперь в V1)
- Удалено поле password из User domain модели
- Обновлены репозитории и сервисы для работы с новой схемой БД

Результат:
- Проект успешно собирается (./gradlew build)
- Все 35 тестов проходят
- OAuth2 код сохранен для будущего использования
2026-05-05 00:58:20 +03:00
skettiksandGitHub 649978f791 Merge branch 'develop' into feat/oauth2-user-service-26 2026-05-03 16:06:02 +03:00
skettiksandGitHub c03b49157f Merge branch 'develop' into feat/oauth2-user-service-26 2026-05-03 15:13:14 +03:00
skettiks 312c18645b - Перенесён OAuth2UserInfo.kt из adapters/security/ в application/ports/input/security/; обновлены импорты во всех зависимых файла
- В UserPrincipal.kt заменён star import java.util.* на явный java.util.UUID.

- В YandexOAuth2UserInfo.kt и VkOAuth2UserInfo.kt убрана аннотация @Suppress("UNCHECKED_CAST")

- В build.gradle.kts хардкод версии заменён на version catalog

- Из CustomOAuth2UserService.kt удалены комментарии на русском языке
2026-04-23 23:59:16 +03:00
skettiks d50e2e640c Исправлена V1 миграция - добавлена запятая после email
Реализован saveWithOAuth2() в UserRepository
Реализован findByProviderAndProviderId() в UserRepository
Обновлен CustomOAuth2UserService для использования OAuth2 методов
Исправил package declaration во всех 7 OAuth2 файлах
2026-04-23 18:47:30 +03:00
skettiks 689ee91d1f Merge remote-tracking branch 'origin/feat/oauth2-migration-20' into feat/oauth2-user-service
merge: интеграция полей OAuth2 миграции в user service.
2026-04-23 17:18:57 +03:00
Elena Ponomareva c64fc35353 OAuth2 (без тестов и ошибка с java) 2026-04-22 08:46:58 +03:00
skettiks 3cb52c019f Добавил V2__add_oauth2_fields.sql
Все UserRepositoryIntegrationTest тесты прошли
2026-04-21 16:28:47 +03:00