Hexlet / hexlet-correction

Typo Reporter
https://fixit.hexlet.io/
GNU Affero General Public License v3.0
41 stars 72 forks source link

[#236] Add allowed URLs and dynamic CORS filter #265

Closed bazilval closed 4 weeks ago

bazilval commented 1 month ago

Demo: hexlet-correction-filtration.onrender.com

Issue: https://github.com/Hexlet/hexlet-correction/issues/236

fey commented 1 month ago

@bazilval URLs можно унести в Settings, тк именно к настройкам это относится

Main URL for reports is

это не совсем корректно. Урл относится к воркспейсу и больше служит опозновательным знаком, для какого сайта мы делаем обработку опечаток. Точнее даже проект. Это может быть просто ссылка на главную страницу, на которой может и не быть самого виджета, а он будет на поддоменах. Поэтому лучше не смешивать Allowed hosts и урл воркспейса. Можно например при создании воркспейса по дефолту добавлять разрешенный урл, который был указан при создании. Тогда нам придется обрабатывать только один список урлов

bazilval commented 1 month ago

@fey Ну в целом сейчас это так и работает. УРЛ, который указывается при создании доски автоматически добавляется в список разрешённых. Но его нельзя удалить оттуда (как дополнительные). И именно поэтому я его вынес отдельно, чтобы не смешивать.

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

fey commented 1 month ago

@bazilval да, зачем нужен волшебный урл? Урл для отправки опечаток может не соотвествовать урлу сайта.

Malcom1986 commented 1 month ago

@bazilval Привет! А тут еще будут какие-то правки или работа закончена?

bazilval commented 1 month ago

@Malcom1986 привет! Да, поправлю по замечаниям @fey

У тебя есть что-то добавить, что поправить?

Malcom1986 commented 1 month ago

Нет пока, я просто не понял, продолжатется ли работа)

bazilval commented 1 month ago

@Malcom1986 @fey поправил Демо там же hexlet-correction-filtration.onrender.com

fey commented 1 month ago

Ждем ревью от Максима :+1:

fey commented 4 weeks ago

Макс в отпуске, как придет, посмотрит. Я пока смержу