Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add privacy setting to strip EXIF image data on import #2015

Closed
ProbablePrime opened this issue May 9, 2024 · 7 comments
Closed

Add privacy setting to strip EXIF image data on import #2015

ProbablePrime opened this issue May 9, 2024 · 7 comments
Assignees
Labels
New Feature A new addition, whose complexity hasn't been evaluated yet

Comments

@ProbablePrime
Copy link
Member

ProbablePrime commented May 9, 2024

Is your feature request related to a problem? Please describe.

  1. Most cameras these days are smart devices, that record information such as the GPS coordinates of a photo. These are saved to an area of the photo file called "EXIF" data(https://en.wikipedia.org/wiki/Exif).
    image
  2. Privacy minded users would like this information not to be included when an image is imported in Resonite.]

Describe the solution you'd like

  1. A setting that when enabled, will automatically strip any EXIF information from an image on import.

Describe alternatives you've considered

Requiring users to strip EXIF before they import.

Additional Context

  1. I took a look and FreeImage actually... mostly supported this. (see: feat(exif): Enable EXIF Writing using FIMD_EXIF_RAW FreeImage#14)
  2. Therefore its easy to flow into our existing image importing/exporting flows.

This report came via the ticket desk as a security issue, but I see it as more of a privacy feature, additionally as the FreeImage changes need to be public due to the nature of FreeImage, it makes sense to open a public issue.

Requesters

No response

@EmergencyTemporalShift
Copy link

Glad it got implemented, thanks for the work.

@ProbablePrime
Copy link
Member Author

Not yet, we're on the way to though!

@ProbablePrime
Copy link
Member Author

@TisFoolish
Copy link

Can you please include info in the description that the data can include location? I know that the description mentions GEOTIFF metadata, I just fear that is far too technical for most users and won't understand what that means.

@ProbablePrime
Copy link
Member Author

Sure!

@ProbablePrime
Copy link
Member Author

Updated: Yellow-Dog-Man/Locale@3e24f71

@TisFoolish
Copy link

I think that works!

@ProbablePrime ProbablePrime moved this from In progress to In review in ProbablePrime's Task Organization May 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
New Feature A new addition, whose complexity hasn't been evaluated yet
Projects
Development

No branches or pull requests

4 participants