feat/authorization - Githubissues

effektivnayarabota1 commented 2 months ago

closes https://github.com/atls/team-management/issues/30 closes https://github.com/atls/team-management/issues/34

effektivnayarabota1 commented 2 months ago

@Nelfimov review request

effektivnayarabota1 commented 2 months ago

допишу получение данных для этой штуки, потом сольем

effektivnayarabota1 commented 2 months ago

добавил ветку. в ней последние изменения

https://github.com/atls/team-management/tree/feat/authorized-profile

Nelfimov commented 2 months ago

Ну и privates - так же такое себе наименование. Ты ведь в нем проверяешь корректность окружения

effektivnayarabota1 commented 2 months ago

После регистрации

я убрал throw Error на входе, обернул в Condition элементы, от которых завясят эти пропсы код обновил

покажи пожалуйста какие свойства до тебя не доходят? у меня все отображается корректно:

effektivnayarabota1 commented 2 months ago

Ну и с версткой беда:

@ssagg поправишь верстку?

ssagg commented 2 months ago

Ну и с версткой беда:

@ssagg поправишь верстку?

@effektivnayarabota1 принял. у @Nelfimov safari и скролл там совсем дико выглядит. Надо в других браузерах чекнуть.

effektivnayarabota1 commented 2 months ago

Прошло, похоже почту не тянет.

сейчас удалил из установленных GH app и тоже почту перестало подгружать

как мы будем пользоваться приложением? настраивать так, чтобы без установки все подкачивало? под установкой имею ввиду установку в профиль github

странно, что при авторизации приложение запрашивает доступ к этим данным, но при запросе не дает.

на начальном этапе долго разбирался с этим моментом и пробовал разные типы авторизации. в доках octokit много вариантов, самый простой oAuth app - не работал, взял самый жирный. и то, он заработал только после установки, сейчас вспомнил об этомю

не работал именно момент с почтой

effektivnayarabota1 commented 2 months ago

Ну и с версткой беда:

@ssagg поправишь верстку?

@effektivnayarabota1 принял. у @Nelfimov safari и скролл там совсем дико выглядит. Надо в других браузерах чекнуть.

скролл реально мясной

effektivnayarabota1 commented 2 months ago

весь день разбирался с email в итоге получил его через косоль, код ниже

curl --request GET \
--url "https://api.github.com/user" \
--header "Accept: application/vnd.github+json" \
--header "Authorization: Bearer USER_ACCESS_TOKEN" \
--header "X-GitHub-Api-Version: 2022-11-28"

запрос email делаю по тому же ключу, что с сервера
на сервер 'email' приходит в виде пустой строки
пробовал запрашивать через Rest Api
пробовал запрашивать юсера по логину (сейчас запрос через viewer(авторизированый user))
не приходит именно email

проблема не в ключе и приложении, проблема в запросе

effektivnayarabota1 commented 2 months ago

[x] проверить endpoint через curl

curl -H "Authorization: bearer TOKEN" -X POST -d " \
{ \
\"query\": \"query { viewer { login }}\" \
} \
" https://api.github.com/graphql

если email приходит, то ошбка в интерфейсе octokit, используй другой
если не приходит, то ошибка с endpoint, либо неправильно сформирован запрос. перепроверь ключ на rest (curl в предыдущем сообщении)

ключ корректный, т.к. Rest Api возвращает email
curl на qraphql, email не возвращает, при испльзовании сгенерированного ключа
curl на qtaphql возаращает email, при использовании personal access tokken
запрос по ключу без разрешений не дает ошибку, возвращает пустую строку

effektivnayarabota1 commented 2 months ago

поменял GhApp приложение на oAuth
graphql запрос через curl работает - email получаем
octokit интерфейс авторизации поменял на fetch
запрос с сервера не тестировал, но должно работать
в документации указанно, что можно GitHub app использовать для авторизации аналогично oAuth, по факту у меня не получается им так пользоваться
если я сейчас сделаю авторизацию через oAuth, хватит ли нам функционала oAuth app для дальнейшей работы?
в документации указанно, что разница oAuth и GhApp лишь в настройке разрешений, и GhApp можно использовать, помимо "от лица пользователя" еще и "от лица приложения", поэтому рассчитываю, что хватит.
сейчас протестирую graphql запрос с сервера - если все ок, причешу код, запушу

curl -H "Authorization: bearer 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" -X POST -d " \
 { \                                                                  
   \"query\": \"query { viewer { email }}\" \
 } \
" https://api.github.com/graphql
{"data":{"viewer":{"email":"effectivnayarabota1@gmail.com"}}}

effektivnayarabota1 commented 2 months ago

@Nelfimov верской занимается Александр, проверяем авторизацию, у нас не подгружался email

effektivnayarabota1 commented 2 months ago

вот этот хук выполняется в клиентском компоненте ('use client'), но ошибка падает в серверную консоль. не понимаю почему обертку сделал, все работает. посто не могу понять.

app-1   |  ⨯ ReferenceError: window is not defined
app-1   |     at checkErrorHook (../../../../.yarn/__virtual__/@app-root-layout-virtual-ca4000c068/1/app/fragments/app-root-layout/src/check-error.hook.ts:6:33)
app-1   |     at RootLayoutClient (../../../../.yarn/__virtual__/@app-root-layout-virtual-ca4000c068/1/app/fragments/app-root-layout/src/root-layout.client.tsx:20:109)
app-1   |   1 | export const checkErrorHook = () => {
app-1   | > 2 |   const CURRENT_URL = new URL(window.location.href)
app-1   |     |                              ^
app-1   |   3 |   const { searchParams } = CURRENT_URL
app-1   |   4 |   const errorMessage = searchParams.get('errorMessage')
app-1   |   5 |   const errorCode = searchParams.get('errorCode')

Nelfimov commented 2 months ago

вот этот хук выполняется в клиентском компоненте ('use client'), но ошибка падает в серверную консоль. не понимаю почему обертку сделал, все работает. посто не могу понять.

app-1   |  ⨯ ReferenceError: window is not defined
app-1   |     at checkErrorHook (../../../../.yarn/__virtual__/@app-root-layout-virtual-ca4000c068/1/app/fragments/app-root-layout/src/check-error.hook.ts:6:33)
app-1   |     at RootLayoutClient (../../../../.yarn/__virtual__/@app-root-layout-virtual-ca4000c068/1/app/fragments/app-root-layout/src/root-layout.client.tsx:20:109)
app-1   |   1 | export const checkErrorHook = () => {
app-1   | > 2 |   const CURRENT_URL = new URL(window.location.href)
app-1   |     |                              ^
app-1   |   3 |   const { searchParams } = CURRENT_URL
app-1   |   4 |   const errorMessage = searchParams.get('errorMessage')
app-1   |   5 |   const errorCode = searchParams.get('errorCode')

Сделай проверку на наличие window.

Скорее всего у тебя выполняется хук до момента когда он "прогрузился" в браузере. Тебе бы туда useLayoutEffect

effektivnayarabota1 commented 2 months ago

вот этот хук выполняется в клиентском компоненте ('use client'), но ошибка падает в серверную консоль. не понимаю почему обертку сделал, все работает. посто не могу понять.
app-1   |  ⨯ ReferenceError: window is not defined
app-1   |     at checkErrorHook (../../../../.yarn/__virtual__/@app-root-layout-virtual-ca4000c068/1/app/fragments/app-root-layout/src/check-error.hook.ts:6:33)
app-1   |     at RootLayoutClient (../../../../.yarn/__virtual__/@app-root-layout-virtual-ca4000c068/1/app/fragments/app-root-layout/src/root-layout.client.tsx:20:109)
app-1   |   1 | export const checkErrorHook = () => {
app-1   | > 2 |   const CURRENT_URL = new URL(window.location.href)
app-1   |     |                              ^
app-1   |   3 |   const { searchParams } = CURRENT_URL
app-1   |   4 |   const errorMessage = searchParams.get('errorMessage')
app-1   |   5 |   const errorCode = searchParams.get('errorCode')
Сделай проверку на наличие window.

Скорее всего у тебя выполняется хук до момента когда он "прогрузился" в браузере. Тебе бы туда useLayoutEffect

все добавил

atls / team-management

feat/authorization #31

проблема не в ключе и приложении, проблема в запросе

curl на qtaphql возаращает email, при использовании personal access tokken

запрос с сервера не тестировал, но должно работать

сейчас протестирую `graphql` запрос с сервера - если все ок, причешу код, запушу

atls / team-management

feat/authorization #31

проблема не в ключе и приложении, проблема в запросе

curl на qtaphql возаращает email, при использовании personal access tokken

запрос с сервера не тестировал, но должно работать

сейчас протестирую graphql запрос с сервера - если все ок, причешу код, запушу

сейчас протестирую `graphql` запрос с сервера - если все ок, причешу код, запушу