{"action":"created","issue":{"url":"https://api.github.com/repos/apache/incubator-netbeans/issues/936","repository_url":"https://api.github.com/repos/apache/incubator-netbeans","labels_url":"https://api.github.com/repos/apache/incubator-netbeans/issues/936/labels{/name}","comments_url":"https://api.github.com/repos/apache/incubator-netbeans/issues/936/comments","events_url":"https://api.github.com/repos/apache/incubator-netbeans/issues/936/events","html_url":"https://github.com/apache/incubator-netbeans/pull/936","id":365606311,"node_id":"MDExOlB1bGxSZXF1ZXN0MjE5NDg3MjA5","number":936,"title":"[NETBEANS-1074] : module review web.monitor","user":{"login":"arusinha","id":32263739,"node_id":"MDQ6VXNlcjMyMjYzNzM5","avatar_url":"https://avatars0.githubusercontent.com/u/32263739?v=4","gravatar_id":"","url":"https://api.github.com/users/arusinha","html_url":"https://github.com/arusinha","followers_url":"https://api.github.com/users/arusinha/followers","following_url":"https://api.github.com/users/arusinha/following{/other_user}","gists_url":"https://api.github.com/users/arusinha/gists{/gist_id}","starred_url":"https://api.github.com/users/arusinha/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/arusinha/subscriptions","organizations_url":"https://api.github.com/users/arusinha/orgs","repos_url":"https://api.github.com/users/arusinha/repos","events_url":"https://api.github.com/users/arusinha/events{/privacy}","received_events_url":"https://api.github.com/users/arusinha/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":4,"created_at":"2018-10-01T19:23:42Z","updated_at":"2018-10-09T10:07:49Z","closed_at":null,"author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/apache/incubator-netbeans/pulls/936","html_url":"https://github.com/apache/incubator-netbeans/pull/936","diff_url":"https://github.com/apache/incubator-netbeans/pull/936.diff","patch_url":"https://github.com/apache/incubator-netbeans/pull/936.patch"},"body":"Changes.\r\n1. Added licenseinfo.xml for image file\r\n2.  servlet-2.3.jar  is not present in maven repo, it is now replaced with generated jar from servlet-api2.4.jar and jsp-api-2.0.jar\r\n3. Added license header for l10n.list,monitor.dtd\r\n"},"comment":{"url":"https://api.github.com/repos/apache/incubator-netbeans/issues/comments/428136203","html_url":"https://github.com/apache/incubator-netbeans/pull/936#issuecomment-428136203","issue_url":"https://api.github.com/repos/apache/incubator-netbeans/issues/936","id":428136203,"node_id":"MDEyOklzc3VlQ29tbWVudDQyODEzNjIwMw==","user":{"login":"matthiasblaesing","id":2179736,"node_id":"MDQ6VXNlcjIxNzk3MzY=","avatar_url":"https://avatars2.githubusercontent.com/u/2179736?v=4","gravatar_id":"","url":"https://api.github.com/users/matthiasblaesing","html_url":"https://github.com/matthiasblaesing","followers_url":"https://api.github.com/users/matthiasblaesing/followers","following_url":"https://api.github.com/users/matthiasblaesing/following{/other_user}","gists_url":"https://api.github.com/users/matthiasblaesing/gists{/gist_id}","starred_url":"https://api.github.com/users/matthiasblaesing/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/matthiasblaesing/subscriptions","organizations_url":"https://api.github.com/users/matthiasblaesing/orgs","repos_url":"https://api.github.com/users/matthiasblaesing/repos","events_url":"https://api.github.com/users/matthiasblaesing/events{/privacy}","received_events_url":"https://api.github.com/users/matthiasblaesing/received_events","type":"User","site_admin":false},"created_at":"2018-10-09T10:07:49Z","updated_at":"2018-10-09T10:07:49Z","author_association":"CONTRIBUTOR","body":"Thank you - while investigating the version difference between the `servlet-api` and the `jsp-api` dependcy, I noticed, that the `jsp-api` is not used. The code in `serversrc` only depends on the `servlet-api`.\r\n\r\nFor the `servlet-api-6.0.13-license.txt` please add a `Type: compile-time` header.\r\n\r\nWith these changes I think this is good to be merged."}}
{"actor":{"display_login":"matthiasblaesing"}}