Hi guys,
Just found out the hard way that uploaded photos will retain any location (ie. GPS Lat/Lon) data stored in the image’s EXIF (in fact, the whole EXIF is retained, including device, f-stop, white balance, ISO etc data). While I understand there’d be a bit of work involved in stripping any EXIF data out, the removal of at least the location data would be a privacy concern of many users (both current and potential), especially if it’s expected they may be uploading from a mobile device (which they more than likely took said photo on) - while someone might feel comfortable having their home town in their profile, having the GPS coordinates of their house publicly available is likely another story.
That’s a sensible idea, but at the same time I think Discourse shouldn’t be modifying users’ uploaded files unless the user explicitly told it to do so.
This could be an option in the “Add image or file” dialog, however; a “strip private metadata” checkbox, for instance. When images are uploaded via drag&drop and contain problematic EXIF headers, a warning message in the composer should be enough.
I know twitter strips the EXIF headers. And I’m not totally sure how Wordpress does it, but I think if the image is resized the EXIF headers are stripped. I doubt 500px strips any of the headers, but they are more of a photo centric application.
I personally think the private information should be removed but I think we should also look and see what other apps are doing and learn from them.
Would definitely accept a community PR in this area!
Add a checkbox, checked by default, labelled “remove EXIF data”.
You mean you’d put this in as a standard option? I think this is definately plugin territory. Or at least have the ability to turn it off.
Safe by default is what we try to deliver and I think stripping exif data is safer by default. Helps with rotated by metadata images as well, which @zogstrip already completed, right @zogstrip?
Since EXIF data also contains copyright information which can be used to help fight copyright violations, I think it’s a really, really bad idea to strip it - regardless of the fact that Twitter and Pinterest do it.
Removing copyright information might even be illegal in some countries.
(b) Removal or Alteration of Copyright Management Information. — No person shall, without the authority of the copyright owner or the law —
(1) intentionally remove or alter any copyright management information,
I don’t see anything bad about stripping EXIF location, you don’t need to touch copyright
Looks like the action is here?
def self.fix_image_orientation(path)
`convert #{path} -auto-orient #{path}`
end
Maybe someone can extend that into a PR to remove EXIF location information.
Unfortunately, after a quick lookup, Image Magick only support stripping all the EXIF information. We’ll need either a new gem or a new tool to deal with that issue properly.
Personally I think that’s fine. If it’s good enough for Pinterest and Twitter, it’s certainly good enough for us.
No creo que Discourse deba eliminar ningún dato EXIF, al menos no de forma predeterminada, y en cualquier caso, ofrecer la opción de eliminarlo como una opción global o de usuario (¿quizás mediante un complemento?).
Los usuarios que no deseen que se comparta cierto dato EXIF en ninguna red social deben eliminar sus datos EXIF al exportarlos a JPEG.
Como administrador de un foro de fotografía, los datos EXIF de las imágenes son importantes para nuestros usuarios.
Quizás la solución para ti sería enlazar a fotos de Flickr en lugar de subirlas?
Para nada… queremos que las fotos subidas a nuestro sitio respeten el acuerdo de licencia y, además, hemos creado el foro porque no nos gusta Flickr ni Facebook, donde no recibes comentarios, solo ‘me gusta’.
Creamos la comunidad para obtener opiniones sobre nuestras fotos, y las fotos deben permanecer en la comunidad.
Si te preocupa tu privacidad o los datos sensibles en tus fotos, simplemente no los guardes en tu imagen al exportarla.
Interesante. Pensé que Flickr era el lugar en línea más respetuoso para el almacenamiento y la compartición de fotos, nada parecido a Facebook. Pero no he usado ninguno de los dos desde hace un tiempo. Personalmente, guardo mis fotos sin conexión y en un servidor Nextcloud, y solo comparto una foto ocasional en un foro de Discourse.
Parece que lo que necesitas es un complemento que haga opcional la eliminación de los datos EXIF. Podrías solicitarlo en Marketplace.
Lo más respetuoso podría ser 1x, 500 píxeles también, y Flickr es respetuoso también, pero no es gratuito y no podemos depender de lo que Flickr, 500px u otros piensen sobre cuáles son nuestras necesidades.
Estás pasando por alto nuestro punto (quizás no lo he explicado bien, perdón por mi inglés): somos una comunidad de fotógrafos que compartimos nuestras fotos para obtener opiniones de los demás.
No usamos Discourse como un lugar para subir todas o la mayoría de nuestras fotos.
La información EXIF es útil para dar una opinión, a veces.
NO, no necesito un plugin; quien lo necesita es quien está pidiendo cambiar el comportamiento de Discourse.
Discourse actualmente sirve los datos EXIF y no los elimina del archivo.
Tener una opción para seleccionar si deseas eliminarlos globalmente o como preferencia de usuario es una buena solución.
Eso puede implementarse como un plugin, por supuesto.
Podrías ampliarlo con una casilla de verificación sencilla al momento de subir las imágenes; eso sería una característica increíble.
A menudo, tomo fotografías de piezas o artículos en venta con mi teléfono y las subo a nuestro foro, lo que luego revela mi ubicación en el hogar ![]()
Nuestro foro también es muy pesado en imágenes, así que estoy de acuerdo en que mantener los datos EXIF intactos es realmente útil para nosotros.
La mayoría de las veces ![]()
Sí, añadir la posibilidad de eliminar algunos EXIF sería genial.
Pero para ello, es necesario implementar una gestión completa de EXIF en el sistema.
Creo que la mejor opción sería:
Establecer un parámetro global con tres opciones:
- Mantener todos los EXIF (activado por defecto para no romper la funcionalidad de las instalaciones actuales).
- Eliminar los EXIF de las imágenes subidas.
- Permitir que el usuario elija si desea eliminar los EXIF.
Si se selecciona la tercera opción a nivel global, los usuarios deberían tener una preferencia personal para elegir:
- Mantener todos los EXIF (activado por defecto para no romper la funcionalidad de las instalaciones actuales).
- Eliminar los EXIF de las imágenes subidas.
Al subir una imagen, el sistema podría ofrecer la opción de eliminar los datos de la imagen de forma individual (incluso si se ha seleccionado mantenerlos).
Puedes añadir esa opción en el diálogo de subida de imágenes.
Idealmente, si alguien selecciona eliminar datos, debería poder elegir:
- Eliminar solo los datos de GPS y ubicación.
- Mantener solo los datos básicos (modelo de cámara y objetivo, y parámetros de la toma).
- Eliminar todos los datos de la imagen.