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

Update build.ps1 #165

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Update build.ps1 #165

wants to merge 2 commits into from

Conversation

Fishy78
Copy link

@Fishy78 Fishy78 commented Jul 18, 2019

Due windows system rights it is not allowed for a non-elevated user to create or change the KeePassConfiguration.xml due it is located at "C:\Program Files\WindowsPowerShell\Modules\PoShKeePass\2.1.3.0".
The proposed extension checks the right to write the configuration and creates, if necessary, a new empty configuration file either at $PSScriptRoot or in %AppData%\PoShKeePass.
Advantage:

  • In any case the module finds a fully working configuration file
  • Any user can use his own KeePass Containers
    Neutral:
  • There is usually no global configuration file except for elevated Administrators
    Disadvantage:
  • Is the script is used e.g. in a scheduled task, the "executing" user needs to have writing rights to the configuration. Maybe it is necessary to add a switch to override the write check.

@Fishy78
Copy link
Author

Fishy78 commented Jul 18, 2019

Btw, the below changes (@@ -114,4 +146,4 @@ $RootFilesToCopy | ForEach-Object {) have not been commited by me

@jkdba jkdba self-assigned this Jul 19, 2019
@Fishy78
Copy link
Author

Fishy78 commented Aug 27, 2019

Any news?

@O-Rud
Copy link

O-Rud commented Aug 3, 2022

I would really like to see this implemented

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.

3 participants