{"action":"created","comment":{"url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/pulls/comments/137188246","pull_request_review_id":60811190,"id":137188246,"diff_hunk":"@@ -170,16 +193,17 @@ private void setupBuildVariables(AbstractBuild build, String serverUrl) {\n \t\tif (StringUtils.isNotEmpty(serverUrl)) {\n \t\t\tparameters.add(new StringParameterValue(BuildVarKeys.BUILD_VAR_KEY_GLOBAL_SERVER_URL, serverUrl));\n \t\t}\n-\t\tif (StringUtils.isNotEmpty(globalConfig.username)) {\n+\t\tif (globalConfig != null && StringUtils.isNotEmpty(globalConfig.username)) {\n \t\t\tparameters.add(new StringParameterValue(BuildVarKeys.BUILD_VAR_KEY_GLOBAL_USERNAME, globalConfig.username));\n \t\t}\n-\t\tif (StringUtils.isNotEmpty(globalConfig.password)) {\n+\t\tif (globalConfig != null && StringUtils.isNotEmpty(globalConfig.password)) {","path":"src/main/java/com/dynatrace/jenkins/dashboard/TABuildWrapper.java","position":174,"original_position":174,"commit_id":"7cb124d31cc9b5ec60e0702c52707bb812699614","original_commit_id":"7cb124d31cc9b5ec60e0702c52707bb812699614","user":{"login":"dglugla","id":3448216,"avatar_url":"https://avatars1.githubusercontent.com/u/3448216?v=4","gravatar_id":"","url":"https://api.github.com/users/dglugla","html_url":"https://github.com/dglugla","followers_url":"https://api.github.com/users/dglugla/followers","following_url":"https://api.github.com/users/dglugla/following{/other_user}","gists_url":"https://api.github.com/users/dglugla/gists{/gist_id}","starred_url":"https://api.github.com/users/dglugla/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dglugla/subscriptions","organizations_url":"https://api.github.com/users/dglugla/orgs","repos_url":"https://api.github.com/users/dglugla/repos","events_url":"https://api.github.com/users/dglugla/events{/privacy}","received_events_url":"https://api.github.com/users/dglugla/received_events","type":"User","site_admin":false},"body":"change to:\r\n```\r\nif(globalConfig != null) {\r\n  if(username not empty) ...\r\n  if(password not empty) ...\r\n}\r\n```","created_at":"2017-09-06T07:17:55Z","updated_at":"2017-09-06T07:33:11Z","html_url":"https://github.com/jenkinsci/dynatrace-plugin/pull/7#discussion_r137188246","pull_request_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/pulls/7","author_association":"COLLABORATOR","_links":{"self":{"href":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/pulls/comments/137188246"},"html":{"href":"https://github.com/jenkinsci/dynatrace-plugin/pull/7#discussion_r137188246"},"pull_request":{"href":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/pulls/7"}}},"pull_request":{"url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/pulls/7","id":139156660,"html_url":"https://github.com/jenkinsci/dynatrace-plugin/pull/7","diff_url":"https://github.com/jenkinsci/dynatrace-plugin/pull/7.diff","patch_url":"https://github.com/jenkinsci/dynatrace-plugin/pull/7.patch","issue_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/issues/7","number":7,"state":"open","locked":false,"title":"Pipeline compatibility","user":{"login":"lugowskii","id":22976540,"avatar_url":"https://avatars3.githubusercontent.com/u/22976540?v=4","gravatar_id":"","url":"https://api.github.com/users/lugowskii","html_url":"https://github.com/lugowskii","followers_url":"https://api.github.com/users/lugowskii/followers","following_url":"https://api.github.com/users/lugowskii/following{/other_user}","gists_url":"https://api.github.com/users/lugowskii/gists{/gist_id}","starred_url":"https://api.github.com/users/lugowskii/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lugowskii/subscriptions","organizations_url":"https://api.github.com/users/lugowskii/orgs","repos_url":"https://api.github.com/users/lugowskii/repos","events_url":"https://api.github.com/users/lugowskii/events{/privacy}","received_events_url":"https://api.github.com/users/lugowskii/received_events","type":"User","site_admin":false},"body":"","created_at":"2017-09-04T13:20:47Z","updated_at":"2017-09-06T07:33:12Z","closed_at":null,"merged_at":null,"merge_commit_sha":"f1cdf450c341974d0a360632245acdf27b4aa85c","assignee":null,"assignees":[],"requested_reviewers":[],"milestone":null,"commits_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/pulls/7/commits","review_comments_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/pulls/7/comments","review_comment_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/pulls/comments{/number}","comments_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/issues/7/comments","statuses_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/statuses/7cb124d31cc9b5ec60e0702c52707bb812699614","head":{"label":"jenkinsci:pipeline-compatibility","ref":"pipeline-compatibility","sha":"7cb124d31cc9b5ec60e0702c52707bb812699614","user":{"login":"jenkinsci","id":107424,"avatar_url":"https://avatars0.githubusercontent.com/u/107424?v=4","gravatar_id":"","url":"https://api.github.com/users/jenkinsci","html_url":"https://github.com/jenkinsci","followers_url":"https://api.github.com/users/jenkinsci/followers","following_url":"https://api.github.com/users/jenkinsci/following{/other_user}","gists_url":"https://api.github.com/users/jenkinsci/gists{/gist_id}","starred_url":"https://api.github.com/users/jenkinsci/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jenkinsci/subscriptions","organizations_url":"https://api.github.com/users/jenkinsci/orgs","repos_url":"https://api.github.com/users/jenkinsci/repos","events_url":"https://api.github.com/users/jenkinsci/events{/privacy}","received_events_url":"https://api.github.com/users/jenkinsci/received_events","type":"Organization","site_admin":false},"repo":{"id":25816451,"name":"dynatrace-plugin","full_name":"jenkinsci/dynatrace-plugin","owner":{"login":"jenkinsci","id":107424,"avatar_url":"https://avatars0.githubusercontent.com/u/107424?v=4","gravatar_id":"","url":"https://api.github.com/users/jenkinsci","html_url":"https://github.com/jenkinsci","followers_url":"https://api.github.com/users/jenkinsci/followers","following_url":"https://api.github.com/users/jenkinsci/following{/other_user}","gists_url":"https://api.github.com/users/jenkinsci/gists{/gist_id}","starred_url":"https://api.github.com/users/jenkinsci/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jenkinsci/subscriptions","organizations_url":"https://api.github.com/users/jenkinsci/orgs","repos_url":"https://api.github.com/users/jenkinsci/repos","events_url":"https://api.github.com/users/jenkinsci/events{/privacy}","received_events_url":"https://api.github.com/users/jenkinsci/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/jenkinsci/dynatrace-plugin","description":"Plugin for Jenkins CI server that display data collected by the dynaTrace Test Automation features","fork":false,"url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin","forks_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/forks","keys_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/keys{/key_id}","collaborators_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/teams","hooks_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/hooks","issue_events_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/issues/events{/number}","events_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/events","assignees_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/assignees{/user}","branches_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/branches{/branch}","tags_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/tags","blobs_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/git/refs{/sha}","trees_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/git/trees{/sha}","statuses_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/statuses/{sha}","languages_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/languages","stargazers_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/stargazers","contributors_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/contributors","subscribers_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/subscribers","subscription_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/subscription","commits_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/commits{/sha}","git_commits_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/git/commits{/sha}","comments_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/comments{/number}","issue_comment_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/issues/comments{/number}","contents_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/contents/{+path}","compare_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/compare/{base}...{head}","merges_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/merges","archive_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/downloads","issues_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/issues{/number}","pulls_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/pulls{/number}","milestones_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/milestones{/number}","notifications_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/labels{/name}","releases_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/releases{/id}","deployments_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/deployments","created_at":"2014-10-27T11:49:39Z","updated_at":"2017-01-17T16:15:03Z","pushed_at":"2017-09-05T07:08:01Z","git_url":"git://github.com/jenkinsci/dynatrace-plugin.git","ssh_url":"git@github.com:jenkinsci/dynatrace-plugin.git","clone_url":"https://github.com/jenkinsci/dynatrace-plugin.git","svn_url":"https://github.com/jenkinsci/dynatrace-plugin","homepage":"https://community.dynatrace.com/community/display/DL/Test+Automation+Plugin+for+Jenkins","size":3426,"stargazers_count":0,"watchers_count":0,"language":"Java","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":false,"has_pages":false,"forks_count":9,"mirror_url":null,"open_issues_count":3,"forks":9,"open_issues":3,"watchers":0,"default_branch":"master"}},"base":{"label":"jenkinsci:master","ref":"master","sha":"3bcb6f671c564ddd128938777941877a38efb6e0","user":{"login":"jenkinsci","id":107424,"avatar_url":"https://avatars0.githubusercontent.com/u/107424?v=4","gravatar_id":"","url":"https://api.github.com/users/jenkinsci","html_url":"https://github.com/jenkinsci","followers_url":"https://api.github.com/users/jenkinsci/followers","following_url":"https://api.github.com/users/jenkinsci/following{/other_user}","gists_url":"https://api.github.com/users/jenkinsci/gists{/gist_id}","starred_url":"https://api.github.com/users/jenkinsci/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jenkinsci/subscriptions","organizations_url":"https://api.github.com/users/jenkinsci/orgs","repos_url":"https://api.github.com/users/jenkinsci/repos","events_url":"https://api.github.com/users/jenkinsci/events{/privacy}","received_events_url":"https://api.github.com/users/jenkinsci/received_events","type":"Organization","site_admin":false},"repo":{"id":25816451,"name":"dynatrace-plugin","full_name":"jenkinsci/dynatrace-plugin","owner":{"login":"jenkinsci","id":107424,"avatar_url":"https://avatars0.githubusercontent.com/u/107424?v=4","gravatar_id":"","url":"https://api.github.com/users/jenkinsci","html_url":"https://github.com/jenkinsci","followers_url":"https://api.github.com/users/jenkinsci/followers","following_url":"https://api.github.com/users/jenkinsci/following{/other_user}","gists_url":"https://api.github.com/users/jenkinsci/gists{/gist_id}","starred_url":"https://api.github.com/users/jenkinsci/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jenkinsci/subscriptions","organizations_url":"https://api.github.com/users/jenkinsci/orgs","repos_url":"https://api.github.com/users/jenkinsci/repos","events_url":"https://api.github.com/users/jenkinsci/events{/privacy}","received_events_url":"https://api.github.com/users/jenkinsci/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/jenkinsci/dynatrace-plugin","description":"Plugin for Jenkins CI server that display data collected by the dynaTrace Test Automation features","fork":false,"url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin","forks_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/forks","keys_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/keys{/key_id}","collaborators_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/teams","hooks_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/hooks","issue_events_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/issues/events{/number}","events_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/events","assignees_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/assignees{/user}","branches_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/branches{/branch}","tags_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/tags","blobs_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/git/refs{/sha}","trees_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/git/trees{/sha}","statuses_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/statuses/{sha}","languages_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/languages","stargazers_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/stargazers","contributors_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/contributors","subscribers_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/subscribers","subscription_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/subscription","commits_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/commits{/sha}","git_commits_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/git/commits{/sha}","comments_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/comments{/number}","issue_comment_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/issues/comments{/number}","contents_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/contents/{+path}","compare_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/compare/{base}...{head}","merges_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/merges","archive_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/downloads","issues_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/issues{/number}","pulls_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/pulls{/number}","milestones_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/milestones{/number}","notifications_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/labels{/name}","releases_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/releases{/id}","deployments_url":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/deployments","created_at":"2014-10-27T11:49:39Z","updated_at":"2017-01-17T16:15:03Z","pushed_at":"2017-09-05T07:08:01Z","git_url":"git://github.com/jenkinsci/dynatrace-plugin.git","ssh_url":"git@github.com:jenkinsci/dynatrace-plugin.git","clone_url":"https://github.com/jenkinsci/dynatrace-plugin.git","svn_url":"https://github.com/jenkinsci/dynatrace-plugin","homepage":"https://community.dynatrace.com/community/display/DL/Test+Automation+Plugin+for+Jenkins","size":3426,"stargazers_count":0,"watchers_count":0,"language":"Java","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":false,"has_pages":false,"forks_count":9,"mirror_url":null,"open_issues_count":3,"forks":9,"open_issues":3,"watchers":0,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/pulls/7"},"html":{"href":"https://github.com/jenkinsci/dynatrace-plugin/pull/7"},"issue":{"href":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/issues/7"},"comments":{"href":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/issues/7/comments"},"review_comments":{"href":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/pulls/7/comments"},"review_comment":{"href":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/pulls/7/commits"},"statuses":{"href":"https://api.github.com/repos/jenkinsci/dynatrace-plugin/statuses/7cb124d31cc9b5ec60e0702c52707bb812699614"}},"author_association":"COLLABORATOR"}}
{"actor":{"display_login":"dglugla"}}