Remove location EXIF data from uploaded photos

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.

5 „Gefällt mir“

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.

2 „Gefällt mir“

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”.

2 „Gefällt mir“

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?

9 „Gefällt mir“

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

8 „Gefällt mir“

Yes I did.

https://github.com/discourse/discourse/commit/a52c80e2a8130b770a57c11b76dbdcd21f51ae28

1 „Gefällt mir“

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.

5 „Gefällt mir“

Ich denke nicht, dass Discourse EXIF-Daten entfernen sollte, zumindest nicht standardmäßig. Auf jeden Fall sollte es die Option geben, diese global oder pro Benutzer zu entfernen (vielleicht als Plugin?).

Benutzer, die nicht möchten, dass EXIF-Daten in soziale Netzwerke gelangen, sollten ihre EXIF-Daten entfernen, bevor sie sie als JPEG exportieren.

Als Administrator eines Fotografieforums sind EXIF-Daten von Bildern für unsere Nutzer wichtig.

3 „Gefällt mir“

Vielleicht wäre die Lösung für dich, stattdessen auf Flickr-Fotos zu verlinken, anstatt sie hochzuladen?

Überhaupt nicht… wir möchten, dass die auf unsere Website hochgeladenen Fotos die Lizenzvereinbarung einhalten. Wir haben das Forum gegründet, weil wir Flickr oder Facebook nicht mögen, wo man keine Kommentare, sondern nur Likes erhält.

Wir haben die Community gegründet, um Meinungen zu unseren Fotos zu erhalten, und die Fotos sollten in der Community verbleiben.

Wenn dir deine Privatsphäre oder sensible Daten in deinen Fotos wichtig sind, speichere sie einfach nicht in deinem Bild, wenn du es exportierst.

2 „Gefällt mir“

Interessant. Ich dachte, Flickr sei der respektvollste Ort im Internet für die Speicherung und den Austausch von Fotos, überhaupt nicht mit Facebook vergleichbar. Aber ich habe beide schon eine Weile nicht mehr genutzt. Persönlich speichere ich meine Fotos offline und auf einem Nextcloud-Server und teile nur gelegentlich ein Foto in einem Discourse-Forum.

Klingt so, als bräuchtest du ein Plugin, das die optionale Entfernung von EXIF-Daten ermöglicht. Du könntest dies im Marketplace anfragen.

Am respektvollsten ist vielleicht 1x, 500px auch, und Flickr ist ebenfalls respektvoll, aber es ist nicht kostenlos, und wir können uns nicht darauf verlassen, was Flickr, 500px oder andere darüber denken, was unsere Bedürfnisse sind.

Sie haben unseren Punkt nicht verstanden (vielleicht habe ich mich nicht gut erklärt, sorry für mein Englisch): Wir sind eine Gemeinschaft von Fotografen, die ihre Fotos teilen, um Meinungen von anderen zu erhalten.
Wir nutzen Discourse nicht als Ort, um alle oder die meisten unserer Fotos abzulegen.

EXIF-Informationen sind manchmal nützlich, um eine Meinung abzugeben.

NEIN, ich brauche kein Plugin. Wer es braucht, ist derjenige, der eine Änderung des Verhaltens von Discourse fordert.

Discourse liefert derzeit EXIF-Daten aus und entfernt sie nicht aus der Datei.

Eine Option zu haben, um global oder als Benutzereinstellung auszuwählen, ob man sie entfernen möchte, wäre eine gute Lösung.

Das kann natürlich als Plugin implementiert werden.

2 „Gefällt mir“

Man könnte das noch mit einem einfachen Kontrollkästchen beim Hochladen erweitern – das wäre eine fantastische Funktion.

Oft mache ich mit meinem Handy Fotos von Teilen oder zum Verkauf stehenden Gegenständen und lade sie in unser Forum hoch, wodurch dann mein Standort zu Hause enthüllt wird :man_facepalming:

Unser Forum ist auch sehr bildlastig, daher stimme ich zu, dass es für uns wirklich nützlich ist, die EXIF-Daten intakt zu lassen.

Meistens :blush:

1 „Gefällt mir“

Ja, die Möglichkeit, bestimmte EXIF-Daten zu löschen, wäre großartig.

Dafür muss jedoch ein vollständiges EXIF-Management in das System integriert werden.

Ich denke, die beste Lösung wäre:

Ein globales Parameter mit drei Optionen festlegen:

  • Alle EXIF-Daten behalten (standardmäßig aktiviert, um die Funktionalität bestehender Installationen nicht zu beeinträchtigen).
  • EXIF-Daten von hochgeladenen Bildern entfernen.
  • Dem Benutzer die Auswahl lassen, ob EXIF-Daten entfernt werden sollen.

Wird die dritte Option auf globaler Ebene gewählt, sollten Benutzer eine eigene Einstellung haben, um zu wählen:

  • Alle EXIF-Daten behalten (standardmäßig aktiviert, um die Funktionalität bestehender Installationen nicht zu beeinträchtigen).
  • EXIF-Daten von hochgeladenen Bildern entfernen.

Beim Hochladen eines Bildes sollte das System zudem die Möglichkeit bieten, Bilddaten bildweise zu entfernen (selbst wenn Sie die Option zum Beibehalten gewählt haben).
Diese Option können Sie im Dialogfenster zum Hochladen von Bildern hinzufügen.

Idealerweise sollte ein Benutzer, der die Datenentfernung wählt, folgende Optionen haben:

  • Nur GPS- und Standortdaten entfernen.
  • Nur grundlegende Daten behalten (Kamera- und Objektivmodell sowie Aufnahmeparameter).
  • Alle Bilddaten entfernen.
6 „Gefällt mir“