Changeset 4bbbb285070c244ad7eca783a459593a741c6d3b

Timestamp:

04/02/12 17:00:24 (1 year ago)

Author:

Philip Maddox <pmaddox@circonus.com>

git-committer:

Philip Maddox <pmaddox@circonus.com> 1333386024 -0400

git-parent:

[b2a08ebd72aed85e425a8b853ac201b7faac0453]

git-author:

Philip Maddox <pmaddox@circonus.com> 1333386024 -0400

Message:

Moved declaring and initializing DocumentBuilder? object into dopost method - the way it was, if more than one thread attempted to call dopost, they would clash and produce an error; this makes it thread safe

Files:

• src/java/com/omniti/jezebel/JezebelDispatch.java (modified) (2 diffs)

src/java/com/omniti/jezebel/JezebelDispatch.java

@@ -65 +65 @@
-      new JezebelClassLoader(JezebelDispatch.class.getClassLoader());
-
-    DocumentBuilder builder;
-
-    public JezebelDispatch() {
-      try { builder = builderfactory.newDocumentBuilder(); }
-      catch (Exception e) { throw new RuntimeException(e); }
-    }
-    public void doPost(HttpServletRequest request,
@@ -75 +71 @@
-      Document postDoc;
-      Hashtable<String,String> info, config;
+      DocumentBuilder builder;
-
+      try { builder = builderfactory.newDocumentBuilder(); }
+      catch (Exception e) { throw new RuntimeException(e); }
-      try {
-        postDoc = builder.parse(request.getInputStream());