Zaczero / osm-revert

⏪ OpenStreetMap Changeset Reverter — CLI + Website
https://revert.monicz.dev
GNU Affero General Public License v3.0
62 stars 6 forks source link

Show information message on users with active blocks, hinting to contact DWG instead #18

Open mmd-osm opened 11 months ago

mmd-osm commented 11 months ago

I've noticed that some of the nodes that have been reverted in https://www.openstreetmap.org/changeset/143128522 seem to be lacking their tags from previous versions.

Example: https://www.openstreetmap.org/node/540424027/history

I would expect a website tag in version 19, which was present in version 13 or 15. Both of which I would consider as a good versions. The full history also shows that all versions have different timestamps, so the infamous Overpass issue with multiple versions at the same time shouldn't apply here.

Zaczero commented 11 months ago

image

https://osm.mapki.com/history/node/540424027

Please take note of the changeset ID that has been reverted (143124536). This revert tool automatically resolves conflicts in tags, relations, etc.. To restore the original set of tags, the changeset IDs 143124619 and 143124645 should also be reverted.

mmd-osm commented 11 months ago

Thank you for your quick and detailed analysis. So when I get this right, the idea is to compensate the changes which have been done between versions 15 and 16 as part of cs 143124536, and apply the result on 18?

It seems people are having a bit of a hard time to figure out, that vandals upload multiple versions of the same objects, and that some kind of transitive closure of these changes would be needed to do a full revert. I'm not sure if this is in scope of Thanos?

Zaczero commented 11 months ago

Yes, this is within the scope of Thanos (full revert + revert to a specific version). In many cases, individuals revert certain changesets without realizing the full extent of the issue. I'm considering adding an informational notice to the page. Do you have any suggestions for what it should say?

mmd-osm commented 11 months ago

DWG folks are probably in a better position to suggest a meaningful text here, since they're most impacted by partial reverts.

However, maybe you could check, if the changeset to be reverted has been created by a user with an active block, and show some hint to align with the DWG first, or even don't let users perform the revert at all. DWG probably has an opinion on this topic. I would again recommend to reach out to them.

In the particular case I have mentioned in this issue, https://www.openstreetmap.org/user_blocks/15267 was created a few minutes after the malicious upload, while the revert attempt happened only two hours later.