search

U-Me-Chan / pissychan-front

SSR Frontend for Pissykaka imageboard engine (https://github.com/U-Me-Chan/pissykaka)
Do What The F*ck You Want To Public License
1 stars 2 forks source link

added filestore integration #15

Closed ridouchire closed 2 years ago

ridouchire commented 2 years ago

Забыл добавить поле выбора файла в форму создания треда.

ridouchire commented 2 years ago

Забыл добавить поле выбора файла в форму создания треда.

Теперь добавил.

Также постинг-мод отобразил и в форме создания треда.

Oxore commented 2 years ago

Привет, здарова! А почему, когда я пощу текст с картинкой, постится только текст, а картинка не постится? Это нормально?

Собственно я не прошу это фиксить. Для начала я хочу разобраться задумывалось ли это изначально? Если нет, то как задумывалось? Если задумывалось, то думал ли ты, что это может быть неинтуитивно? Мне кажется это неинтуитивно и, соответственно, неожиданно. Может предупреждение какое-нибудь написать, мол, "картинка не запостится, если в посте будет текст"?

Oxore commented 2 years ago

Неясным остаётся то, что происходит с файлами, которые промежуточно сохраняются в директорию /tmp. Они удаляются когда-нибудь? Предполагается ли, что эти файлы нужно удалять самостоятельно?

Мне кажется, что данные загружаемого изображения в идеале даже в виде файлов на диске появляться не должны - всё должно происходить в памяти ОЗУ. А если без файла никак, то файлы должны удаляться сразу после успешного постинга.

Oxore commented 2 years ago

Ооо, нормас прям линтинг выстрелил.

ridouchire commented 2 years ago

@Oxore, я починил.

ridouchire commented 2 years ago

Неясным остаётся то, что происходит с файлами, которые промежуточно сохраняются в директорию /tmp. Они удаляются когда-нибудь? Предполагается ли, что эти файлы нужно удалять самостоятельно?

Мне кажется, что данные загружаемого изображения в идеале даже в виде файлов на диске появляться не должны - всё должно происходить в памяти ОЗУ. А если без файла никак, то файлы должны удаляться сразу после успешного постинга.

Не хочет оно работать в памяти, потому что хуй знает почему. Так что реализовал удаление временного фаела после загрузки.

Oxore commented 2 years ago

Не хочет оно работать в памяти, потому что хуй знает почему.

Надо мне заняться и посмотреть почему так. Я очень не хочу дрочить ФС по такому случаю, потому что это потенциальная дыра в безопасности.

kugimiya commented 2 years ago

Не хочет оно работать в памяти, потому что хуй знает почему.

Надо мне заняться и посмотреть почему так. Я очень не хочу дрочить ФС по такому случаю, потому что это потенциальная дыра в безопасности.

Ты можешь ручками реализовать. Форма с файликами у тебя отправляется как multipart/form-data, по сути файл уже в body POST-запроса. Тебе его вырезать из исходного body, сделать из этой йоба-строки Stream/Blob (скорее всего Blob), append'ить в новую FormData, отправить в filestorage;

Oxore commented 2 years ago

"Ну чё ранарод погнали нахуй! Ебаны рот!"