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

Ban some (copyrighted) custom background #9708

Closed
zbycz opened this issue Jun 22, 2023 · 5 comments
Closed

Ban some (copyrighted) custom background #9708

zbycz opened this issue Jun 22, 2023 · 5 comments
Labels
validation An issue with the validation or Q/A code wontfix-not-iD Issue is with a different project or service

Comments

@zbycz
Copy link
Contributor

zbycz commented Jun 22, 2023

Description

We have this issue in czech community: some begginers are using copyrighted custom background when editing OSM.

In the whole database, there is around 11000 edits so far. Of course members of community are educating those users and reverting the changes, but we would like to ask, if it could be somehow banned in iD itself. It is becoming a little tedious :-)

  • Proposed change: when custom background hostname contains "mapy.cz", warn the user and disable the Save button.

.

Here is our thread in czech: https://openstreetmap.cz/talkcz/c3793 (translator)

Screenshots

No response

@1ec5
Copy link
Collaborator

1ec5 commented Jun 22, 2023

iD has a validation rule for warning about the use of some common copyrighted sources. mapy.cz can be added to this object:

const incompatibleRules = [
{
id: 'amap',
regex: /(^amap$|^amap\.com|autonavi|mapabc|高德)/i
},
{
id: 'baidu',
regex: /(baidu|mapbar|百度)/i
},
{
id: 'google',
regex: /google/i,
exceptRegex: /((books|drive)\.google|google\s?(books|drive|plus))|(esri\/Google_Africa_Buildings)/i
}
];

@1ec5 1ec5 added the validation An issue with the validation or Q/A code label Jun 22, 2023
Jakuje added a commit to Jakuje/iD that referenced this issue Jun 23, 2023
I went through the changesets and most of them use directly `mapserver.mapy.cz` (some older used `mX.mapserver.mapy.cz` so I hope this regex should work and would not cause any false positives.

Fixes openstreetmap#9708
Jakuje added a commit to Jakuje/iD that referenced this issue Jun 23, 2023
I went through the changesets and most of them use directly `mapserver.mapy.cz` (some older used `mX.mapserver.mapy.cz` so I hope this regex should work and would not cause any false positives.

Fixes openstreetmap#9708
@pnorman
Copy link
Contributor

pnorman commented Jun 23, 2023

iD has a validation rule for warning about the use of some common copyrighted sources. mapy.cz can be added to this object:

Aren't these rules about source tags, while this issue is about imagery URLs? Blocked imagery URLs are set by the editing API in the /capabilities response, which are set at https://github.com/openstreetmap/openstreetmap-website/blob/0e00857d51f2890071698b8ac9e699ce444aa3a3/config/settings.yml#L85

@tyrasd tyrasd added the wontfix-not-iD Issue is with a different project or service label Jul 12, 2023
@tyrasd
Copy link
Member

tyrasd commented Jul 12, 2023

@pnorman is right: The proper place to add this restriction is at the imagery policy of the OSM API. Please open a ticket for that on the openstreetmap-website.

@tyrasd tyrasd closed this as not planned Won't fix, can't repro, duplicate, stale Jul 12, 2023
@Jakuje
Copy link

Jakuje commented Jul 12, 2023

Filled openstreetmap/openstreetmap-website#4085 for this.

@tyrasd
Copy link
Member

tyrasd commented Jul 12, 2023

perfect. 👍

Just a small nit-pick:

they do not have a way to fix it on their side

this is not quite right… we could fix it on iD's side, but it just doesn't make sense: it's much better to do this using the existing functionality of the OSM API, as it will then have an effect in all (properly implemented) OSM editors, not just in iD. 😊

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
validation An issue with the validation or Q/A code wontfix-not-iD Issue is with a different project or service
Projects
None yet
Development

No branches or pull requests

5 participants