Skip to content

Update action.yml to remove initial auth check - #46

Open
tyndyll wants to merge 1 commit into
mainfrom
tyndyll-patch-1
Open

Update action.yml to remove initial auth check#46
tyndyll wants to merge 1 commit into
mainfrom
tyndyll-patch-1

Conversation

@tyndyll

@tyndyll tyndyll commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@upwind-code-us

upwind-code-us Bot commented Aug 10, 2026

Copy link
Copy Markdown

Upwind Upwind Code Scan - ✅ Proceed with Deployment

0 newly introduced vulnerabilities · 0 resolved · 0 total in this PR vs main

View full analysis in Upwind Console

Scan completed in 14s

Scan history (1 scan)
Commit Scanned at New Resolved Net
6edee36 < 2026-08-10 07:04 UTC 0 0 0

Last scanned: 6edee36 · 2026-08-10 07:04 UTC

@upwind-code-us

upwind-code-us Bot commented Aug 10, 2026

Copy link
Copy Markdown

Upwind Upwind IaC Scan - ✅ Proceed with Deployment

0 newly introduced misconfigurations · 0 resolved · 0 total in this PR vs main

View full analysis in Upwind Console →

Scan completed in 2s

Scan history (1 scan)
Commit Scanned at New Resolved Net
6edee36 < 2026-08-10 07:04 UTC 0 0 0

Last scanned: 6edee36 · 2026-08-10 07:04 UTC

Comment thread action.yml
echo "No access_token found in response: $response" >&2
exit 1
fi

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

with this block removed, is anything still setting $TOKEN? Line 148 below still sends Authorization: Bearer $TOKEN when downloading the agent binary, so unless the download endpoint no longer requires auth, this will now send an empty token and fail. If the endpoint is public now, should we drop the header too?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

The endpoint doesn't require auth at all. It can be removed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants