-
Notifications
You must be signed in to change notification settings - Fork 69
Comparing changes
Open a pull request
base repository: sourcegraph/src-cli
base: main
head repository: sourcegraph/src-cli
compare: release/6.12
- 15 commits
- 32 files changed
- 6 contributors
Commits on Mar 23, 2026
-
feat(oauth): use oauth flow to authenticate with predefined src-cli O…
…Auth client (#1223) * removed unused func * add refresh token to device response unmarshall * make NewClient take ClientID as param * add oauth flow and use oauth token when SRC_ACCESS_TOKEN is empty * feat(oauth): Add refresh to oauthdevice.Client (#1227) * add refresh to oauthdevice.Client * oauthdevice: add RefreshToken field and Refresh method * feat(oauth): Use keyring to store oauth token (#1228) * add refresh to oauthdevice.Client * add OAuth Transport and use it if no access token * secrets: switch to zalando/go-keyring and add context support * secrets: scope keyring by endpoint (cherry picked from commit 0bc535e)
Configuration menu - View commit details
-
Copy full SHA for bcfd2a2 - Browse repository at this point
Copy the full SHA bcfd2a2View commit details -
oauth: fix oauth token race in http_transport (#1269)
(cherry picked from commit 8711473)
Configuration menu - View commit details
-
Copy full SHA for c7f00ad - Browse repository at this point
Copy the full SHA c7f00adView commit details -
- split login into distinct flows - add specific flow for oauth - add specific flows for error conditions - update tests (cherry picked from commit c6de254)
Configuration menu - View commit details
-
Copy full SHA for df649a6 - Browse repository at this point
Copy the full SHA df649a6View commit details -
refactor: refreshToken -> getToken (#1270)
- fold getToken and refreshToken into one so that we reduce locking (cherry picked from commit b5a0552)
Configuration menu - View commit details
-
Copy full SHA for caf1b3e - Browse repository at this point
Copy the full SHA caf1b3eView commit details -
feat: dynamically start oauth flow if no SRC_ACCESS_TOKEN (#1271)
* add AuthMode to config - AuthModeAccessToken and AuthModeOAuth - And check for CI and required AuthModeAccessToken * decide loginFlow based on config AuthMode * remove oauth flag * check for OAuth token precense before starting a new OAuth flow * remove unused ctx (cherry picked from commit 4c3f356)
Configuration menu - View commit details
-
Copy full SHA for de1fb40 - Browse repository at this point
Copy the full SHA de1fb40View commit details -
fix/security: prevent command injection on windows (#1272)
(cherry picked from commit 22191a7)
Configuration menu - View commit details
-
Copy full SHA for 841bad0 - Browse repository at this point
Copy the full SHA 841bad0View commit details -
feat/auth: add
src auth token(#1275)* add TokenRefresher and NewTransport - TokenRefresher centralizes refreshing of OAuth tokens - NewTransport creates an OAuth transport while making sure it is initialized with a TokenRefresher * add cmd `src auth token` that prints access token or oauth token * set refresh window to 5 min (cherry picked from commit d206288)
Configuration menu - View commit details
-
Copy full SHA for e0576a7 - Browse repository at this point
Copy the full SHA e0576a7View commit details -
Configuration menu - View commit details
-
Copy full SHA for 12c8bcf - Browse repository at this point
Copy the full SHA 12c8bcfView commit details
Commits on Mar 26, 2026
-
bug: only require CI=true and AccessToken when making API requests (#…
…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)
Configuration menu - View commit details
-
Copy full SHA for 5175c14 - Browse repository at this point
Copy the full SHA 5175c14View commit details
Commits on Apr 10, 2026
-
fix/users: remove query for usageStatistics (#1264)
This field was deprecated in March last year and was removed November last year. Before this change any user subcommand just failed. Will be adding some integration smoke tests to catch these sort of issues. Additionally found one place we didn't set Content-Type which Sourcegraph also recently started enforcing.
Configuration menu - View commit details
-
Copy full SHA for e3e0140 - Browse repository at this point
Copy the full SHA e3e0140View commit details -
Configuration menu - View commit details
-
Copy full SHA for 05c8c47 - Browse repository at this point
Copy the full SHA 05c8c47View commit details -
Configuration menu - View commit details
-
Copy full SHA for 84430c9 - Browse repository at this point
Copy the full SHA 84430c9View commit details -
Configuration menu - View commit details
-
Copy full SHA for e2697a7 - Browse repository at this point
Copy the full SHA e2697a7View commit details -
Configuration menu - View commit details
-
Copy full SHA for b791337 - Browse repository at this point
Copy the full SHA b791337View commit details -
fix/repo-list: print partial data when there are graphql errors (#1282)
* update GraphQLErrors to have constructor + handle paths * add test for repo list with some repos failing but not all * update src repo list to handle graphql errors - errors are treated as warnings when getting partial data - if we have no data and just errors that is a fatal error * add comment * Handle partial repo list warnings
Configuration menu - View commit details
-
Copy full SHA for bdbe215 - Browse repository at this point
Copy the full SHA bdbe215View commit details
This comparison is taking too long to generate.
Unfortunately it looks like we can’t render this comparison for you right now. It might be too big, or there might be something weird with your repository.
You can try running this command locally to see the comparison on your machine:
git diff main...release/6.12