keystonejs/keystone-classic

mime type error on file upload with FS adapter (when only specifying path)

Open

#4 017 ouverte le 19 févr. 2017

Voir sur GitHub
 (5 commentaires) (0 réactions) (1 assigné)JavaScript (14 656 stars) (2 288 forks)batch import
bughelp wanted

Description

Steps to reproduce the behavior

  1. have a model with a field of type File and the file system adapter
  2. try to upload a file by calling model._.yourFileField.upload({path: specifiy your path})

Expected behavior

the file appears in the directory specified in the file system adapter

Actual behavior

an error occurs: mime is not a function

it happens because in lib/storage/index.js line 120 the mime type is tried to be evaluated when it wasn't specified in the file parameter, but the call has to be mime.lookup(file.path)

Guide contributeur

Stack technique
javascriptnodejsexpress
Domaine
backendapi
Type d'issue
bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
under 1 hour
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
basic Node.jsKeystoneJS file uploads
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
80
Direction de recherche
The bug is in lib/storage/index.js at line 120 where `mime(file.path)` is used instead of `mime.lookup(file.path)`. The fix is straightforward: replace `mime(file.path)` with `mime.lookup(file.path)`. No linked PRs or recent comments suggest any blockers. The assignee may have a solution, but the issue is open. Ensure the fix aligns with the existing codebase style and testing practices.