bug: only require CI=true and AccessToken when making API requests by burmudar · Pull Request #1278 · sourcegraph/src-cli
added 5 commits
March 25, 2026 09:58- rename `RequireAccessToken` to `checkIfCIAccessTokenRequired` - provide more context in error on why it happened
burmudar
deleted the
wb/bug-src-token-and-ci
branch
burmudar added a commit that referenced this pull request
Mar 26, 2026…1278) * check when inCI and whether we required Access Token * fast fail in APIClient when CI=true and no Access Token * use apiClient constructor in search_jobs * fail fast in login / auth token when CI=true and AccessToken not set * add comment * review feedback - rename `RequireAccessToken` to `checkIfCIAccessTokenRequired` - provide more context in error on why it happened * use sentinel error * use lib errors (cherry picked from commit 0e5e006)
burmudar added a commit that referenced this pull request
Mar 26, 2026…1278) * check when inCI and whether we required Access Token * fast fail in APIClient when CI=true and no Access Token * use apiClient constructor in search_jobs * fail fast in login / auth token when CI=true and AccessToken not set * add comment * review feedback - rename `RequireAccessToken` to `checkIfCIAccessTokenRequired` - provide more context in error on why it happened * use sentinel error * use lib errors (cherry picked from commit 0e5e006)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters