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

Support version for /nodes, /ways, /relations API calls #1067

Open
simon04 opened this issue Oct 6, 2015 · 3 comments · May be fixed by #1189
Open

Support version for /nodes, /ways, /relations API calls #1067

simon04 opened this issue Oct 6, 2015 · 3 comments · May be fixed by #1189

Comments

@simon04
Copy link
Member

simon04 commented Oct 6, 2015

When reverting a changeset (e.g., using the JOSM plugin reverter), one needs to obtain a large number of objects in a specific version. At the moment, those are obtained sequentially one by one:

GET https://api.openstreetmap.org/api/0.6/node/618158454/6
GET https://api.openstreetmap.org/api/0.6/node/223240985/18
GET https://api.openstreetmap.org/api/0.6/node/1341712258/2
GET https://api.openstreetmap.org/api/0.6/node/674055923/7
GET https://api.openstreetmap.org/api/0.6/node/31446843/26

To improve performance, it would be great to obtain them in a single call:

GET https://api.openstreetmap.org/api/0.6/nodes?nodes=618158454/6,223240985/18,1341712258/2,674055923/7,31446843/26

Would it be acceptable to enhance node_controller#nodes with this feature? Depending on whether the version is present or not, either a Node or an OldNode would be fetched.

@bhousel
Copy link
Member

bhousel commented Oct 6, 2015

Following.. Would really be useful for iD too.
Though we'd really want this implemented in cgimap.

@iandees
Copy link
Contributor

iandees commented Oct 6, 2015

I might suggest using dots rather than slashes to save in URL length? Slashes would have to be urlencoded to %2F.

@bhousel
Copy link
Member

bhousel commented Oct 6, 2015

I might suggest using dots rather than slashes to save in URL length? Slashes would have to be urlencoded to %2F.

or the letter 'v' would be cool too

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants