Improve error messsages for a few sharp edges

Signed-off-by: Mitch Gaffigan <mitch.gaffigan@comcast.net>

Author: mgaffigan

server/src/com/mirth/connect/client/core/Client.java

@@ -196,7 +196,7 @@ public Connector getConnector(javax.ws.rs.client.Client client, Configuration ru
                 try {
                     config.register(Class.forName(apiProviderClass));
                 } catch (Throwable t) {
-                    logger.error("Error registering API provider class: " + apiProviderClass);
+                    logger.error("Error registering API provider class: {}", apiProviderClass, t);
                 }
             }
         }
@@ -219,7 +219,7 @@ public void registerApiProviders(Set<String> packageNames, Set<String> classes)
                         client.register(clazz);
                     }
                 } catch (Throwable t) {
-                    logger.error("Error registering API provider package: " + packageName);
+                    logger.error("Error registering API provider package: {}", packageName, t);
                 }
             }
         }
@@ -229,7 +229,7 @@ public void registerApiProviders(Set<String> packageNames, Set<String> classes)
                 try {
                     client.register(Class.forName(clazz));
                 } catch (Throwable t) {
-                    logger.error("Error registering API provider class: " + clazz);
+                    logger.error("Error registering API provider class: {}", clazz, t);
                 }
             }
         }

server/src/com/mirth/connect/server/api/MirthServlet.java

@@ -210,6 +210,9 @@ private void setContext() {
     }
 
     public void setOperation(Operation operation) {
+        if (operation == null) {
+            throw new MirthApiException("Method operation cannot be null.");
+        }
         if (extensionName != null) {
             operation = new ExtensionOperation(extensionName, operation);
         }