Skip validation if heif - #202
Conversation
|
@werner veo que falló el build, parece ser relevante |
|
@nubis si, estoy en eso, lo que me parece extraño es que no puedo reproducirlo localmente, es como si circleci no me acepta el content_type_mappings del initializer de Paperclip. |
|
Será porque internamente Paperclip usa el comando 'file' del sistema para identificar, o alguna cuestión similar con algún comando local del sistema operativo. |
|
la vuelvo a poner in progress hasta que este pasando en circleci |
|
El tema es el siguiente: El problema es que circleci tiene una versión de Ubuntu muy vieja, si los servidores de compliance tienen al menos 16.04 (LTS), se puede instalar sin problemas, si no, el comando que usa Paperclip para detectar el mime type fallará y esta implementación no servirá. |
|
la verdad que no gusta mucho la idea de instalar cosas extras... no hay manera de decirle a paperclip que si viene ese mimetype, lo deje pasar sin validar nada ? |
|
Bueno, eso no crearía un tema de seguridad? me imagino que esa validación existe para verificar que el mime type que se envía sea el correcto, tal vez al ignorarlo alguien pudiera enviar un troyano en formato heif/heic. |
|
Si, podria ser un tema de seguridad..pero al margen de eso..es posible hacerlo ? |
|
Creo que si, un monkey patch a la clase MediaTypeSpoofDetector cuando viene un archivo heic/heif y listo, el punto sería, ¿es eso realmente lo que queremos? |
|
+1 a tener el spoof detection funcionando, tengamos en cuenta que estos archivos los descargan en mac a veces, y ahí hay un mundo distinto de vulnerabilidades y de mime type detection. Se pueden instalar las librerias del sistema operativo que hagan falta, las agregamos a los scripts de setup de nuevas instancias también. En los ubuntu 18.04 hay que instalar algo? ya está en los repos oficiales en 18.04 la librería HEIF? Ya le pongo un tag al issue para que no se nos pase de instalar esa librería antes de hacer deploy. |
|
En 18.04 funciona perfecto, el problema era que en circle la versión de Ubuntu era la 14.04 y el paquete En 18.04 el paquete está en los repositorios principales. |
|
voy a agregar una issue nueva para actualizar circle ci a ubuntu 18.04. luego retomamos esta issue |
|
Di de alta la issue https://github.com/bitex-la/storyboard/issues/683 para actualizar circle ci |
3 participants
Este es otra idea para evitar problemas con los archivos heif.
La idea básica era ignorar validaciones, sin embargo, Paperclip tiene una opción llamada content_type_mappings, esto permite agregar un content type no incluido en Paperclip para que pueda detectarlo y no arrojar error.
fixes: bitex-la/storyboard#620