Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Commit ff26f68

Browse filesBrowse files
committed
hub4j#1671 Create a test demoing the issue
1 parent 7403e25 commit ff26f68
Copy full SHA for ff26f68

File tree

Expand file treeCollapse file tree

7 files changed

+371
-1
lines changed
Filter options
Expand file treeCollapse file tree

7 files changed

+371
-1
lines changed

‎src/test/java/org/kohsuke/github/GHRepositoryTest.java

Copy file name to clipboardExpand all lines: src/test/java/org/kohsuke/github/GHRepositoryTest.java
+12-1Lines changed: 12 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1594,4 +1594,15 @@ public void testRepoActionVariable() throws Exception {
15941594
GHRepositoryVariable variable = repository.getRepoVariable("myvar");
15951595
assertThat(variable.getValue(), is("this is my var value"));
15961596
}
1597-
}
1597+
1598+
/**
1599+
* Red test demoing the issue with a user having the maintain permission on a repository
1600+
* @throws IOException the exception
1601+
*/
1602+
@Test
1603+
public void cannotRetrievePermissionMaintainUser() throws IOException {
1604+
GHRepository r = gitHub.getRepository("hub4j-test-org/maintain-permission-issue");
1605+
GHPermissionType permission = r.getPermission("alecharp");
1606+
// we should be able to assert on permission but the test crashes
1607+
}
1608+
}
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,141 @@
1+
{
2+
"id": 649600716,
3+
"node_id": "R_kgDOJrgezA",
4+
"name": "maintain-permission-issue",
5+
"full_name": "hub4j-test-org/maintain-permission-issue",
6+
"private": true,
7+
"owner": {
8+
"login": "hub4j-test-org",
9+
"id": 7544739,
10+
"node_id": "MDEyOk9yZ2FuaXphdGlvbjc1NDQ3Mzk=",
11+
"avatar_url": "https://avatars.githubusercontent.com/u/7544739?v=4",
12+
"gravatar_id": "",
13+
"url": "https://api.github.com/users/hub4j-test-org",
14+
"html_url": "https://github.com/hub4j-test-org",
15+
"followers_url": "https://api.github.com/users/hub4j-test-org/followers",
16+
"following_url": "https://api.github.com/users/hub4j-test-org/following{/other_user}",
17+
"gists_url": "https://api.github.com/users/hub4j-test-org/gists{/gist_id}",
18+
"starred_url": "https://api.github.com/users/hub4j-test-org/starred{/owner}{/repo}",
19+
"subscriptions_url": "https://api.github.com/users/hub4j-test-org/subscriptions",
20+
"organizations_url": "https://api.github.com/users/hub4j-test-org/orgs",
21+
"repos_url": "https://api.github.com/users/hub4j-test-org/repos",
22+
"events_url": "https://api.github.com/users/hub4j-test-org/events{/privacy}",
23+
"received_events_url": "https://api.github.com/users/hub4j-test-org/received_events",
24+
"type": "Organization",
25+
"site_admin": false
26+
},
27+
"html_url": "https://github.com/hub4j-test-org/maintain-permission-issue",
28+
"description": "A repository to demo the maintain permission issue",
29+
"fork": false,
30+
"url": "https://api.github.com/repos/hub4j-test-org/maintain-permission-issue",
31+
"forks_url": "https://api.github.com/repos/hub4j-test-org/maintain-permission-issue/forks",
32+
"keys_url": "https://api.github.com/repos/hub4j-test-org/maintain-permission-issue/keys{/key_id}",
33+
"collaborators_url": "https://api.github.com/repos/hub4j-test-org/maintain-permission-issue/collaborators{/collaborator}",
34+
"teams_url": "https://api.github.com/repos/hub4j-test-org/maintain-permission-issue/teams",
35+
"hooks_url": "https://api.github.com/repos/hub4j-test-org/maintain-permission-issue/hooks",
36+
"issue_events_url": "https://api.github.com/repos/hub4j-test-org/maintain-permission-issue/issues/events{/number}",
37+
"events_url": "https://api.github.com/repos/hub4j-test-org/maintain-permission-issue/events",
38+
"assignees_url": "https://api.github.com/repos/hub4j-test-org/maintain-permission-issue/assignees{/user}",
39+
"branches_url": "https://api.github.com/repos/hub4j-test-org/maintain-permission-issue/branches{/branch}",
40+
"tags_url": "https://api.github.com/repos/hub4j-test-org/maintain-permission-issue/tags",
41+
"blobs_url": "https://api.github.com/repos/hub4j-test-org/maintain-permission-issue/git/blobs{/sha}",
42+
"git_tags_url": "https://api.github.com/repos/hub4j-test-org/maintain-permission-issue/git/tags{/sha}",
43+
"git_refs_url": "https://api.github.com/repos/hub4j-test-org/maintain-permission-issue/git/refs{/sha}",
44+
"trees_url": "https://api.github.com/repos/hub4j-test-org/maintain-permission-issue/git/trees{/sha}",
45+
"statuses_url": "https://api.github.com/repos/hub4j-test-org/maintain-permission-issue/statuses/{sha}",
46+
"languages_url": "https://api.github.com/repos/hub4j-test-org/maintain-permission-issue/languages",
47+
"stargazers_url": "https://api.github.com/repos/hub4j-test-org/maintain-permission-issue/stargazers",
48+
"contributors_url": "https://api.github.com/repos/hub4j-test-org/maintain-permission-issue/contributors",
49+
"subscribers_url": "https://api.github.com/repos/hub4j-test-org/maintain-permission-issue/subscribers",
50+
"subscription_url": "https://api.github.com/repos/hub4j-test-org/maintain-permission-issue/subscription",
51+
"commits_url": "https://api.github.com/repos/hub4j-test-org/maintain-permission-issue/commits{/sha}",
52+
"git_commits_url": "https://api.github.com/repos/hub4j-test-org/maintain-permission-issue/git/commits{/sha}",
53+
"comments_url": "https://api.github.com/repos/hub4j-test-org/maintain-permission-issue/comments{/number}",
54+
"issue_comment_url": "https://api.github.com/repos/hub4j-test-org/maintain-permission-issue/issues/comments{/number}",
55+
"contents_url": "https://api.github.com/repos/hub4j-test-org/maintain-permission-issue/contents/{+path}",
56+
"compare_url": "https://api.github.com/repos/hub4j-test-org/maintain-permission-issue/compare/{base}...{head}",
57+
"merges_url": "https://api.github.com/repos/hub4j-test-org/maintain-permission-issue/merges",
58+
"archive_url": "https://api.github.com/repos/hub4j-test-org/maintain-permission-issue/{archive_format}{/ref}",
59+
"downloads_url": "https://api.github.com/repos/hub4j-test-org/maintain-permission-issue/downloads",
60+
"issues_url": "https://api.github.com/repos/hub4j-test-org/maintain-permission-issue/issues{/number}",
61+
"pulls_url": "https://api.github.com/repos/hub4j-test-org/maintain-permission-issue/pulls{/number}",
62+
"milestones_url": "https://api.github.com/repos/hub4j-test-org/maintain-permission-issue/milestones{/number}",
63+
"notifications_url": "https://api.github.com/repos/hub4j-test-org/maintain-permission-issue/notifications{?since,all,participating}",
64+
"labels_url": "https://api.github.com/repos/hub4j-test-org/maintain-permission-issue/labels{/name}",
65+
"releases_url": "https://api.github.com/repos/hub4j-test-org/maintain-permission-issue/releases{/id}",
66+
"deployments_url": "https://api.github.com/repos/hub4j-test-org/maintain-permission-issue/deployments",
67+
"created_at": "2023-06-05T08:29:48Z",
68+
"updated_at": "2023-06-05T08:29:48Z",
69+
"pushed_at": "2023-06-05T08:29:48Z",
70+
"git_url": "git://github.com/hub4j-test-org/maintain-permission-issue.git",
71+
"ssh_url": "git@github.com:hub4j-test-org/maintain-permission-issue.git",
72+
"clone_url": "https://github.com/hub4j-test-org/maintain-permission-issue.git",
73+
"svn_url": "https://github.com/hub4j-test-org/maintain-permission-issue",
74+
"homepage": null,
75+
"size": 0,
76+
"stargazers_count": 0,
77+
"watchers_count": 0,
78+
"language": null,
79+
"has_issues": true,
80+
"has_projects": true,
81+
"has_downloads": true,
82+
"has_wiki": false,
83+
"has_pages": false,
84+
"has_discussions": false,
85+
"forks_count": 0,
86+
"mirror_url": null,
87+
"archived": false,
88+
"disabled": false,
89+
"open_issues_count": 0,
90+
"license": null,
91+
"allow_forking": false,
92+
"is_template": false,
93+
"web_commit_signoff_required": false,
94+
"topics": [],
95+
"visibility": "private",
96+
"forks": 0,
97+
"open_issues": 0,
98+
"watchers": 0,
99+
"default_branch": "main",
100+
"permissions": {
101+
"admin": true,
102+
"maintain": true,
103+
"push": true,
104+
"triage": true,
105+
"pull": true
106+
},
107+
"temp_clone_token": "ACLMQO62GXXZCCUB5ECCP3DEPWSHO",
108+
"allow_squash_merge": true,
109+
"allow_merge_commit": true,
110+
"allow_rebase_merge": true,
111+
"allow_auto_merge": false,
112+
"delete_branch_on_merge": false,
113+
"allow_update_branch": false,
114+
"use_squash_pr_title_as_default": false,
115+
"squash_merge_commit_message": "COMMIT_MESSAGES",
116+
"squash_merge_commit_title": "COMMIT_OR_PR_TITLE",
117+
"merge_commit_message": "PR_TITLE",
118+
"merge_commit_title": "MERGE_MESSAGE",
119+
"organization": {
120+
"login": "hub4j-test-org",
121+
"id": 7544739,
122+
"node_id": "MDEyOk9yZ2FuaXphdGlvbjc1NDQ3Mzk=",
123+
"avatar_url": "https://avatars.githubusercontent.com/u/7544739?v=4",
124+
"gravatar_id": "",
125+
"url": "https://api.github.com/users/hub4j-test-org",
126+
"html_url": "https://github.com/hub4j-test-org",
127+
"followers_url": "https://api.github.com/users/hub4j-test-org/followers",
128+
"following_url": "https://api.github.com/users/hub4j-test-org/following{/other_user}",
129+
"gists_url": "https://api.github.com/users/hub4j-test-org/gists{/gist_id}",
130+
"starred_url": "https://api.github.com/users/hub4j-test-org/starred{/owner}{/repo}",
131+
"subscriptions_url": "https://api.github.com/users/hub4j-test-org/subscriptions",
132+
"organizations_url": "https://api.github.com/users/hub4j-test-org/orgs",
133+
"repos_url": "https://api.github.com/users/hub4j-test-org/repos",
134+
"events_url": "https://api.github.com/users/hub4j-test-org/events{/privacy}",
135+
"received_events_url": "https://api.github.com/users/hub4j-test-org/received_events",
136+
"type": "Organization",
137+
"site_admin": false
138+
},
139+
"network_count": 0,
140+
"subscribers_count": 21
141+
}
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,32 @@
1+
{
2+
"permission": "maintain",
3+
"user": {
4+
"login": "alecharp",
5+
"id": 985955,
6+
"node_id": "MDQ6VXNlcjk4NTk1NQ==",
7+
"avatar_url": "https://avatars.githubusercontent.com/u/985955?v=4",
8+
"gravatar_id": "",
9+
"url": "https://api.github.com/users/alecharp",
10+
"html_url": "https://github.com/alecharp",
11+
"followers_url": "https://api.github.com/users/alecharp/followers",
12+
"following_url": "https://api.github.com/users/alecharp/following{/other_user}",
13+
"gists_url": "https://api.github.com/users/alecharp/gists{/gist_id}",
14+
"starred_url": "https://api.github.com/users/alecharp/starred{/owner}{/repo}",
15+
"subscriptions_url": "https://api.github.com/users/alecharp/subscriptions",
16+
"organizations_url": "https://api.github.com/users/alecharp/orgs",
17+
"repos_url": "https://api.github.com/users/alecharp/repos",
18+
"events_url": "https://api.github.com/users/alecharp/events{/privacy}",
19+
"received_events_url": "https://api.github.com/users/alecharp/received_events",
20+
"type": "User",
21+
"site_admin": false,
22+
"permissions": {
23+
"admin": false,
24+
"maintain": true,
25+
"push": true,
26+
"triage": true,
27+
"pull": true
28+
},
29+
"role_name": "maintain"
30+
},
31+
"role_name": "maintain"
32+
}
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,34 @@
1+
{
2+
"login": "PierreBtz",
3+
"id": 9881659,
4+
"node_id": "MDQ6VXNlcjk4ODE2NTk=",
5+
"avatar_url": "https://avatars.githubusercontent.com/u/9881659?v=4",
6+
"gravatar_id": "",
7+
"url": "https://api.github.com/users/PierreBtz",
8+
"html_url": "https://github.com/PierreBtz",
9+
"followers_url": "https://api.github.com/users/PierreBtz/followers",
10+
"following_url": "https://api.github.com/users/PierreBtz/following{/other_user}",
11+
"gists_url": "https://api.github.com/users/PierreBtz/gists{/gist_id}",
12+
"starred_url": "https://api.github.com/users/PierreBtz/starred{/owner}{/repo}",
13+
"subscriptions_url": "https://api.github.com/users/PierreBtz/subscriptions",
14+
"organizations_url": "https://api.github.com/users/PierreBtz/orgs",
15+
"repos_url": "https://api.github.com/users/PierreBtz/repos",
16+
"events_url": "https://api.github.com/users/PierreBtz/events{/privacy}",
17+
"received_events_url": "https://api.github.com/users/PierreBtz/received_events",
18+
"type": "User",
19+
"site_admin": false,
20+
"name": "Pierre Beitz",
21+
"company": "@cloudbees ",
22+
"blog": "",
23+
"location": null,
24+
"email": "pibeitz@gmail.com",
25+
"hireable": null,
26+
"bio": null,
27+
"twitter_username": null,
28+
"public_repos": 89,
29+
"public_gists": 7,
30+
"followers": 12,
31+
"following": 11,
32+
"created_at": "2014-11-21T10:26:34Z",
33+
"updated_at": "2023-05-31T07:47:04Z"
34+
}
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,51 @@
1+
{
2+
"id": "1c85c1aa-c054-4ee0-88ac-7a1f093140ce",
3+
"name": "repos_hub4j-test-org_maintain-permission-issue",
4+
"request": {
5+
"url": "/repos/hub4j-test-org/maintain-permission-issue",
6+
"method": "GET",
7+
"headers": {
8+
"Accept": {
9+
"equalTo": "application/vnd.github.v3+json"
10+
}
11+
}
12+
},
13+
"response": {
14+
"status": 200,
15+
"bodyFileName": "repos_hub4j-test-org_maintain-permission-issue-2.json",
16+
"headers": {
17+
"Server": "GitHub.com",
18+
"Date": "Mon, 05 Jun 2023 08:56:43 GMT",
19+
"Content-Type": "application/json; charset=utf-8",
20+
"Cache-Control": "private, max-age=60, s-maxage=60",
21+
"Vary": [
22+
"Accept, Authorization, Cookie, X-GitHub-OTP",
23+
"Accept-Encoding, Accept, X-Requested-With"
24+
],
25+
"ETag": "W/\"aff085d3b49db6fbe3ef8f48e6ca4fe314d68abfdbd7b6e866c4122a3eb0e611\"",
26+
"Last-Modified": "Mon, 05 Jun 2023 08:29:48 GMT",
27+
"X-OAuth-Scopes": "admin:org, repo",
28+
"X-Accepted-OAuth-Scopes": "repo",
29+
"github-authentication-token-expiration": "2023-07-05 08:21:35 UTC",
30+
"X-GitHub-Media-Type": "github.v3; format=json",
31+
"x-github-api-version-selected": "2022-11-28",
32+
"X-RateLimit-Limit": "5000",
33+
"X-RateLimit-Remaining": "4953",
34+
"X-RateLimit-Reset": "1685955508",
35+
"X-RateLimit-Used": "47",
36+
"X-RateLimit-Resource": "core",
37+
"Access-Control-Expose-Headers": "ETag, Link, Location, Retry-After, X-GitHub-OTP, X-RateLimit-Limit, X-RateLimit-Remaining, X-RateLimit-Used, X-RateLimit-Resource, X-RateLimit-Reset, X-OAuth-Scopes, X-Accepted-OAuth-Scopes, X-Poll-Interval, X-GitHub-Media-Type, X-GitHub-SSO, X-GitHub-Request-Id, Deprecation, Sunset",
38+
"Access-Control-Allow-Origin": "*",
39+
"Strict-Transport-Security": "max-age=31536000; includeSubdomains; preload",
40+
"X-Frame-Options": "deny",
41+
"X-Content-Type-Options": "nosniff",
42+
"X-XSS-Protection": "0",
43+
"Referrer-Policy": "origin-when-cross-origin, strict-origin-when-cross-origin",
44+
"Content-Security-Policy": "default-src 'none'",
45+
"X-GitHub-Request-Id": "C925:6C90:53F8419:AC45847:647DA34A"
46+
}
47+
},
48+
"uuid": "1c85c1aa-c054-4ee0-88ac-7a1f093140ce",
49+
"persistent": true,
50+
"insertionIndex": 2
51+
}
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,50 @@
1+
{
2+
"id": "003b11c3-6e12-4da6-a439-02095d7d1328",
3+
"name": "repos_hub4j-test-org_maintain-permission-issue_collaborators_alecharp_permission",
4+
"request": {
5+
"url": "/repos/hub4j-test-org/maintain-permission-issue/collaborators/alecharp/permission",
6+
"method": "GET",
7+
"headers": {
8+
"Accept": {
9+
"equalTo": "application/vnd.github.v3+json"
10+
}
11+
}
12+
},
13+
"response": {
14+
"status": 200,
15+
"bodyFileName": "repos_hub4j-test-org_maintain-permission-issue_collaborators_alecharp_permission-3.json",
16+
"headers": {
17+
"Server": "GitHub.com",
18+
"Date": "Mon, 05 Jun 2023 08:56:43 GMT",
19+
"Content-Type": "application/json; charset=utf-8",
20+
"Cache-Control": "private, max-age=60, s-maxage=60",
21+
"Vary": [
22+
"Accept, Authorization, Cookie, X-GitHub-OTP",
23+
"Accept-Encoding, Accept, X-Requested-With"
24+
],
25+
"ETag": "W/\"e2dc20577322b0be94a78538a4d6c166b39c016f02cb4db80cb63ba33bbf8afb\"",
26+
"X-OAuth-Scopes": "admin:org, repo",
27+
"X-Accepted-OAuth-Scopes": "",
28+
"github-authentication-token-expiration": "2023-07-05 08:21:35 UTC",
29+
"X-GitHub-Media-Type": "github.v3; format=json",
30+
"x-github-api-version-selected": "2022-11-28",
31+
"X-RateLimit-Limit": "5000",
32+
"X-RateLimit-Remaining": "4952",
33+
"X-RateLimit-Reset": "1685955508",
34+
"X-RateLimit-Used": "48",
35+
"X-RateLimit-Resource": "core",
36+
"Access-Control-Expose-Headers": "ETag, Link, Location, Retry-After, X-GitHub-OTP, X-RateLimit-Limit, X-RateLimit-Remaining, X-RateLimit-Used, X-RateLimit-Resource, X-RateLimit-Reset, X-OAuth-Scopes, X-Accepted-OAuth-Scopes, X-Poll-Interval, X-GitHub-Media-Type, X-GitHub-SSO, X-GitHub-Request-Id, Deprecation, Sunset",
37+
"Access-Control-Allow-Origin": "*",
38+
"Strict-Transport-Security": "max-age=31536000; includeSubdomains; preload",
39+
"X-Frame-Options": "deny",
40+
"X-Content-Type-Options": "nosniff",
41+
"X-XSS-Protection": "0",
42+
"Referrer-Policy": "origin-when-cross-origin, strict-origin-when-cross-origin",
43+
"Content-Security-Policy": "default-src 'none'",
44+
"X-GitHub-Request-Id": "C926:0FAC:4E05B5D:A05855E:647DA34B"
45+
}
46+
},
47+
"uuid": "003b11c3-6e12-4da6-a439-02095d7d1328",
48+
"persistent": true,
49+
"insertionIndex": 3
50+
}
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,51 @@
1+
{
2+
"id": "3316e123-fb79-4f7c-b41c-4eaa2840d133",
3+
"name": "user",
4+
"request": {
5+
"url": "/user",
6+
"method": "GET",
7+
"headers": {
8+
"Accept": {
9+
"equalTo": "application/vnd.github.v3+json"
10+
}
11+
}
12+
},
13+
"response": {
14+
"status": 200,
15+
"bodyFileName": "user-1.json",
16+
"headers": {
17+
"Server": "GitHub.com",
18+
"Date": "Mon, 05 Jun 2023 08:56:42 GMT",
19+
"Content-Type": "application/json; charset=utf-8",
20+
"Cache-Control": "private, max-age=60, s-maxage=60",
21+
"Vary": [
22+
"Accept, Authorization, Cookie, X-GitHub-OTP",
23+
"Accept-Encoding, Accept, X-Requested-With"
24+
],
25+
"ETag": "W/\"316d532924f9884c6ce327f8dae23b9a39885cb124f317758c5092b9c829fff2\"",
26+
"Last-Modified": "Wed, 31 May 2023 07:47:04 GMT",
27+
"X-OAuth-Scopes": "admin:org, repo",
28+
"X-Accepted-OAuth-Scopes": "",
29+
"github-authentication-token-expiration": "2023-07-05 08:21:35 UTC",
30+
"X-GitHub-Media-Type": "github.v3; format=json",
31+
"x-github-api-version-selected": "2022-11-28",
32+
"X-RateLimit-Limit": "5000",
33+
"X-RateLimit-Remaining": "4955",
34+
"X-RateLimit-Reset": "1685955508",
35+
"X-RateLimit-Used": "45",
36+
"X-RateLimit-Resource": "core",
37+
"Access-Control-Expose-Headers": "ETag, Link, Location, Retry-After, X-GitHub-OTP, X-RateLimit-Limit, X-RateLimit-Remaining, X-RateLimit-Used, X-RateLimit-Resource, X-RateLimit-Reset, X-OAuth-Scopes, X-Accepted-OAuth-Scopes, X-Poll-Interval, X-GitHub-Media-Type, X-GitHub-SSO, X-GitHub-Request-Id, Deprecation, Sunset",
38+
"Access-Control-Allow-Origin": "*",
39+
"Strict-Transport-Security": "max-age=31536000; includeSubdomains; preload",
40+
"X-Frame-Options": "deny",
41+
"X-Content-Type-Options": "nosniff",
42+
"X-XSS-Protection": "0",
43+
"Referrer-Policy": "origin-when-cross-origin, strict-origin-when-cross-origin",
44+
"Content-Security-Policy": "default-src 'none'",
45+
"X-GitHub-Request-Id": "C923:9715:4F96DE0:A3824FD:647DA349"
46+
}
47+
},
48+
"uuid": "3316e123-fb79-4f7c-b41c-4eaa2840d133",
49+
"persistent": true,
50+
"insertionIndex": 1
51+
}

0 commit comments

Comments
0 (0)
Morty Proxy This is a proxified and sanitized view of the page, visit original site.