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

add oauth2 github authentification #159

Draft
wants to merge 16 commits into
base: main
Choose a base branch
from
Draft

Conversation

devbulygin
Copy link

Добавил первую версию oauth2, подскажите куда двигаться дальше

@ssssank
Copy link
Contributor

ssssank commented Jun 19, 2023

@devbulygin попробуй задеплоить, если хоть немного работает, мы посмотрим
@Malcom1986 посмотри, пожалуйст :)

# Conflicts:
#	src/main/java/io/hexlet/typoreporter/config/SecurityConfig.java
#	src/main/resources/templates/login.html
@devbulygin
Copy link
Author

@ssssank да, у меня не получается залить на рендер, он просит dockerfile, а тут docker compose, не могу разобраться

@Malcom1986
Copy link
Collaborator

@devbulygin Привет! Этот docker compose тебе не пригодится для деплоя, он нужен чтобы локально запустить базу данных в контейнере. А render предоставляет нам свою базу данных как сервис. Так что да, тут понадобится докерфайл. Тут ты верно начал. Внутри нужно будет скопировать приложение в контейнер, запустить сборку и упаковку. А в качестве команды запуска запустить джар архив с приложением. Глянь статью https://ru.hexlet.io/blog/posts/render-java, она поможет с общей последовательностью действий разобраться. Отличие в том, что тут мы мавен используем, так что команды поменяются. Но принцип тот же останется. Примеры команд упаковки и запуска приложения можно в мейкфайле подсмотреть

@devbulygin
Copy link
Author

devbulygin commented Jun 20, 2023

@Malcom1986
мне нужна помощь,

FROM openjdk:19

COPY / .

RUN ./mvnw -B -ntp -fae clean verify

CMD java -jar -Dspring.profiles.active=dev ./target/typoreporter-*.jar

EXPOSE 5432

@Malcom1986
Copy link
Collaborator

Malcom1986 commented Jun 22, 2023

Привет! Ты верно мыслишь в целом. Но смотри, у нас используется мавен, а значит нужно будет его установить или взять за основу такой образ, где помимо джавы еще и мавен установлен сразу. Можно взять официальный образ мавена:

FROM maven:3.9.2-eclipse-temurin-20

Следущий момент. Скопировать внутрь мы должны файлы проекта. А значит должны копировать не из корня файловой системы, а из текущей директории (там, где у нас докерфайл лежит)

COPY . .

Дальше мы должны собрать проект и упаковать в джар. При этом тесты запускать нам не обязательно

RUN mvn clean package -DskipTests

Теперь у нас есть джарник с приложением. Осталось указать команду, которая выполнитс при старте контейнера. Это команда на запуск нашего приложения. Но только не с дев профилем, а уже с прод. Так как это не локальная разработка. Команду старта можно в procfile подглядеть:

CMD java -Xmx256m -jar target/typoreporter-*.jar --spring.profiles.active=default,prod --server.port=$PORT

Итого получаем такой докерфайл:

FROM maven:3.9.2-eclipse-temurin-20

COPY . .

RUN mvn clean package -DskipTests

CMD java -Xmx256m -jar target/typoreporter-*.jar --spring.profiles.active=default,prod --server.port=$PORT

Осталось только создать в рендере базу данных и установить приложению в рендере переменные окружения (url базы, пользователь, пароль). Тут тебе статья про рендер поможет

@Malcom1986
Copy link
Collaborator

Давай только докерфайл не будем в ПР добавлять, он тебе только для деплоя на рендер нужен. Локально мы без докера пока приложение запускаем (в докере только база)

@devbulygin
Copy link
Author

devbulygin commented Jun 22, 2023

получилось!

http://hexlet-correction-production.up.railway.app

@ssssank
Copy link
Contributor

ssssank commented Jun 26, 2023

Здорово, что получилось! Есть несколько замечаний, которые попались сразу: на странице входа есть возможность войти только через гитхаб, зарегистрироваться/войти через почту нельзя, кнопки потерялись. И ещё сервис падает с 500 ошибкой при попытке создать воркспейс, там кажется что-то с обработкой ошибок

@Malcom1986 посмотри, что скажешь? :)

@Malcom1986
Copy link
Collaborator

@devbulygin Привет! Сборка упала, поправь пожалуйста

@devbulygin
Copy link
Author

@Malcom1986 Поправил!

@ssssank
Copy link
Contributor

ssssank commented Jun 30, 2023

@devbulygin как-будто бы ничего не изменилось? :)

@ssssank
Copy link
Contributor

ssssank commented Jul 21, 2023

К сожалению, экшен так и падает

@ola-9
Copy link
Contributor

ola-9 commented Jul 28, 2023

@devbulygin привет! Скажи, пожалуйста, будет у тебя возможность доделать этот ПР, нужна какая-нибудь помощь?

@dzencot
Copy link
Collaborator

dzencot commented Aug 7, 2023

Давайте гляну на неделе

@fey fey marked this pull request as draft August 11, 2023 17:14
@fey
Copy link
Collaborator

fey commented Aug 11, 2023

Пока перевожу в драфт @Malcom1986 можешь обрисовать небольшой, что нужно сделать дальше?

d1z3d added a commit to d1z3d/hexlet-correction that referenced this pull request Jun 27, 2024
d1z3d added a commit to d1z3d/hexlet-correction that referenced this pull request Jun 28, 2024
d1z3d added a commit to d1z3d/hexlet-correction that referenced this pull request Jun 28, 2024
@d1z3d
Copy link
Contributor

d1z3d commented Jun 28, 2024

@fey, здравствуйте!
Добавил аутентификацию с помощью гитхаба. Можете посмотреть? Есть недочеты пока на беке, поэтому прошу посмотреть как на прототип/черновик )

https://hexlet-correction-ac3h.onrender.com/

d1z3d added a commit to d1z3d/hexlet-correction that referenced this pull request Jul 29, 2024
d1z3d added a commit to d1z3d/hexlet-correction that referenced this pull request Aug 1, 2024
d1z3d added a commit to d1z3d/hexlet-correction that referenced this pull request Aug 6, 2024
d1z3d added a commit to d1z3d/hexlet-correction that referenced this pull request Aug 17, 2024
d1z3d added a commit to d1z3d/hexlet-correction that referenced this pull request Aug 17, 2024
d1z3d added a commit to d1z3d/hexlet-correction that referenced this pull request Aug 26, 2024
d1z3d added a commit to d1z3d/hexlet-correction that referenced this pull request Aug 26, 2024
d1z3d added a commit to d1z3d/hexlet-correction that referenced this pull request Aug 26, 2024
d1z3d added a commit to d1z3d/hexlet-correction that referenced this pull request Aug 26, 2024
d1z3d added a commit to d1z3d/hexlet-correction that referenced this pull request Sep 2, 2024
d1z3d added a commit to d1z3d/hexlet-correction that referenced this pull request Sep 8, 2024
d1z3d added a commit to d1z3d/hexlet-correction that referenced this pull request Oct 12, 2024
d1z3d added a commit to d1z3d/hexlet-correction that referenced this pull request Oct 12, 2024
d1z3d added a commit to d1z3d/hexlet-correction that referenced this pull request Oct 12, 2024
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.

7 participants