josh/Catalyst
1
0
Fork 0
Code Issues 2 Pull Requests Actions Packages Projects Releases 16 Wiki Activity

fix: base64-encode Patchmon Basic auth credentials server-side #75

Closed
josh wants to merge 0 commits from dev into main
pull from: dev
merge into: josh:main
Conversation 0 Commits - Files Changed -
+11 -3
josh commented 2026-05-30 18:59:23 -04:00
Owner
Copy Link
Copy Source
No description provided.
josh added 3 commits 2026-05-30 18:59:23 -04:00
fix: base64-encode Patchmon Basic auth credentials server-side efa1750cac 
Patchmon's API uses standard RFC 7617 Basic auth — `Basic base64(token_key:token_secret)`. The handler was sending the api_token field verbatim, so it only worked if the user had manually base64-encoded the credential. After a Patchmon upgrade, the sync started returning HTML (the SPA, served when auth is rejected) and failing with "Unexpected token '<'" on JSON.parse.

Now: if the token contains ':' (raw key:secret), encode it server-side; otherwise pass through unchanged for backward compatibility. UI gets a placeholder hint showing the expected format.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
chore: bump to version 1.7.1
CI / test (pull_request) Successful in 19s
Details
CI / build-dev (pull_request) Has been skipped
Details
ac71ea2c49 
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Merge pull request 'fix: base64-encode Patchmon Basic auth credentials server-side' (#74) from fix/patchmon-sync-basic-auth into dev
CI / test (push) Successful in 9s
Details
CI / build-dev (push) Successful in 23s
Details
CI / test (pull_request) Successful in 18s
Details
CI / build-dev (pull_request) Has been skipped
Details
a28867b398 
Reviewed-on: #74
josh closed this pull request 2026-05-30 19:01:21 -04:00
All checks were successful
CI / test (push) Successful in 9s
Details
CI / build-dev (push) Successful in 23s
Details
CI / test (pull_request) Successful in 18s
Required
Details
CI / build-dev (pull_request) Has been skipped
Details

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
josh (Josh Wright)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: josh/Catalyst#75
Delete Branch "dev"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?