-
Notifications
You must be signed in to change notification settings - Fork 6
Add support for POST, PUT, and DELETE methods #6
Comments
|
_@alush A bit confused. Is there support now for PUT and POST request. I usually _use bash for my api calls but now need to do a request with javascript. Thanks. |
Hi @ururk and @Elwyn1979. @alulsh is no longer maintaining this repo, and Mapbox plans to archive the repo since it is not in use. Since it is public, however, and has been published as an NPM module, I plan to keep it active outside of Mapbox in my own Github account. I have not found another Node.js SDK for the Jamf API, so I think there is public benefit to keeping this active and maintained. @ururk thanks for the PR, I will take a look and aim to get it merged and deployed to NPM before too long. |
any future updates or maintenance will be done at this library's new home: https://github.com/trevorspecht/jamf-sdk-node |
Currently this wrapper only supports
GET
requests for the JSS REST API. We should add support for other HTTP methods such asPOST
,PUT
, andDELETE
.For example, we can add a
post
function to theJamfApiClient
object prototype and update themethod
for therequest
fromGET
toPOST
.For these HTTP methods in particular we should add solid test fixtures for Nock as well to avoid having to test on a production API.
We should also add code samples and update the README with useful documentation for these other methods.
/cc @mapbox/security @ianshward
The text was updated successfully, but these errors were encountered: