Skip to content

fix: scope the OBS token cache to each LINE login - #265

Merged
Fizzadar merged 1 commit into
mainfrom
nick/per-client-obs-token
Oct 13, 2026
Merged

Fizzadar merged 1 commit into
mainfrom
nick/per-client-obs-token

Conversation

@Fizzadar

Copy link
Copy Markdown
Contributor

The OBS access token cache was a package-level global, so every LINE login in a process sent whichever login's OBS token was cached last as x-line-access, and any login's re-authentication cleared it for all of them. This moves the cache onto LineClient and attaches it to every line.Client that login creates, so caching and invalidation are per login.

🤖 Generated with Claude Code

The OBS access token cache was a package-level global, so every LINE
login in the process sent whichever login's OBS token was cached last as
x-line-access, and any login's re-authentication cleared it for all.

Move the cache onto LineClient and attach it to every line.Client that
login creates, so caching and invalidation are per login.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@Fizzadar
Fizzadar requested a review from highesttt October 13, 2026 11:53
@Fizzadar
Fizzadar marked this pull request as ready for review October 13, 2026 11:53
@indent

indent Bot commented Oct 13, 2026 •

Copy link
Copy Markdown

Issues

No issues found.

CI Checks

All CI checks passed on 3b9e5c5.

Review agents

Select any unchecked box below to run or rerun that agent.

Passed (1)
  • Indent Review Agent · Every OBS path now uses its login's cache, and I found no regressions or nil-cache risks.
Full results

Indent Review Agent

  • Summary: Every OBS path now uses its login's cache, and I found no regressions or nil-cache risks.
  • Last ran on commit: 3b9e5c50
  • Latest result
    {
      "summary": "Every OBS path now uses its login's cache, and I found no regressions or nil-cache risks.",
      "findings": []
    }

@Fizzadar
Fizzadar merged commit b81acdd into main Oct 13, 2026
9 checks passed
@Fizzadar
Fizzadar deleted the nick/per-client-obs-token branch October 13, 2026 14:55
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