Skip to content

fix settings session api client#1043

Open
Tomeshwari-02 wants to merge 1 commit into
souma9830:masterfrom
Tomeshwari-02:codex/fix-settings-api-client
Open

fix settings session api client#1043
Tomeshwari-02 wants to merge 1 commit into
souma9830:masterfrom
Tomeshwari-02:codex/fix-settings-api-client

Conversation

@Tomeshwari-02

Copy link
Copy Markdown

Summary

  • replace hardcoded relative session fetch calls with the shared API client
  • preserve authenticated cookie behavior for session list and revoke calls
  • keep error handling in the settings page UI

Why

Deployments that configure VITE_API_URL for a separate backend origin should not have SettingsPage session requests sent to the frontend origin.

Fixes #1033

Validation

  • npm run build from frontend

@vercel

vercel Bot commented Jul 1, 2026

Copy link
Copy Markdown

Someone is attempting to deploy a commit to the souma9830's projects Team on Vercel.

A member of the Team first needs to authorize it.

@Tomeshwari-02 Tomeshwari-02 changed the title [codex] fix settings session api client fix settings session api client Jul 1, 2026
@Tomeshwari-02 Tomeshwari-02 marked this pull request as ready for review July 1, 2026 17:37
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.

bug: SettingsPage bypasses configured API base URL for session endpoints

1 participant