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

Sprint8 #3

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

Sprint8 #3

wants to merge 7 commits into from

Conversation

tomintaiga
Copy link

Выполнил все задания спринта

@kostya05983
Copy link

kostya05983 commented Nov 20, 2024

Привет, не грузится страница при заходе на localhost:3000. Скорее всего из-за хардкода айпишников.
image

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

frontend/.env Outdated
@@ -1,4 +1,4 @@
REACT_APP_API_URL=http://localhost:8000
REACT_APP_KEYCLOAK_URL=http://localhost:8080
REACT_APP_API_URL=http://192.168.68.104:8090

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

тут хардкод айпишников, так лучше не делать

@@ -114,8 +114,8 @@
"clientId": "reports-frontend",
"enabled": true,
"publicClient": true,
"redirectUris": ["http://localhost:3000/*"],
"webOrigins": ["http://localhost:3000"],
"redirectUris": ["http://192.168.68.104:3000/*"],

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

тут тоже

Remove real IP from it
@tomintaiga
Copy link
Author

Убрал IP адреса.

Мой смуззи ноут не тянет docker, пришлось поднимать еще машины =)

API/main.go Outdated
router := gin.Default()
router.Use(ginkeycloak.RequestLogger([]string{"uid"}, "data"))
router.Use(gin.Recovery())
router.Use(cors.Default())

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

тут используется cors.Default().
В Default Authorization header запрещён https://github.com/gin-contrib/cors/blob/master/cors.go#L179

В итоге при авторизации через protheric1 и скачивании отчёта, ловлю следующую ошибку:
"Access to fetch at 'http://localhost:8090/reports/' from origin 'http://localhost:3000' has been blocked by CORS policy: Request header field authorization is not allowed by Access-Control-Allow-Headers in preflight response."
image

Нужно поправить cors

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.

2 participants