Skip to content

Handling a change in the payload URL from CircleCI app #93

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
larsoner merged 31 commits into from
Mar 2, 2026
Merged

Handling a change in the payload URL from CircleCI app #93

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
31 commits
Select commit Hold shift + click to select a range
ae86d8a
Removed header with credential info from step 1 fetch
DavidT3 Sep 16, 2025
d36620b
Merge pull request #1 from DavidT3/bug/lengthOfUndefinedError
DavidT3 Sep 16, 2025
e2731ef
Update index.js
DavidT3 Sep 16, 2025
c71cf4e
[autofix.ci] apply automated fixes
autofix-ci[bot] Sep 16, 2025
ce96682
Merge pull request #2 from DavidT3/bug/lengthOfUndefinedError
DavidT3 Sep 16, 2025
8706d70
Merge branch 'scientific-python:master' into master
DavidT3 Jan 15, 2026
2709c80
Made alterations to index.js that should make this action compatible …
Jan 15, 2026
f776e17
Messed up by defining const job within an if statement, which wouldn'…
Jan 15, 2026
bb12511
Made a possible debug message use CircleCI job names without prepende…
Jan 15, 2026
affc378
[autofix.ci] apply automated fixes
autofix-ci[bot] Jan 15, 2026
95d11cb
Adding debug messages to my additions - it isn't working right yet.
Jan 15, 2026
3f07a8c
[autofix.ci] apply automated fixes
autofix-ci[bot] Jan 15, 2026
c99edde
Merge pull request #3 from DavidT3/bug/multipleJobWorkflow
DavidT3 Jan 15, 2026
9bd5d15
Merge branch 'scientific-python:master' into master
DavidT3 Feb 23, 2026
953c4bf
Log entire payload for debugging purposes
DavidT3 Feb 23, 2026
0a161bd
[autofix.ci] apply automated fixes
autofix-ci[bot] Feb 23, 2026
8ad0663
Merge pull request #4 from DavidT3/info/showAllPayload
DavidT3 Feb 23, 2026
a2eaf89
Change payload logging to show only keys
DavidT3 Feb 23, 2026
553c31b
Merge pull request #5 from DavidT3/debug/logPayloadKeys
DavidT3 Feb 23, 2026
e355d9b
Enhance comments for payload state handling
DavidT3 Feb 23, 2026
9892f83
[autofix.ci] apply automated fixes
autofix-ci[bot] Feb 23, 2026
04080e0
Merge pull request #6 from DavidT3/debug/logPayloadKeys
DavidT3 Feb 23, 2026
83e7540
Altered the check for 'new format' URLs, as I think the standard has …
Feb 23, 2026
709e09d
[autofix.ci] apply automated fixes
autofix-ci[bot] Feb 23, 2026
7d4f07a
Merge pull request #7 from DavidT3/bug/urlFormatChange
DavidT3 Feb 23, 2026
1655322
Added comment to detail the other type of GitHub app URL.
Feb 23, 2026
23621ed
[autofix.ci] apply automated fixes
autofix-ci[bot] Feb 23, 2026
be58a0c
Merge pull request #8 from DavidT3/bug/urlFormatChange
DavidT3 Feb 23, 2026
97682ea
Update index.js
larsoner Mar 2, 2026
f067cbe
Merge branch 'master' into master
larsoner Mar 2, 2026
72eb09b
[autofix.ci] apply automated fixes
autofix-ci[bot] Mar 2, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 6 additions & 1 deletion dist/index.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -35852,6 +35852,9 @@ async function run() {
core.debug(`Ignoring context: ${payload.context}`)
return
}

// Read out 'state' (whether CircleCI process was successful or not), then
// store in debug output along with the target_url
const state = payload.state
core.debug(`context: ${payload.context}`)
core.debug(`state: ${state}`)
Expand All @@ -35861,9 +35864,11 @@ async function run() {
// Set the new status
let artifacts_url = '';
const target = payload.target_url.split('?')[0]; // strip any ?utm=…
if (target.includes('/pipelines/circleci/')) {
if (target.includes('/pipelines/circleci/') || target.includes('app.circleci.com/workflow/')) {
Copy link
Copy Markdown
Collaborator

@larsoner larsoner Mar 2, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
if (target.includes('/pipelines/circleci/') || target.includes('app.circleci.com/workflow/')) {
if (target.includes('/pipelines/circleci/') || target.includes('app.circleci.com/workflow/')) {

// ───── New GitHub‑App URL ───────────────────────────────────────────
// .../pipelines/circleci/<org‑id>/<project‑id>/<pipe‑seq>/workflows/<workflow‑id>
// OR
// .../workflow/<workflow-id>
const workflowId = target.split('/').pop();
core.debug(`workflow: ${workflowId}`);

Expand Down
7 changes: 6 additions & 1 deletion index.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,9 @@ async function run() {
core.debug(`Ignoring context: ${payload.context}`)
return
}

// Read out 'state' (whether CircleCI process was successful or not), then
// store in debug output along with the target_url
const state = payload.state
core.debug(`context: ${payload.context}`)
core.debug(`state: ${state}`)
Expand All @@ -44,9 +47,11 @@ async function run() {
// Set the new status
let artifacts_url = '';
const target = payload.target_url.split('?')[0]; // strip any ?utm=…
if (target.includes('/pipelines/circleci/')) {
if (target.includes('/pipelines/circleci/') || target.includes('app.circleci.com/workflow/')) {
// ───── New GitHub‑App URL ───────────────────────────────────────────
// .../pipelines/circleci/<org‑id>/<project‑id>/<pipe‑seq>/workflows/<workflow‑id>
// OR
// .../workflow/<workflow-id>
const workflowId = target.split('/').pop();
core.debug(`workflow: ${workflowId}`);

Expand Down