- Notifications
You must be signed in to change notification settings - Fork1
impl: store last used URL in Toolbox Settings Store#200
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
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
Context: Toolbox can store key/value pairs in two places:- a settings store which is backed by a clear text json file per each plugin- native keystore for sensitive dataAt the same time some of Coder's clients (ex: Netflix) would like to deploy atscale preconfigured settings for Toolbox. Most of the needed settings are part ofjson backed store except the last used URL.This PR reworks the code around the last used URL/token and moves the URL in thejson backed store, making it easy to configure. At the same time we still supportthe pair stored in the native keystore for backward compatibility reasons.
After a successful connection to a deployment URL we usedto save the same token associated with two keys:- last-token- and associated with the last urlBut it does not make sense, once we have to URL we can easily resolve the tokenassociated with the URL. In other words this commit removes the ability to read/savea token associated with `last-token` key.
code-asher approved these changesSep 26, 2025
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
URL stored in the native OS secrets store is now superseded bythe URL stored in the JSON backed settings store.
With a takIf/let pattern. Theoretically you can have two threadsracing - one changing the value of lastDeploymentURL and another\one reading. In our code that should be pretty much impossible.But the compiler can't be sure of that so it throws and error.
94b329f
intomain 6 checks passed
Uh oh!
There was an error while loading.Please reload this page.
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
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.
Context: Toolbox can store key/value pairs in two places:
At the same time some of Coder's clients (ex: Netflix) would like to deploy at scale preconfigured settings for Toolbox. Most of the needed settings are part of json backed store except the last used URL.
This PR reworks the code around the last used URL/token and moves the URL in the json backed store, making it easy to configure. At the same time we still support the pair stored in the native keystore for backward compatibility reasons.