- Notifications
You must be signed in to change notification settings - Fork914
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
to135bd48
Compare3469608
tof56be8e
Compare135bd48
toecef2a9
Comparef56be8e
to24a943c
Compareecef2a9
to0b0b1a4
Compare24a943c
to16322d1
Comparedeansheather approved these changesJan 22, 2024
0b0b1a4
to7a301f1
Compare16322d1
toc91b1c0
Compare7a301f1
to0e38400
Comparec91b1c0
to671a2e9
Compare0e38400
to35d99a8
Compare671a2e9
to5fa6879
Compare35d99a8
to66da7c6
Compare5fa6879
to8ad2024
Compare66da7c6
to62e95de
Compare8ad2024
tod7a1794
Compare62e95de
to5695841
Compared7a1794
to59e8c2f
Compare59e8c2f
to8cbda21
Compare This was referencedJan 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
version
on 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
CurrentVersion
variables, to be part of theproto
packages, since the versions refer to the APIs defined therein.