freefeed-server icon indicating copy to clipboard operation
freefeed-server copied to clipboard

Restrict the allowed attachment types to explicit list

Open davidmz opened this issue 4 years ago • 11 comments

davidmz avatar Aug 17 '20 12:08 davidmz

Нам стоит вернуться к этой задаче. @indeyets @davidmz ?

n1313 avatar Feb 19 '21 06:02 n1313

@n1313 у нас нет до конца сформулированного ответа на вопрос "зачем?". Вопросы безопасности кажется уже закрыты.

Было мнение о том, что не хочется превращать фидик в файловый хостинг, но whitelist в этом и не поможет

indeyets avatar Feb 19 '21 06:02 indeyets

А в чем у нас сейчас состоит защита от заливки html-файлов с активным содержимым?

n1313 avatar Feb 19 '21 06:02 n1313

Защита не от заливки, а от исполнения. Они отдаются с заголовком "attachment" и скачиваются вместо отображения

indeyets avatar Feb 19 '21 06:02 indeyets

Это касается только свежезалитых файлов? Моя старая хтмлька с алертом внутри все еще работает, но если ее заново залить, то да, отдается аттачем

n1313 avatar Feb 19 '21 06:02 n1313

Это касается только свежезалитых файлов? Моя старая хтмлька с алертом внутри все еще работает, но если ее заново залить, то да, отдается аттачем

Да. По старым не проходились

indeyets avatar Feb 19 '21 06:02 indeyets

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

n1313 avatar Feb 19 '21 06:02 n1313

Я голосую за whitelist.

dsumin avatar Feb 19 '21 06:02 dsumin

Я против голосования :) Вопрос во многом философский и я за медленную дискуссию без дедлайна. Нужен спокойный консенсус

indeyets avatar Feb 19 '21 06:02 indeyets

@kukutz @indeyets @dsumin прошел год с первого коммита и полгода с последнего комментария, достаточно ли медленная дискуссия без дедлайна получается? :)

n1313 avatar Aug 06 '21 03:08 n1313

да давайте уже заберем в код :)

dsumin avatar Aug 06 '21 05:08 dsumin