Omit href

docs/registry-post-route.md

@@ -44,4 +44,5 @@ curl http://my-components-registry.mydomain.com/
 |components[index].name|`string`|`yes`|Component name|
 |components[index].version|`string`|`no`|Default latest, the component's version|
 |components[index].parameters|`object`|`no`|Component's parameters|
+|omitHref|`boolean`|`no`|Default false, when `true` omits the href value in the response of each component|
 |parameters|`object`|`no`|Global parameters for all components to retrieve. When component has its own parameters, globals will be overwritten|

registry/routes/components.js

@@ -42,13 +42,12 @@ module.exports = function(conf, repository){
     }
 
     async.map(components, function(component, callback){
-      getComponent({
+      getComponent(_.extend(component, {
         conf: res.conf,
         headers: req.headers,
-        name: component.name,
-        parameters: _.extend(_.clone(req.body.parameters) || {}, component.parameters || {}),
-        version: component.version
-      }, function(result){
+        omitHref: !!req.body.omitHref,
+        parameters: _.extend(_.clone(req.body.parameters) || {}, component.parameters || {})
+      }), function(result){
         callback(null, result);
       });
     }, function(err, results){

registry/routes/helpers/get-component.js

@@ -122,14 +122,17 @@ module.exports = function(conf, repository){
             renderMode = isUnrendered ? 'unrendered' : 'rendered';
 
         var response = {
-          href: componentHref,
           type: conf.local ? 'oc-component-local' : 'oc-component',
           version: component.version,
           requestVersion: requestedComponent.version,
           name: requestedComponent.name,
           renderMode: renderMode
         };
 
+        if(!options.omitHref){
+          response.href = componentHref;
+        }
+
         retrievingInfo.extend({
           href: componentHref,
           version: component.version,

test/acceptance/registry.js

@@ -250,7 +250,7 @@ describe('registry', function(){
       });
     });
 
-    describe('when body contains two components', function(){
+    describe('when body contains multiple components', function(){
 
       describe('when Accept header not specified', function(){
 
@@ -281,6 +281,55 @@ describe('registry', function(){
         });
       });
 
+      describe('when omitHref=true', function(){
+        describe('when getting rendered components', function(){
+
+          before(function(done){
+            request({
+              url: 'http://localhost:3030/',
+              method: 'post',
+              json: true,
+              body: {
+                omitHref: true,
+                components: [
+                  {name:'hello-world'},
+                  {name:'no-containers'}
+                ]
+              }
+            }, next(done));
+          });
+
+          it('should respond without href parameter', function(){
+            expect(result[0].response.href).not.to.exist;
+            expect(result[1].response.href).not.to.exist;
+          });
+        });
+
+        describe('when getting unrendered components', function(){
+
+          before(function(done){
+            request({
+              url: 'http://localhost:3030/',
+              method: 'post',
+              headers: {'Accept': 'application/vnd.oc.unrendered+json'},
+              json: true,
+              body: {
+                omitHref: true,
+                components: [
+                  {name:'hello-world'},
+                  {name:'no-containers'}
+                ]
+              }
+            }, next(done));
+          });
+
+          it('should respond without href parameter', function(){
+            expect(result[0].response.href).not.to.exist;
+            expect(result[1].response.href).not.to.exist;
+          });
+        });
+      });
+
       describe('when Accept header set to application/vnd.oc.unrendered+json', function(){
 
         before(function(done){