Skip to content

fix Quick Connect authentication failing due to missing auth token#81

Merged
RadicalMuffinMan merged 1 commit into
Moonfin-Client:masterfrom
harleykradovill:master
May 13, 2026
Merged

fix Quick Connect authentication failing due to missing auth token#81
RadicalMuffinMan merged 1 commit into
Moonfin-Client:masterfrom
harleykradovill:master

Conversation

@harleykradovill
Copy link
Copy Markdown

Pull Request

Summary

Fix Quick Connect from failing to authenticate. It was successfully creating an auth token but subsequent API calls were failing because the token wasn't available in the runtime context.

Type of Change

  • Bug fix
  • New feature
  • Refactor
  • Performance improvement
  • UI/UX update
  • Documentation update
  • Build/CI change
  • Other (describe):

Changes Made

List the key changes included in this PR.

  • Replace getJson() with postJson() for the initiating a new quick connect request. This was taken from the upstream repository.
  • Set auth token in the runtime context after AuthenticateViaQuickConnect() succeeds, and before subsequent API calls are made.

Testing

  • Tested on physical Roku device
  • Tested via sideload
  • Manual testing completed
  • Not tested (explain why):

Checklist

  • Code builds successfully
  • Code follows project style and conventions
  • No unnecessary commented-out code
  • No new warnings introduced

@RadicalMuffinMan RadicalMuffinMan merged commit e3fada9 into Moonfin-Client:master May 13, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants