[brightbox] Added check for identifiers being passed to requests to prevent requests to resource root

lib/fog/brightbox/requests/compute/destroy_api_client.rb

@@ -4,6 +4,7 @@ module Fog
       class Real
 
         def destroy_api_client(identifier, options = {})
+          return nil if identifier.nil? || identifier == ""
           request(
             :expects  => [200],
             :method   => 'DELETE',

lib/fog/brightbox/requests/compute/destroy_cloud_ip.rb

@@ -4,6 +4,7 @@ module Fog
       class Real
 
         def destroy_cloud_ip(identifier, options = {})
+          return nil if identifier.nil? || identifier == ""
           request(
             :expects  => [200],
             :method   => 'DELETE',

lib/fog/brightbox/requests/compute/destroy_image.rb

@@ -4,6 +4,7 @@ module Fog
       class Real
 
         def destroy_image(identifier, options = {})
+          return nil if identifier.nil? || identifier == ""
           request(
             :expects  => [202],
             :method   => 'DELETE',

lib/fog/brightbox/requests/compute/destroy_server.rb

@@ -4,6 +4,7 @@ module Fog
       class Real
 
         def destroy_server(identifier, options = {})
+          return nil if identifier.nil? || identifier == ""
           request(
             :expects  => [202],
             :method   => 'DELETE',

lib/fog/brightbox/requests/compute/get_api_client.rb

@@ -4,6 +4,7 @@ module Fog
       class Real
 
         def get_api_client(identifier, options = {})
+          return nil if identifier.nil? || identifier == ""
           request(
             :expects  => [200],
             :method   => 'GET',

lib/fog/brightbox/requests/compute/get_cloud_ip.rb

@@ -4,6 +4,7 @@ module Fog
       class Real
 
         def get_cloud_ip(identifier, options = {})
+          return nil if identifier.nil? || identifier == ""
           request(
             :expects  => [200],
             :method   => 'GET',

lib/fog/brightbox/requests/compute/get_image.rb

@@ -4,6 +4,7 @@ module Fog
       class Real
 
         def get_image(identifier, options = {})
+          return nil if identifier.nil? || identifier == ""
           request(
             :expects  => [200],
             :method   => 'GET',

lib/fog/brightbox/requests/compute/get_interface.rb

@@ -4,6 +4,7 @@ module Fog
       class Real
 
         def get_interface(identifier, options = {})
+          return nil if identifier.nil? || identifier == ""
           request(
             :expects  => [200],
             :method   => 'GET',

lib/fog/brightbox/requests/compute/get_server.rb

@@ -4,6 +4,7 @@ module Fog
       class Real
 
         def get_server(identifier, options = {})
+          return nil if identifier.nil? || identifier == ""
           request(
             :expects  => [200],
             :method   => 'GET',

lib/fog/brightbox/requests/compute/get_server_type.rb

@@ -4,6 +4,7 @@ module Fog
       class Real
 
         def get_server_type(identifier, options = {})
+          return nil if identifier.nil? || identifier == ""
           request(
             :expects  => [200],
             :method   => 'GET',

lib/fog/brightbox/requests/compute/get_user.rb

@@ -4,6 +4,7 @@ module Fog
       class Real
 
         def get_user(identifier, options = {})
+          return nil if identifier.nil? || identifier == ""
           request(
             :expects  => [200],
             :method   => 'GET',

lib/fog/brightbox/requests/compute/get_zone.rb

@@ -4,6 +4,7 @@ module Fog
       class Real
 
         def get_zone(identifier, options = {})
+          return nil if identifier.nil? || identifier == ""
           request(
             :expects  => [200],
             :method   => 'GET',

lib/fog/brightbox/requests/compute/map_cloud_ip.rb

@@ -4,6 +4,7 @@ module Fog
       class Real
 
         def map_cloud_ip(identifier, options = {})
+          return nil if identifier.nil? || identifier == ""
           request(
             :expects  => [202],
             :method   => 'POST',

lib/fog/brightbox/requests/compute/restart_server.rb

@@ -4,6 +4,7 @@ module Fog
       class Real
 
         def restart_server(identifier, options = {})
+          return nil if identifier.nil? || identifier == ""
           request(
             :expects  => [202],
             :method   => 'POST',

lib/fog/brightbox/requests/compute/shutdown_server.rb

@@ -4,6 +4,7 @@ module Fog
       class Real
 
         def shutdown_server(identifier, options = {})
+          return nil if identifier.nil? || identifier == ""
           request(
             :expects  => [202],
             :method   => 'POST',

lib/fog/brightbox/requests/compute/snapshot_server.rb

@@ -4,6 +4,7 @@ module Fog
       class Real
 
         def snapshot_server(identifier, options = {})
+          return nil if identifier.nil? || identifier == ""
           request(
             :expects  => [202],
             :method   => 'POST',

lib/fog/brightbox/requests/compute/start_server.rb

@@ -4,6 +4,7 @@ module Fog
       class Real
 
         def start_server(identifier, options = {})
+          return nil if identifier.nil? || identifier == ""
           request(
             :expects  => [202],
             :method   => 'POST',

lib/fog/brightbox/requests/compute/stop_server.rb

@@ -4,6 +4,7 @@ module Fog
       class Real
 
         def stop_server(identifier, options = {})
+          return nil if identifier.nil? || identifier == ""
           request(
             :expects  => [202],
             :method   => 'POST',

lib/fog/brightbox/requests/compute/unmap_cloud_ip.rb

@@ -4,6 +4,7 @@ module Fog
       class Real
 
         def unmap_cloud_ip(identifier, options = {})
+          return nil if identifier.nil? || identifier == ""
           request(
             :expects  => [202],
             :method   => 'POST',

lib/fog/brightbox/requests/compute/update_api_client.rb

@@ -4,6 +4,7 @@ module Fog
       class Real
 
         def update_api_client(identifier, options = {})
+          return nil if identifier.nil? || identifier == ""
           request(
             :expects  => [200],
             :method   => 'PUT',

lib/fog/brightbox/requests/compute/update_image.rb

@@ -4,6 +4,7 @@ module Fog
       class Real
 
         def update_image(identifier, options = {})
+          return nil if identifier.nil? || identifier == ""
           request(
             :expects  => [200],
             :method   => 'PUT',

lib/fog/brightbox/requests/compute/update_server.rb

@@ -4,6 +4,7 @@ module Fog
       class Real
 
         def update_server(identifier, options = {})
+          return nil if identifier.nil? || identifier == ""
           request(
             :expects  => [200],
             :method   => 'PUT',

lib/fog/brightbox/requests/compute/update_user.rb

@@ -4,6 +4,7 @@ module Fog
       class Real
 
         def update_user(identifier, options = {})
+          return nil if identifier.nil? || identifier == ""
           request(
             :expects  => [200],
             :method   => 'PUT',