-
-
Notifications
You must be signed in to change notification settings - Fork 131
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
Changed ConfigServer to be in module's scope #370
Conversation
What about saving a config file to the user profile? Or something like this? |
Removing persistent sessions feels like a step backwards. I agree with @sstodd7532, why not store it in the user profile or in a module\user specific folder in app data roaming. The password is encrypted on disk so a centralized location for a session file shouldn't be a security risk. This will also provide session persistence through module upgrades, which is another issue I have with how sessions are handled currently. |
fair point. @Dejulia489 : it's not the session which is stored. only the address of the server. and I don't know how you think this would be encrypted |
That’s right you aren’t storing credentials along with the server Config. I think that should be addressed as well. Regardless of what is stored, it can be moved to $env:appdata to resolve this issue without removing functionality. It will also add functionality by persisting through module upgrades. Edit: Removed spacing in AppData |
2f71d31
to
6d34977
Compare
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[Environment]::GetFolderPath('ApplicationData')
is the cross-platform solution.
No more breaking change, LGTM
Description
Have
Set-JiraConfigServer
store the server in the module's session instead of in a file.Motivation and Context
Several issues were raised asking for help because the
config.xml
could not be written (or changed)Types of changes
Checklist