Bug fixes vultr#434

Author: devhun

CHANGELOG.md

@@ -1,5 +1,9 @@
 # Change Log
 
+## [v2.0.1](https://github.com/vultr/vultr-node/compare/v1.0.4..v2.0.0) (2020-02-25)
+### Bug Fixes
+* Fixes issue create a dns record parameter missing [#434](https://github.com/vultr/vultr-node/issues/434)
+
 ## [v2.0.0](https://github.com/vultr/vultr-node/compare/v1.0.4..v2.0.0) (2020-01-11)
 * Upgrade library to use Vultr API V2 syntax and endpoints. [#388](https://github.com/vultr/vultr-node/issues/388)
 

package-lock.json

@@ -1,7 +1,7 @@
 {
   "name": "@vultr/vultr-node",
   "private": false,
-  "version": "2.0.0",
+  "version": "2.0.1",
   "description": "Node module to communicate with the Vultr API",
   "main": "src/index.js",
   "scripts": {

package.json

@@ -179,6 +179,11 @@ exports.createRecord = {
   requestType: 'POST',
   apiKeyRequired: true,
   parameters: {
+    'dns-domain': {
+      type: 'string',
+      path: true,
+      required: true
+    },
     name: {
       type: 'string',
       required: true

src/api/dns.js

@@ -12,23 +12,23 @@ exports.makeApiRequest = (config, endpoint, userParameters) => {
   if (userParameters !== undefined) {
     const { requestType, parameters } = endpoint
 
+    // All methods may have path parameters
+    const pathParams = Object.keys(userParameters).filter(
+      (key) => parameters[key].path
+    )
+
+    if (pathParams.length) {
+      pathParams.forEach((param) => {
+        // Ex. '/bare-metals/{baremetal-id}/ipv4' becomes '/bare-metals/123456/ipv4'
+        fetchUrl = fetchUrl.replace(`{${param}}`, userParameters[param])
+      })
+    }
+
     if (requestType === 'POST') {
       // POST requests will just send all data as JSON to the endpoint
       options.body = JSON.stringify(userParameters)
       options.headers['Content-Type'] = 'application/json'
     } else {
-      // GET, DELETE, PATCH, and PUT may have path parameters
-      const pathParams = Object.keys(userParameters).filter(
-        (key) => parameters[key].path
-      )
-
-      if (pathParams.length) {
-        pathParams.forEach((param) => {
-          // Ex. '/bare-metals/{baremetal-id}/ipv4' becomes '/bare-metals/123456/ipv4'
-          fetchUrl = fetchUrl.replace(`{${param}}`, userParameters[param])
-        })
-      }
-
       if (requestType === 'GET' || requestType === 'DELETE') {
         // GET and DELETE requests may have path parameters as well as query parameters
         const queryParams = Object.keys(userParameters)