- Notifications
You must be signed in to change notification settings - Fork1.1k
feat: check agent API version on connection#11696
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Merged
Merged
Uh oh!
There was an error while loading.Please reload this page.
Conversation
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
This was referencedJan 19, 2024
This was referencedJan 19, 2024
ContributorAuthor
spikecurtis commentedJan 19, 2024 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
This was referencedJan 19, 2024
c7f73d4 to135bd48Compare3469608 tof56be8eCompare135bd48 toecef2a9Comparef56be8e to24a943cCompareecef2a9 to0b0b1a4Compare24a943c to16322d1Comparedeansheather approved these changesJan 22, 2024
0b0b1a4 to7a301f1Compare16322d1 toc91b1c0Compare7a301f1 to0e38400Comparec91b1c0 to671a2e9Compare0e38400 to35d99a8Compare671a2e9 to5fa6879Compare35d99a8 to66da7c6Compare5fa6879 to8ad2024Compare66da7c6 to62e95deCompare8ad2024 tod7a1794Compare62e95de to5695841Compared7a1794 to59e8c2fCompare59e8c2f to8cbda21Compare This was referencedJan 23, 2024
ContributorAuthor
spikecurtis commentedJan 23, 2024
Merge activity
|
Sign up for freeto subscribe to this conversation on GitHub. Already have an account?Sign in.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

Uh oh!
There was an error while loading.Please reload this page.
fixes#10531
Adds a check for
versionon connection to the Agent API websocket endpoint. This is primarily for future-proofing, so that up-level agents get a sensible error if they connect to a back-level Coderd.It also refactors the location of the
CurrentVersionvariables, to be part of theprotopackages, since the versions refer to the APIs defined therein.