Merge pull request #68 from osher/patch-2

private attrs should be stripped from params too

Author: theganyo

fittings/swagger_raw.js

@@ -60,10 +60,18 @@ function filterKeysRecursive(object, dropTagRegex, privateTags) {
             debug('dropping object at %s', key);
             delete(result[key]);
           }
+        } else {
+            debug("dropping value at %s", key)
         }
       });
       return result;
     }
+  } else if (Array.isArray(object) ) {
+     object = object.reduce(function(reduced, value) {
+        var v = filterKeysRecursive(value, dropTagRegex, privateTags);
+        if (v !== undefined) reduced.push(v);
+        return reduced
+     }, [])
   }
   return object;
 }

test/assets/project/api/swagger/swagger.yaml

@@ -22,6 +22,7 @@ paths:
           description: The name of the person to whom to say hello
           required: false
           type: string
+          x-remove-me: lol
       responses:
         200:
           description: Success

test/fittings/swagger_raw.js

@@ -18,6 +18,7 @@ describe('swagger_raw', function() {
 
     var filteredSwagger = _.cloneDeep(swagger);
     delete(filteredSwagger.paths['/invalid_header']);
+    delete(filteredSwagger.paths['/hello'].get.parameters[0]['x-remove-me'])
 
     // hokey algorithm, but at least it's different than the one it's testing
     var OMIT = ['x-swagger-router-controller', 'x-swagger-pipe', 'x-hidden', 'x-private', 'x-controller-interface'];