{"action":"created","issue":{"url":"https://api.github.com/repos/DSpace/DSpace/issues/1054","repository_url":"https://api.github.com/repos/DSpace/DSpace","labels_url":"https://api.github.com/repos/DSpace/DSpace/issues/1054/labels{/name}","comments_url":"https://api.github.com/repos/DSpace/DSpace/issues/1054/comments","events_url":"https://api.github.com/repos/DSpace/DSpace/issues/1054/events","html_url":"https://github.com/DSpace/DSpace/pull/1054","id":106254709,"number":1054,"title":"DS-2748: Reduce Cocoon 404 log noise and add URIs to messages","user":{"login":"cwilper","id":118935,"avatar_url":"https://avatars.githubusercontent.com/u/118935?v=3","gravatar_id":"","url":"https://api.github.com/users/cwilper","html_url":"https://github.com/cwilper","followers_url":"https://api.github.com/users/cwilper/followers","following_url":"https://api.github.com/users/cwilper/following{/other_user}","gists_url":"https://api.github.com/users/cwilper/gists{/gist_id}","starred_url":"https://api.github.com/users/cwilper/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cwilper/subscriptions","organizations_url":"https://api.github.com/users/cwilper/orgs","repos_url":"https://api.github.com/users/cwilper/repos","events_url":"https://api.github.com/users/cwilper/events{/privacy}","received_events_url":"https://api.github.com/users/cwilper/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/DSpace/DSpace/labels/bug","name":"bug","color":"fc2929"},{"url":"https://api.github.com/repos/DSpace/DSpace/labels/merge%20conflict","name":"merge conflict","color":"fef2c0"}],"state":"open","locked":false,"assignee":{"login":"tdonohue","id":483997,"avatar_url":"https://avatars.githubusercontent.com/u/483997?v=3","gravatar_id":"","url":"https://api.github.com/users/tdonohue","html_url":"https://github.com/tdonohue","followers_url":"https://api.github.com/users/tdonohue/followers","following_url":"https://api.github.com/users/tdonohue/following{/other_user}","gists_url":"https://api.github.com/users/tdonohue/gists{/gist_id}","starred_url":"https://api.github.com/users/tdonohue/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tdonohue/subscriptions","organizations_url":"https://api.github.com/users/tdonohue/orgs","repos_url":"https://api.github.com/users/tdonohue/repos","events_url":"https://api.github.com/users/tdonohue/events{/privacy}","received_events_url":"https://api.github.com/users/tdonohue/received_events","type":"User","site_admin":false},"assignees":[{"login":"tdonohue","id":483997,"avatar_url":"https://avatars.githubusercontent.com/u/483997?v=3","gravatar_id":"","url":"https://api.github.com/users/tdonohue","html_url":"https://github.com/tdonohue","followers_url":"https://api.github.com/users/tdonohue/followers","following_url":"https://api.github.com/users/tdonohue/following{/other_user}","gists_url":"https://api.github.com/users/tdonohue/gists{/gist_id}","starred_url":"https://api.github.com/users/tdonohue/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tdonohue/subscriptions","organizations_url":"https://api.github.com/users/tdonohue/orgs","repos_url":"https://api.github.com/users/tdonohue/repos","events_url":"https://api.github.com/users/tdonohue/events{/privacy}","received_events_url":"https://api.github.com/users/tdonohue/received_events","type":"User","site_admin":false}],"milestone":{"url":"https://api.github.com/repos/DSpace/DSpace/milestones/9","html_url":"https://github.com/DSpace/DSpace/milestones/6.0","labels_url":"https://api.github.com/repos/DSpace/DSpace/milestones/9/labels","id":942273,"number":9,"title":"6.0","description":null,"creator":{"login":"helix84","id":201026,"avatar_url":"https://avatars.githubusercontent.com/u/201026?v=3","gravatar_id":"","url":"https://api.github.com/users/helix84","html_url":"https://github.com/helix84","followers_url":"https://api.github.com/users/helix84/followers","following_url":"https://api.github.com/users/helix84/following{/other_user}","gists_url":"https://api.github.com/users/helix84/gists{/gist_id}","starred_url":"https://api.github.com/users/helix84/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/helix84/subscriptions","organizations_url":"https://api.github.com/users/helix84/orgs","repos_url":"https://api.github.com/users/helix84/repos","events_url":"https://api.github.com/users/helix84/events{/privacy}","received_events_url":"https://api.github.com/users/helix84/received_events","type":"User","site_admin":false},"open_issues":37,"closed_issues":375,"state":"open","created_at":"2015-01-19T18:03:38Z","updated_at":"2016-08-25T17:30:41Z","due_on":null,"closed_at":null},"comments":5,"created_at":"2015-09-14T01:21:06Z","updated_at":"2016-08-26T14:09:13Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/DSpace/DSpace/pulls/1054","html_url":"https://github.com/DSpace/DSpace/pull/1054","diff_url":"https://github.com/DSpace/DSpace/pull/1054.diff","patch_url":"https://github.com/DSpace/DSpace/pull/1054.patch"},"body":"https://jira.duraspace.org/browse/DS-2748\r\n\r\nThis switches to using a patched version of cocoon-sitemap-impl[1] which avoids putting lengthy ERROR level stack traces in the Cocoon logs when ResourceNotFoundExceptions are encountered. Instead, these are now logged at WARN level and do not include the stack trace -- only the sitemap URI.\r\n\r\nBefore testing this, you'll need to run the following:\r\n\r\n    git clone https://github.com/cwilper/dspace-cocoon-sitemap-impl.git\r\n    cd dspace-cocoon-sitemap-impl\r\n    mvn install\r\n\r\nThe above will put dspace-cocoon-sitemap-impl-1.0.0.jar in the right place under your local repository (~/.m2/repository/), and the dependency will be resolved correctly when you build DSpace.\r\n\r\n[1] Patched version of cocoon-sitemap-impl:\r\nhttps://github.com/cwilper/dspace-cocoon-sitemap-impl\r\n\r\nIf this PR is accepted, I'd advise moving this repository to the DSpace github organization and publishing the jar to Maven Central prior to merging this branch."},"comment":{"url":"https://api.github.com/repos/DSpace/DSpace/issues/comments/242745629","html_url":"https://github.com/DSpace/DSpace/pull/1054#issuecomment-242745629","issue_url":"https://api.github.com/repos/DSpace/DSpace/issues/1054","id":242745629,"user":{"login":"tdonohue","id":483997,"avatar_url":"https://avatars.githubusercontent.com/u/483997?v=3","gravatar_id":"","url":"https://api.github.com/users/tdonohue","html_url":"https://github.com/tdonohue","followers_url":"https://api.github.com/users/tdonohue/followers","following_url":"https://api.github.com/users/tdonohue/following{/other_user}","gists_url":"https://api.github.com/users/tdonohue/gists{/gist_id}","starred_url":"https://api.github.com/users/tdonohue/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tdonohue/subscriptions","organizations_url":"https://api.github.com/users/tdonohue/orgs","repos_url":"https://api.github.com/users/tdonohue/repos","events_url":"https://api.github.com/users/tdonohue/events{/privacy}","received_events_url":"https://api.github.com/users/tdonohue/received_events","type":"User","site_admin":false},"created_at":"2016-08-26T14:09:13Z","updated_at":"2016-08-26T14:09:13Z","body":"Tried giving this a test yesterday, but there are some dependency convergence issues in the `dspace-cocoon-sitemap-impl` which don't allow it to build with the latest DSpace POM.  I've logged a PR against @cwilper's codebase to show off how to reproduce them: https://github.com/cwilper/dspace-cocoon-sitemap-impl/pull/1\r\n\r\nA few seem easy enough to solve with POM exclusions, but a few more seem a bit tricky (as they seem to be dependency convergence issues in Cocoon's own codebase, sigh)"}}
{"actor":{"display_login":"tdonohue"}}