Merge pull request #837 from jvshahid/fix-appdynamics-appname-escaping

Support appdynamics application names with spaces

Author: ekcasey

lib/java_buildpack/framework/app_dynamics_agent.rb

@@ -80,7 +80,7 @@ def supports?
       def application_name(java_opts, credentials)
         name = credentials['application-name'] || @configuration['default_application_name'] ||
           @application.details['application_name']
-        java_opts.add_system_property('appdynamics.agent.applicationName', name.to_s)
+        java_opts.add_system_property('appdynamics.agent.applicationName', "\\\"#{name}\\\"")
       end
 
       def account_access_key(java_opts, credentials)

spec/java_buildpack/framework/app_dynamics_agent_spec.rb

@@ -66,7 +66,7 @@
 
         expect(java_opts).to include('-javaagent:$PWD/.java-buildpack/app_dynamics_agent/javaagent.jar')
         expect(java_opts).to include('-Dappdynamics.controller.hostName=test-host-name')
-        expect(java_opts).to include('-Dappdynamics.agent.applicationName=test-application-name')
+        expect(java_opts).to include('-Dappdynamics.agent.applicationName=\"test-application-name\"')
         expect(java_opts).to include('-Dappdynamics.agent.tierName=test-application-name')
         expect(java_opts).to include('-Dappdynamics.agent.nodeName=$(expr "$VCAP_APPLICATION" : ' \
                                      '\'.*instance_index[": ]*\\([[:digit:]]*\\).*\')')
@@ -88,7 +88,7 @@
         it 'adds application_name from credentials to JAVA_OPTS if specified' do
           component.release
 
-          expect(java_opts).to include('-Dappdynamics.agent.applicationName=another-test-application-name')
+          expect(java_opts).to include('-Dappdynamics.agent.applicationName=\"another-test-application-name\"')
         end
       end