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.

Sign up for GitHub

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

Jump to bottom

Correct REST verbs for CRUD #5

Open
ricardobalk opened this issue Sep 30, 2020 · 0 comments
Open

Correct REST verbs for CRUD #5

ricardobalk opened this issue Sep 30, 2020 · 0 comments
Assignees
@ricardobalk
Labels
improvement Something that currently works, but can be done in a better way
Milestone
1.2.0

Comments

@ricardobalk
Copy link
Member

ricardobalk commented Sep 30, 2020 •
edited
Loading

The current version of go-osmand-tracker uses GET instead of POST, so it is not a correct REST-y way to receive location updates, but the OsmAnd uses GET requests for location updates, so that's why GET is used.

go-osmand-tracker should accept both GET (for backward compatibility) and POST (with a JSON body) in the future.

Operation HTTP SQL DDS
Create PUT / POST INSERT write
Read GET SELECT read / take
Update PUT / PATCH UPDATE write
Delete DELETE DELETE dispose
@ricardobalk ricardobalk changed the title HTTP GET Correct REST verbs for CRUD Sep 30, 2020
@ricardobalk ricardobalk added the improvement Something that currently works, but can be done in a better way label Sep 30, 2020
@ricardobalk ricardobalk self-assigned this Sep 30, 2020
@ricardobalk ricardobalk added this to the 1.1.0 milestone Sep 30, 2020
@ricardobalk ricardobalk modified the milestones: 1.1.0, 1.2.0 Oct 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ricardobalk ricardobalk

Labels
improvement Something that currently works, but can be done in a better way
Projects
None yet
Milestone
1.2.0
Development

No branches or pull requests
1 participant
@ricardobalk