{"action":"created","issue":{"url":"https://api.github.com/repos/puppetlabs/puppetserver/issues/1231","repository_url":"https://api.github.com/repos/puppetlabs/puppetserver","labels_url":"https://api.github.com/repos/puppetlabs/puppetserver/issues/1231/labels{/name}","comments_url":"https://api.github.com/repos/puppetlabs/puppetserver/issues/1231/comments","events_url":"https://api.github.com/repos/puppetlabs/puppetserver/issues/1231/events","html_url":"https://github.com/puppetlabs/puppetserver/pull/1231","id":180925835,"number":1231,"title":"(SERVER-1571) ensure environment passed to clj shellutils","user":{"login":"cprice404","id":1341071,"avatar_url":"https://avatars.githubusercontent.com/u/1341071?v=3","gravatar_id":"","url":"https://api.github.com/users/cprice404","html_url":"https://github.com/cprice404","followers_url":"https://api.github.com/users/cprice404/followers","following_url":"https://api.github.com/users/cprice404/following{/other_user}","gists_url":"https://api.github.com/users/cprice404/gists{/gist_id}","starred_url":"https://api.github.com/users/cprice404/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cprice404/subscriptions","organizations_url":"https://api.github.com/users/cprice404/orgs","repos_url":"https://api.github.com/users/cprice404/repos","events_url":"https://api.github.com/users/cprice404/events{/privacy}","received_events_url":"https://api.github.com/users/cprice404/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":7,"created_at":"2016-10-04T15:27:24Z","updated_at":"2016-10-04T22:54:25Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/puppetlabs/puppetserver/pulls/1231","html_url":"https://github.com/puppetlabs/puppetserver/pull/1231","diff_url":"https://github.com/puppetlabs/puppetserver/pull/1231.diff","patch_url":"https://github.com/puppetlabs/puppetserver/pull/1231.patch"},"body":"A recent refactor resulted in a subtle change in behavior in the\r\nclojure shellutils wrapper, wherein if you did not pass an\r\nexplicit list of environment variables to the command, you would\r\nend up with an empty map of environment variables for the shell\r\ncommand.  Prior to that refactor, you would instead have inherited\r\nall of the environment variables from the java process, which is\r\nthe desired behavior.  This change broke some acceptance tests which\r\nwere relying on having access to environment variables such as PATH.\r\n\r\nThis commit changes the environment variable behavior back to the\r\nprevious setup, where the java environment variables will be passed\r\nthrough to the shell command unless you explicitly override the\r\nenvironment."},"comment":{"url":"https://api.github.com/repos/puppetlabs/puppetserver/issues/comments/251537037","html_url":"https://github.com/puppetlabs/puppetserver/pull/1231#issuecomment-251537037","issue_url":"https://api.github.com/repos/puppetlabs/puppetserver/issues/1231","id":251537037,"user":{"login":"rlinehan","id":1530016,"avatar_url":"https://avatars.githubusercontent.com/u/1530016?v=3","gravatar_id":"","url":"https://api.github.com/users/rlinehan","html_url":"https://github.com/rlinehan","followers_url":"https://api.github.com/users/rlinehan/followers","following_url":"https://api.github.com/users/rlinehan/following{/other_user}","gists_url":"https://api.github.com/users/rlinehan/gists{/gist_id}","starred_url":"https://api.github.com/users/rlinehan/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rlinehan/subscriptions","organizations_url":"https://api.github.com/users/rlinehan/orgs","repos_url":"https://api.github.com/users/rlinehan/repos","events_url":"https://api.github.com/users/rlinehan/events{/privacy}","received_events_url":"https://api.github.com/users/rlinehan/received_events","type":"User","site_admin":false},"created_at":"2016-10-04T22:54:25Z","updated_at":"2016-10-04T22:54:25Z","body":"👍 works on my laptop now! should we wait for travis though?"}}
{"actor":{"display_login":"rlinehan"}}