Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SSH widget updates when config file changes #3850

Merged
merged 1 commit into from
Sep 13, 2024

Conversation

lauren-ciha
Copy link
Contributor

Summary of the pull request

The SSH widget was not updating when the config file changed because the widget did not allow outside processes to read or write to the file.

This pull request modifies the SSHWalletWidget's FileStreamOptions to allow read/write sharing.

References and relevant issues

Resolves #2672

Detailed description of the pull request / Additional comments

Before:

  • SSH widget didn't update when its config file changed
  • If multiple SSH widgets pointed to the same config file, a file is busy error arose in at least one of the widgets when the config file was updated

After:

  • SSH widget updates when its config file changes
  • Multiple SSH widgets pointing to the same config file update together when the config file changes

Validation steps performed (locally)

  • Created sample config file with two hosts
  • Opened config file in text editor
  • Added one (and multiple) SSH widgets to dashboard
  • Repeatedly removed/added one host, saving after each change
  • Found that with each save, SSH widget updated to reflect the config file

PR checklist

@lauren-ciha lauren-ciha merged commit 3130870 into main Sep 13, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

SSH widget does not reflect changes to the content of the configured config file
3 participants