Merge pull request #29 from MrCreosote/develop

Fix logging bug with auth startup.

Author: Roman Sutormin

RELEASE_NOTES.txt

@@ -4,6 +4,14 @@ OVERVIEW
 -----------------------------------------
 Repo for code shared between Java services.
 
+VERSION: 0.0.22 (Released 9/15/2016)
+--------------------------------------
+
+UPDATED FEATURES / MAJOR BUG FIXES:
+- Fixed a bug that would throw an IllegalStateError when configuration of the
+  authentication client in JsonServerServlet failed rather than logging the
+  error.
+
 VERSION: 0.0.21 (Released 8/26/2016)
 --------------------------------------
 

src/us/kbase/common/service/JsonServerServlet.java

@@ -188,16 +188,18 @@ private ConfigurableAuthService getAuth(final Map<String, String> config) {
 				c.withKBaseAuthServerURL(new URL(authURL));
 			} catch (URISyntaxException | MalformedURLException e) {
 				startupFailed();
-				sysLogger.logErr(String.format(
-						"Authentication url %s is invalid", authURL));
+				sysLogger.log(LOG_LEVEL_ERR, getClass().getName(),
+						String.format(
+								"Authentication url %s is invalid", authURL));
 				return null;
 			}
 		}
 		try {
 			return new ConfigurableAuthService(c);
 		} catch (IOException e) {
 			startupFailed();
-			sysLogger.logErr("Couldn't connect to authentication service at " +
+			sysLogger.log(LOG_LEVEL_ERR, getClass().getName(),
+					"Couldn't connect to authentication service at " +
 					c.getAuthServerURL() + " : " + e.getLocalizedMessage());
 			return null;
 		}