Skip to content

Use per-user callback files for browser auth handoff - #637

Open
jdpipe wants to merge 2 commits into
yuezk:mainfrom
jdpipe:main
Open

Use per-user callback files for browser auth handoff#637
jdpipe wants to merge 2 commits into
yuezk:mainfrom
jdpipe:main

Conversation

@jdpipe

@jdpipe jdpipe commented Jul 10, 2026

Copy link
Copy Markdown

The browser-auth callback port and callback log were previously written to fixed filenames under /tmp. On shared machines, a stale file owned by one user could prevent another user from completing authentication after a crash or restart.

Store these files in XDG_RUNTIME_DIR when available, falling back to user-scoped filenames in the system temp directory. This keeps the callback handoff scoped to the interactive user session instead of sharing one global /tmp pathname across users.

jdpipe added 2 commits July 10, 2026 10:30
Prefer XDG_RUNTIME_DIR for GlobalProtect callback port/log files, with
user-scoped temp file fallbacks to avoid collisions across sessions.
Probe OpenConnect's generated pkg-config private dependencies so the Rust
link step follows the TLS backend selected by OpenConnect configure.
@yuezk

yuezk commented Jul 10, 2026

Copy link
Copy Markdown
Owner

Please remove the unrelated crates/openconnect/build.rs changes from this PR. We want to keep the existing explicit library linking; this PR should only address the gpcallback port/log-file paths.

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