cleaned up IconServlet

Author: FreakyBytes

pom.xml

@@ -37,7 +37,7 @@
 		<dependency>
 			<groupId>de.unirostock.sems</groupId>
 			<artifactId>CombineExt</artifactId>
-			<version>0.3.4</version>
+			<version>1.0.2</version>
 		</dependency>
 		<dependency>
 			<groupId>de.binfalse</groupId>

src/main/java/de/unirostock/sems/cbarchive/web/servlet/IconServlet.java

@@ -20,7 +20,6 @@
 import java.io.IOException;
 import java.io.InputStream;
 import java.io.OutputStream;
-import java.io.OutputStreamWriter;
 import java.io.UnsupportedEncodingException;
 import java.net.URI;
 import java.net.URISyntaxException;
@@ -50,10 +49,11 @@ protected void doGet(HttpServletRequest request, HttpServletResponse response) t
 
 		if( requestUrl != null && !requestUrl.isEmpty() ) {
 
+			String formatString = null;
 			try {
 				response.setContentType( "image/png" );
 
-				String formatString = requestUrl.substring( requestUrl.indexOf("res/icon/") + 9 );
+				formatString = requestUrl.substring( requestUrl.indexOf("res/icon/") + 9 );
 				LOGGER.debug("format: ", formatString);
 
 				URI format = new URI( formatString );
@@ -71,24 +71,11 @@ protected void doGet(HttpServletRequest request, HttpServletResponse response) t
 				response.flushBuffer();
 
 			} catch (IOException e) {
-				// TODO Auto-generated catch block
-				e.printStackTrace();
+				LOGGER.error(e, "IOException while loading icon");
 			} catch (URISyntaxException e) {
-				// TODO Auto-generated catch block
-				e.printStackTrace();
+				LOGGER.warn(e, "Not able to generate URL for Iconizer: ", formatString);
 			}
 
-			
-		}
-		
-		try {
-			OutputStream output = response.getOutputStream();
-			new OutputStreamWriter(output).append("Hello World");
-			output.close();
-			response.flushBuffer();
-		} catch (IOException e) {
-			// TODO Auto-generated catch block
-			e.printStackTrace();
 		}
 
 	}