Skip to content

Commit babaff4

Browse files
maboloshimaboloshi
authored
feat(outputs): app-slug and installation-id (#105)
It is convenient to use `https://api.github.com/users/$app_slug[bot]` to obtain the corresponding account ID later. Then build `Signed-off-by: $app_slug[bot] <$id+$app_slug[bot]@users.noreply.github.com>`. Currently, there is no Linux environment to build test snapshot files
1 parent bf627a5 commit babaff4

13 files changed

+98
-28
lines changed

‎README.md

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -239,6 +239,14 @@ jobs:
239239

240240
GitHub App installation access token.
241241

242+
### `installation-id`
243+
244+
GitHub App installation ID.
245+
246+
### `app-slug`
247+
248+
GitHub App slug.
249+
242250
## How it works
243251

244252
The action creates an installation access token using [the `POST /app/installations/{installation_id}/access_tokens` endpoint](https://docs.github.com/rest/apps/apps?apiVersion=2022-11-28#create-an-installation-access-token-for-an-app). By default,

‎action.yml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -40,6 +40,10 @@ inputs:
4040
outputs:
4141
token:
4242
description: "GitHub installation access token"
43+
installation-id:
44+
description: "GitHub App installation ID"
45+
app-slug:
46+
description: "GitHub App slug"
4347
runs:
4448
using: "node20"
4549
main: "dist/main.cjs"

‎dist/main.cjs

Lines changed: 13 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -29923,28 +29923,30 @@ async function main(appId2, privateKey2, owner2, repositories2, core3, createApp
2992329923
privateKey: privateKey2,
2992429924
request: request2
2992529925
});
29926-
let authentication;
29926+
let authentication, installationId, appSlug;
2992729927
if (parsedRepositoryNames) {
29928-
authentication = await pRetry(() => getTokenFromRepository(request2, auth, parsedOwner, parsedRepositoryNames), {
29928+
({ authentication, installationId, appSlug } = await pRetry(() => getTokenFromRepository(request2, auth, parsedOwner, parsedRepositoryNames), {
2992929929
onFailedAttempt: (error) => {
2993029930
core3.info(
2993129931
`Failed to create token for "${parsedRepositoryNames}" (attempt ${error.attemptNumber}): ${error.message}`
2993229932
);
2993329933
},
2993429934
retries: 3
29935-
});
29935+
}));
2993629936
} else {
29937-
authentication = await pRetry(() => getTokenFromOwner(request2, auth, parsedOwner), {
29937+
({ authentication, installationId, appSlug } = await pRetry(() => getTokenFromOwner(request2, auth, parsedOwner), {
2993829938
onFailedAttempt: (error) => {
2993929939
core3.info(
2994029940
`Failed to create token for "${parsedOwner}" (attempt ${error.attemptNumber}): ${error.message}`
2994129941
);
2994229942
},
2994329943
retries: 3
29944-
});
29944+
}));
2994529945
}
2994629946
core3.setSecret(authentication.token);
2994729947
core3.setOutput("token", authentication.token);
29948+
core3.setOutput("installation-id", installationId);
29949+
core3.setOutput("app-slug", appSlug);
2994829950
if (!skipTokenRevoke2) {
2994929951
core3.saveState("token", authentication.token);
2995029952
core3.setOutput("expiresAt", authentication.expiresAt);
@@ -29970,7 +29972,9 @@ async function getTokenFromOwner(request2, auth, parsedOwner) {
2997029972
type: "installation",
2997129973
installationId: response.data.id
2997229974
});
29973-
return authentication;
29975+
const installationId = response.data.id;
29976+
const appSlug = response.data["app_slug"];
29977+
return { authentication, installationId, appSlug };
2997429978
}
2997529979
async function getTokenFromRepository(request2, auth, parsedOwner, parsedRepositoryNames) {
2997629980
const response = await request2("GET /repos/{owner}/{repo}/installation", {
@@ -29985,7 +29989,9 @@ async function getTokenFromRepository(request2, auth, parsedOwner, parsedReposit
2998529989
installationId: response.data.id,
2998629990
repositoryNames: parsedRepositoryNames.split(",")
2998729991
});
29988-
return authentication;
29992+
const installationId = response.data.id;
29993+
const appSlug = response.data["app_slug"];
29994+
return { authentication, installationId, appSlug };
2998929995
}
2999029996

2999129997
// lib/request.js

‎lib/main.js

Lines changed: 17 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -70,35 +70,36 @@ export async function main(
7070
request,
7171
});
7272

73-
let authentication;
73+
let authentication, installationId, appSlug;
7474
// If at least one repository is set, get installation ID from that repository
7575

7676
if (parsedRepositoryNames) {
77-
authentication = await pRetry(() => getTokenFromRepository(request, auth, parsedOwner, parsedRepositoryNames), {
77+
({ authentication, installationId, appSlug } = await pRetry(() => getTokenFromRepository(request, auth, parsedOwner, parsedRepositoryNames), {
7878
onFailedAttempt: (error) => {
7979
core.info(
8080
`Failed to create token for "${parsedRepositoryNames}" (attempt ${error.attemptNumber}): ${error.message}`
8181
);
8282
},
8383
retries: 3,
84-
});
85-
84+
}));
8685
} else {
8786
// Otherwise get the installation for the owner, which can either be an organization or a user account
88-
authentication = await pRetry(() => getTokenFromOwner(request, auth, parsedOwner), {
87+
({ authentication, installationId, appSlug } = await pRetry(() => getTokenFromOwner(request, auth, parsedOwner), {
8988
onFailedAttempt: (error) => {
9089
core.info(
9190
`Failed to create token for "${parsedOwner}" (attempt ${error.attemptNumber}): ${error.message}`
9291
);
9392
},
9493
retries: 3,
95-
});
94+
}));
9695
}
9796

9897
// Register the token with the runner as a secret to ensure it is masked in logs
9998
core.setSecret(authentication.token);
10099

101100
core.setOutput("token", authentication.token);
101+
core.setOutput("installation-id", installationId);
102+
core.setOutput("app-slug", appSlug);
102103

103104
// Make token accessible to post function (so we can invalidate it)
104105
if (!skipTokenRevoke) {
@@ -132,7 +133,11 @@ async function getTokenFromOwner(request, auth, parsedOwner) {
132133
type: "installation",
133134
installationId: response.data.id,
134135
});
135-
return authentication;
136+
137+
const installationId = response.data.id;
138+
const appSlug = response.data['app_slug'];
139+
140+
return { authentication, installationId, appSlug };
136141
}
137142

138143
async function getTokenFromRepository(request, auth, parsedOwner, parsedRepositoryNames) {
@@ -152,5 +157,8 @@ async function getTokenFromRepository(request, auth, parsedOwner, parsedReposito
152157
repositoryNames: parsedRepositoryNames.split(","),
153158
});
154159

155-
return authentication;
156-
}
160+
const installationId = response.data.id;
161+
const appSlug = response.data['app_slug'];
162+
163+
return { authentication, installationId, appSlug };
164+
}

‎tests/main-repo-skew.js

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@ await test((mockPool) => {
99
const owner = process.env.INPUT_OWNER
1010
const repo = process.env.INPUT_REPOSITORIES
1111
const mockInstallationId = "123456";
12+
const mockAppSlug = "github-actions";
1213

1314
install({ now: 0, toFake: ["Date"] });
1415

@@ -44,7 +45,8 @@ await test((mockPool) => {
4445
return {
4546
statusCode: 200,
4647
data: {
47-
id: mockInstallationId
48+
id: mockInstallationId,
49+
"app_slug": mockAppSlug
4850
},
4951
responseOptions: {
5052
headers: {

‎tests/main-token-get-owner-set-repo-fail-response.test.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@ await test((mockPool) => {
77
const owner = process.env.INPUT_OWNER;
88
const repo = process.env.INPUT_REPOSITORIES;
99
const mockInstallationId = "123456";
10+
const mockAppSlug = "github-actions";
1011

1112
mockPool
1213
.intercept({
@@ -32,7 +33,7 @@ await test((mockPool) => {
3233
})
3334
.reply(
3435
200,
35-
{ id: mockInstallationId },
36+
{ id: mockInstallationId, "app_slug": mockAppSlug },
3637
{ headers: { "content-type": "application/json" } }
3738
);
3839
});

‎tests/main-token-get-owner-set-to-org-repo-unset.test.js

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,8 +5,9 @@ await test((mockPool) => {
55
process.env.INPUT_OWNER = process.env.GITHUB_REPOSITORY_OWNER;
66
delete process.env.INPUT_REPOSITORIES;
77

8-
// Mock installation id request
8+
// Mock installation id and app slug request
99
const mockInstallationId = "123456";
10+
const mockAppSlug = "github-actions";
1011
mockPool
1112
.intercept({
1213
path: `/orgs/${process.env.INPUT_OWNER}/installation`,
@@ -19,7 +20,7 @@ await test((mockPool) => {
1920
})
2021
.reply(
2122
200,
22-
{ id: mockInstallationId },
23+
{ id: mockInstallationId, "app_slug": mockAppSlug },
2324
{ headers: { "content-type": "application/json" } }
2425
);
2526
});

‎tests/main-token-get-owner-set-to-user-fail-response.test.js

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,8 +5,9 @@ await test((mockPool) => {
55
process.env.INPUT_OWNER = "smockle";
66
delete process.env.INPUT_REPOSITORIES;
77

8-
// Mock installation id request
8+
// Mock installation ID and app slug request
99
const mockInstallationId = "123456";
10+
const mockAppSlug = "github-actions";
1011
mockPool
1112
.intercept({
1213
path: `/orgs/${process.env.INPUT_OWNER}/installation`,
@@ -30,7 +31,7 @@ await test((mockPool) => {
3031
})
3132
.reply(
3233
200,
33-
{ id: mockInstallationId },
34+
{ id: mockInstallationId, "app_slug": mockAppSlug },
3435
{ headers: { "content-type": "application/json" } }
3536
);
3637
});

‎tests/main-token-get-owner-set-to-user-repo-unset.test.js

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,8 +5,9 @@ await test((mockPool) => {
55
process.env.INPUT_OWNER = "smockle";
66
delete process.env.INPUT_REPOSITORIES;
77

8-
// Mock installation id request
8+
// Mock installation ID and app slug request
99
const mockInstallationId = "123456";
10+
const mockAppSlug = "github-actions";
1011
mockPool
1112
.intercept({
1213
path: `/orgs/${process.env.INPUT_OWNER}/installation`,
@@ -30,7 +31,7 @@ await test((mockPool) => {
3031
})
3132
.reply(
3233
200,
33-
{ id: mockInstallationId },
34+
{ id: mockInstallationId, "app_slug": mockAppSlug },
3435
{ headers: { "content-type": "application/json" } }
3536
);
3637
});

‎tests/main-token-get-owner-unset-repo-unset.test.js

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,8 +5,9 @@ await test((mockPool) => {
55
delete process.env.INPUT_OWNER;
66
delete process.env.INPUT_REPOSITORIES;
77

8-
// Mock installation id request
8+
// Mock installation ID and app slug request
99
const mockInstallationId = "123456";
10+
const mockAppSlug = "github-actions";
1011
mockPool
1112
.intercept({
1213
path: `/repos/${process.env.GITHUB_REPOSITORY}/installation`,
@@ -19,7 +20,7 @@ await test((mockPool) => {
1920
})
2021
.reply(
2122
200,
22-
{ id: mockInstallationId },
23+
{ id: mockInstallationId, "app_slug": mockAppSlug },
2324
{ headers: { "content-type": "application/json" } }
2425
);
2526
});

0 commit comments

Comments
 (0)