hexlet-correction icon indicating copy to clipboard operation
hexlet-correction copied to clipboard

[#236] Add allowed URLs and dynamic CORS filter

Open bazilval opened this issue 1 year ago • 3 comments

Demo: hexlet-correction-filtration.onrender.com

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

bazilval avatar May 18 '24 12:05 bazilval

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

Main URL for reports is

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

fey avatar May 20 '24 09:05 fey

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

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

bazilval avatar May 20 '24 10:05 bazilval

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

fey avatar May 20 '24 10:05 fey

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

Malcom1986 avatar May 29 '24 06:05 Malcom1986

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

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

bazilval avatar May 29 '24 16:05 bazilval

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

Malcom1986 avatar May 30 '24 06:05 Malcom1986

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

bazilval avatar May 30 '24 22:05 bazilval

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

fey avatar May 31 '24 11:05 fey

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

fey avatar Jun 10 '24 13:06 fey