{"action":"created","issue":{"url":"https://api.github.com/repos/apache/sling/issues/115","labels_url":"https://api.github.com/repos/apache/sling/issues/115/labels{/name}","comments_url":"https://api.github.com/repos/apache/sling/issues/115/comments","events_url":"https://api.github.com/repos/apache/sling/issues/115/events","html_url":"https://github.com/apache/sling/pull/115","id":119102446,"number":115,"title":"SLING-5335: maven-sling-plugin support for SlingPostServlet usage.","user":{"login":"kemuri-9","id":3010610,"avatar_url":"https://avatars.githubusercontent.com/u/3010610?v=3","gravatar_id":"","url":"https://api.github.com/users/kemuri-9","html_url":"https://github.com/kemuri-9","followers_url":"https://api.github.com/users/kemuri-9/followers","following_url":"https://api.github.com/users/kemuri-9/following{/other_user}","gists_url":"https://api.github.com/users/kemuri-9/gists{/gist_id}","starred_url":"https://api.github.com/users/kemuri-9/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kemuri-9/subscriptions","organizations_url":"https://api.github.com/users/kemuri-9/orgs","repos_url":"https://api.github.com/users/kemuri-9/repos","events_url":"https://api.github.com/users/kemuri-9/events{/privacy}","received_events_url":"https://api.github.com/users/kemuri-9/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2015-11-26T21:11:41Z","updated_at":"2015-12-02T13:59:54Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/apache/sling/pulls/115","html_url":"https://github.com/apache/sling/pull/115","diff_url":"https://github.com/apache/sling/pull/115.diff","patch_url":"https://github.com/apache/sling/pull/115.patch"},"body":"* Allow usage of SlingPostServlet for install and uninstall operations."},"comment":{"url":"https://api.github.com/repos/apache/sling/issues/comments/161302073","html_url":"https://github.com/apache/sling/pull/115#issuecomment-161302073","issue_url":"https://api.github.com/repos/apache/sling/issues/115","id":161302073,"user":{"login":"rombert","id":21237,"avatar_url":"https://avatars.githubusercontent.com/u/21237?v=3","gravatar_id":"","url":"https://api.github.com/users/rombert","html_url":"https://github.com/rombert","followers_url":"https://api.github.com/users/rombert/followers","following_url":"https://api.github.com/users/rombert/following{/other_user}","gists_url":"https://api.github.com/users/rombert/gists{/gist_id}","starred_url":"https://api.github.com/users/rombert/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rombert/subscriptions","organizations_url":"https://api.github.com/users/rombert/orgs","repos_url":"https://api.github.com/users/rombert/repos","events_url":"https://api.github.com/users/rombert/events{/privacy}","received_events_url":"https://api.github.com/users/rombert/received_events","type":"User","site_admin":false},"created_at":"2015-12-02T13:59:54Z","updated_at":"2015-12-02T13:59:54Z","body":"> Sure a sanity check could be added to raise an error if both usePut and useSlingPost are both true. I was thinking about adding this, but wasn't sure if it was expected/appropriate.\r\n\r\nI think it's better to fail hard. For instance if usePut is set to true in a parent pom and useSlingPost locally, it can get quite confusion.\r\n\r\n> is it expected that this is an additional parameter and usePut still exists for backwards compatibility purposes\r\n\r\nMy suggestion would be to always obey the new parameter if non-null, and fall back to checking `usePut` if needed."}}