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

Do not always include real filename or URL in imagery_used #4530

Closed
poevak opened this issue Nov 14, 2017 · 4 comments
Closed

Do not always include real filename or URL in imagery_used #4530

poevak opened this issue Nov 14, 2017 · 4 comments
Labels
bug A bug - let's fix this!

Comments

@poevak
Copy link

poevak commented Nov 14, 2017

Pull request #4440, initiated by issue #4385, changed so that the real filenames of local GPX files were made public.

I think this is pretty terrible from a privacy point of view. It's not at all evident in the editor that the filename will be made public: the only way to see that it will is to expand all the tags and then hover over the sources field (the field is only wide enough to show the satellite imagery reference).

In essence, the user is tricked into submitting more information than he or she might have intended to. If we accept that, why stop at sneaky submission of just metadata when we could simply upload and make the entire trace public?

As the editor does not allow the user to edit the filename about to be submitted, despite the fact that the user could have set any filename before doing any edits, it is necessary to redo the edits should a sensitive filename have been accidentally set.

I love the simplicity of the iD editor, so adding options or extra editing fields doesn't seem like the best idea to me. Is there really any real value of including local filenames or unrecognized URLs?

@bhousel
Copy link
Member

bhousel commented Nov 15, 2017

I admit that I did not think that anyone would have an issue with the privacy of their gpx filenames. I guess we don't really need to put an actual filename there. Sorry!

@bhousel bhousel added the bug A bug - let's fix this! label Nov 15, 2017
@JamesKingdom
Copy link
Collaborator

Maybe we could include the URL if it is on OpenStreetMap, but just say "Local GPX" if it's uploaded?

@bhousel
Copy link
Member

bhousel commented Nov 15, 2017

I changed the code to generate a string like '.gpx data file' or '.geojson data file' depending on the extension:

screenshot 2017-11-15 11 44 51

@jidanni
Copy link
Contributor

jidanni commented Feb 12, 2018

Allow us to recall that there are four levels of
https://wiki.openstreetmap.org/wiki/Visibility_of_GPS_traces .
So some consideration of at least the people choosing Identifiable should be made...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug A bug - let's fix this!
Projects
None yet
Development

No branches or pull requests

4 participants