Skip to content

Update build.ps1#165

Open
Fishy78 wants to merge 2 commits intoPSKeePass:masterfrom
Fishy78:patch-1
Open

Update build.ps1#165
Fishy78 wants to merge 2 commits intoPSKeePass:masterfrom
Fishy78:patch-1

Conversation

@Fishy78
Copy link
Copy Markdown

@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
Copy Markdown
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
Copy Markdown
Author

Fishy78 commented Aug 27, 2019

Any news?

@O-Rud
Copy link
Copy Markdown

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