{"action":"created","issue":{"url":"https://api.github.com/repos/galan/packtag/issues/6","labels_url":"https://api.github.com/repos/galan/packtag/issues/6/labels{/name}","comments_url":"https://api.github.com/repos/galan/packtag/issues/6/comments","events_url":"https://api.github.com/repos/galan/packtag/issues/6/events","html_url":"https://github.com/galan/packtag/pull/6","id":70368451,"number":6,"title":"Fix a bug in a tomcat 8 environment","user":{"login":"liggiotti","id":5744574,"avatar_url":"https://avatars.githubusercontent.com/u/5744574?v=3","gravatar_id":"","url":"https://api.github.com/users/liggiotti","html_url":"https://github.com/liggiotti","followers_url":"https://api.github.com/users/liggiotti/followers","following_url":"https://api.github.com/users/liggiotti/following{/other_user}","gists_url":"https://api.github.com/users/liggiotti/gists{/gist_id}","starred_url":"https://api.github.com/users/liggiotti/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/liggiotti/subscriptions","organizations_url":"https://api.github.com/users/liggiotti/orgs","repos_url":"https://api.github.com/users/liggiotti/repos","events_url":"https://api.github.com/users/liggiotti/events{/privacy}","received_events_url":"https://api.github.com/users/liggiotti/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2015-04-23T10:10:24Z","updated_at":"2015-05-11T07:20:36Z","closed_at":"2015-05-06T20:37:15Z","pull_request":{"url":"https://api.github.com/repos/galan/packtag/pulls/6","html_url":"https://github.com/galan/packtag/pull/6","diff_url":"https://github.com/galan/packtag/pull/6.diff","patch_url":"https://github.com/galan/packtag/pull/6.patch"},"body":"Hi,\r\n\r\nIn a tomcat 8/java 8 environment, the method PackTag.storeFile() fails because of the way the directory path is computed.\r\n\r\nFor some reason, getServletContext().getRealPath(\"/foo\") doesn't produce a path with a trailing slash in tomcat 8, which leads to a wrong path because of the concatenation that follows.\r\n\r\nI managed to make it work in both cases (with or without a trailing slash) by using the FilenameUtils.concat() method from commons-io. I figured you wouldn't want to add the whole library in your dependencies just for that small use case, so I copied the relevant code of apache's FilenameUtils class into the utils package.\r\n\r\nI hope you'll find this change useful. Let me know if you want me to change something in the pull request."},"comment":{"url":"https://api.github.com/repos/galan/packtag/issues/comments/100794127","html_url":"https://github.com/galan/packtag/pull/6#issuecomment-100794127","issue_url":"https://api.github.com/repos/galan/packtag/issues/6","id":100794127,"user":{"login":"liggiotti","id":5744574,"avatar_url":"https://avatars.githubusercontent.com/u/5744574?v=3","gravatar_id":"","url":"https://api.github.com/users/liggiotti","html_url":"https://github.com/liggiotti","followers_url":"https://api.github.com/users/liggiotti/followers","following_url":"https://api.github.com/users/liggiotti/following{/other_user}","gists_url":"https://api.github.com/users/liggiotti/gists{/gist_id}","starred_url":"https://api.github.com/users/liggiotti/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/liggiotti/subscriptions","organizations_url":"https://api.github.com/users/liggiotti/orgs","repos_url":"https://api.github.com/users/liggiotti/repos","events_url":"https://api.github.com/users/liggiotti/events{/privacy}","received_events_url":"https://api.github.com/users/liggiotti/received_events","type":"User","site_admin":false},"created_at":"2015-05-11T07:20:36Z","updated_at":"2015-05-11T07:20:36Z","body":"Great! Good catch for the license.\r\nThank you for the merge."}}