[jboss-svn-commits] JBL Code SVN: r30643 - labs/jbossesb/trunk/product/rosetta/src/org/jboss/soa/esb/actions/scripting.

[jboss-svn-commits at lists.jboss.org]

Author: tfennelly
Date: 2009-12-14 12:20:40 -0500 (Mon, 14 Dec 2009)
New Revision: 30643

Modified:
   labs/jbossesb/trunk/product/rosetta/src/org/jboss/soa/esb/actions/scripting/GroovyActionProcessor.java
Log:
https://jira.jboss.org/jira/browse/JBESB-3048
GroovyActionProcessor not thread safe

Modified: labs/jbossesb/trunk/product/rosetta/src/org/jboss/soa/esb/actions/scripting/GroovyActionProcessor.java
===================================================================
--- labs/jbossesb/trunk/product/rosetta/src/org/jboss/soa/esb/actions/scripting/GroovyActionProcessor.java	2009-12-14 16:48:20 UTC (rev 30642)
+++ labs/jbossesb/trunk/product/rosetta/src/org/jboss/soa/esb/actions/scripting/GroovyActionProcessor.java	2009-12-14 17:20:40 UTC (rev 30643)
@@ -131,8 +131,9 @@
      * scriptPath - The path to the script that will be run
      * script - the name of the script that will be run (include .groovy exception)
      */
-    public Message process(Message message) throws ActionProcessingException {
+    public synchronized Message process(Message message) throws ActionProcessingException {
         try {
+        	Script execScript = getScript(message);
             // create a Java/Groovy binding for the message object
             //
             Binding binding = new Binding();
@@ -144,13 +145,10 @@
             binding.setVariable("payloadProxy", payloadProxy);
             binding.setVariable("logger", logger);
 
-            Script execScript = getScript(message);
-            Object returnVal;
-            synchronized (execScript) {
-                execScript.setBinding(binding);
-                returnVal = execScript.run();
-            }
+            execScript.setBinding(binding);
 
+            Object returnVal = execScript.run();
+
             if(returnVal instanceof Message) {
                 return (Message) returnVal;
             } else {