Как правильно отдать файл на загрузку?

Question

[Станислав]

Не пойму в чем проблема при отдаче файла, в частности проблема с мобильными устройствами.
На сайте есть форма, с помощью js отправляю id документа для поиска файла и последующих действий с ним.
В обработчике уменьшаю и обрезаю картинку

function resize(s, w) {
    gm(
        path.image // путь до изображения
    )
        .resize(s.w, s.h, '^')
        .gravity('Center')
        .crop(s.w, s.h)
        .quality(90)
        .noProfile()
        .toBuffer(function (e, buffer) {
            return c(e, buffer);
        });
}

После обработки отдаю следующим образом

res.statusCode = 200;
res.setHeader('content-type', 'image/jpeg');
res.setHeader('Content-Disposition', 'attachment; filename=' + data.get.image); // название картинки вместе с расширением
res.end(buffer);

С ПК идет загрузка файла и все хорошо, но с мобильника "Файл без названия. Не удалось загрузить файл."
Я так понимаю проблема с заголовками? Что с ними не так?

Answer 1

[dummyman]

Вы не указали в заголовках ответа размер буфера
res.setHeader("Content-Length", buffer.length);

и еще content-type я бы писал с больших буков. Хотя express возможно исправит и сама

Comments

[Станислав]

Уже лучше
С указанием размера теперь качается файл с названием (image_name.jpg), но все равно файл при открытии фотки пишет "Фото не найдено". В чем еще дело может быть? На ПК все ок, через мобильник печалька.
В консольку кстати пишет на ПК: Resource interpreted as Document but transferred with MIME type image/jpeg:..

[dummyman]

Станислав:
я же уже написал "content-type" пишется "Content-Type"

[dummyman]

Станислав: то есть "content-type" просто игнорируется как кастомный заголовок для нестандартных нужд. при этом "Content-Type" не передан, потому интерпретирует как "Accept" из заголовков запроса. Если в accept стоит */*, то автоматически браузер интерпретирует как хочет.

[Станислав]

dummyman: да, он у меня с заглавных букв, я его сразу поправил как написали, но не помогло, пришлось на статику редиректить, а хотелось бы конечно без сохранения статики сразу же отдавать файл. В общем пока не решил почему так происходит.
В логах через мобильник происходит следующее:
Отправляет POST запрос на страницу /ajax/download/ где и создается картинка, затем при отдаче буфера почему то мобильный браузер делает GET запрос на этот же URL, соответственно вылетает 404, и файла нет, кроме его названия. Проблема только если с телефона пытаюсь получить файл.

[dummyman]

Станислав: Ааа, дык там не браузер скачивает. Браузер не умеет качать файлы. Ну по крайней мере в андройде. Качает приложение Диспетчер загрузок. Видать, браузер делает запрос, а потом пытается открыть ссылку этим диспетчером.

[Станислав]

dummyman: как быть тогда? Только статикой отдавать как сейчас? Или есть альтернативы? Может GET запрос отправлять на сервер вместо POST?

[dummyman]

Станислав:

Может GET запрос отправлять на сервер вместо POST?

Да, это однозначно решит проблему

[dummyman]

Станислав: Только еще надо быть готовым, что туда будет два запроса приходить вместо одного

[Станислав]

dummyman: фигово конечно. придется 2 раза обрабатывать изображение, тогда наверное оставлю все как есть, буду статику отдавать, а через сутки чекать файлы и удалять старые. По моему так проще будет)
В общем спасибо за помощь)

[dummyman]

Станислав: Зачем два раза обрабатывать? Зачем на сутки? Я бы в сессию прописывал (хотя и не факт что она передается, но попробовать стоит) в переменную что по такой-то ссылке такой то буффер, и отдавал, там должно то быть задержки где-то максимум 5 секунд.
Отчищать есть несколько вариантов:

1. по таймеру (интервалу) запускать garbage collector
   
2. реализовать доступ через LRUCache (получится самоотчищаемое хранилище без каких-либо таймеров)
   
3. можно и статикой, но тогда хотябы примонтируйте для этого папку с файловой системой tmpfs - это хотябы не будет упарывать ssd, ну и плюсом высокая скорость отклика (все данные уже в оперативке). Я вообще так почти везде любую статику так оформляю, монтирую при запуске сервера tmpfs папку, копирую всю статику в эту папку, и сервер работает со статикой из временной папки. - Скорость отдачи растет приблизительно в 20 раз. Время отдачи небольшого файла (до 1Mb) по запросу в 99% не превышает 1ms
   

[Станислав]

есть над чем подумать)