Memory leak in UrlRewriteFilter#getFullVersionString

[GoogleCodeExporter]

The input stream opened for the properties file is never closed. A long running 
container will drain memory. Close the input stream in a finally block.

Original issue reported on code.google.com by 1983-01...@gmx.net on 21 Jul 2012 at 5:49

[GoogleCodeExporter]

Done
http://code.google.com/p/urlrewritefilter/source/detail?r=423

Original comment by p...@tuckey.org on 24 Jul 2012 at 8:28

• Changed state: Fixed

[GoogleCodeExporter]

You can avoid the duplicate try-catch clause by doing:

    public static String getFullVersionString() {
        Properties props = new Properties();
        String buildNumberStr = "";
        InputStream is = null;
        try {
            is = UrlRewriteFilter.class.getResourceAsStream("build.number.properties");
            if ( is != null ) {
                props.load(is);
                String buildNumber = (String) props.get("build.number");
                if (!StringUtils.isBlank(buildNumber)){
                    buildNumberStr =  props.get("project.version") + " build " + props.get("build.number");
                }
            }   
        } catch (IOException e) {
            log.error(e);
        } finally {
            if ( is != null)
                is.close();
        }

        return buildNumberStr;
    }

Original comment by 1983-01...@gmx.net on 24 Jul 2012 at 10:21