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

close#142 Fix cors error #145

Merged
merged 2 commits into from Apr 24, 2023
Merged

close#142 Fix cors error #145

merged 2 commits into from Apr 24, 2023

Conversation

Malcom1986
Copy link
Collaborator

No description provided.

@fey
Copy link
Collaborator

fey commented Apr 24, 2023

@Malcom1986 смотри какая штука. Сделать временно это можно я думаю, но кажется плохой идеей всем позволять слать запросы 🤔
По идее у админа воркспейса в настройках должна быть возможность добавлять адреса, с которых разрешены запросы.

@fey
Copy link
Collaborator

fey commented Apr 24, 2023

Давай временно примем ПР и добавим TODO: #142

@Malcom1986
Copy link
Collaborator Author

По идее ведь для каждого сайта должен быть свой воркспейс. При создании воркспейса мы указываем урл сайта. Наверное только с этого сайта и нужно разрешать запросы, мне кажется каких-то дополнительных адресов не нужно

@fey
Copy link
Collaborator

fey commented Apr 24, 2023

@Malcom1986 ну да, это так и есть.
Но по идее ты можешь использовать в одном воркспейсе два сайта. И в одном пространстве все отрабатывать.
Но кажется это проблема будущих нас. Можно сперва добавлять только 1 ссылку, а потом несколько.
Поставь тудушку и мержи)

@Malcom1986
Copy link
Collaborator Author

@fey Тудушку поставил, а смержить не могу. Прав не хватает. Коль, смержи плиз

@fey
Copy link
Collaborator

fey commented Apr 24, 2023

Права тебе дал. щас мержану.

@fey fey merged commit de08fe5 into Hexlet:main Apr 24, 2023
1 check passed
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.

None yet

2 participants