-
Notifications
You must be signed in to change notification settings - Fork 3
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
Update PUT verb for services #70
Comments
Are there any plans to support the HTTP PATCH verb, which could incrementally (delta) update an existing namespace and/or adapter metadata? (http://restful-api-design.readthedocs.org/en/latest/methods.html#patch-vs-put) |
It's a possibility, indeed. However, I'm experimenting with a restructure of the registration procedure (and other things) in the branch It could still make sense, as a way to update fields on the metadata without rebuilding the services, but it also means that the file I'm open to discussion, anyway. |
Is it plausible to simply allow re-POST without going through the delete cycle? This is the access pattern Agave services uses for apps and systems. The real pain point is having to delete first before updating. |
Yes, we could support this, i.e. issuing an implicit DELETE when POSTing to an existing service. I'll take a stab at this and push to the dev instance. |
Code for handling PUT has gone out-of-date (b/c of duplication of code). Follow DRY and use the same code to specify the fields for creation and updating.
The text was updated successfully, but these errors were encountered: