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

"require user notification" does not take effect immediately #233

Closed
r2evans opened this issue May 9, 2018 · 5 comments
Closed

"require user notification" does not take effect immediately #233

r2evans opened this issue May 9, 2018 · 5 comments

Comments

@r2evans
Copy link

r2evans commented May 9, 2018

Steps that reproduce it (for me). Starting from "require user notification" DEselected.

  1. Select "require user notification", do not lock KeePass;
  2. Initiate something that requires an SSH key ("use ssh"), no confirmation required;
  3. Lock and unlock KeePass;
  4. Use ssh, confirmation is now required;
  5. Deselect "require user notification", do not lock KeePass;
  6. Use ssh, confirmation still required;
  7. Lock and unlock KeePass;
  8. Use ssh, confirmation not required.

If this is reproducible for others, then I suggest either (a) change it so the effect is immediate, or (b) document that a lock/unlock is required to effect the change.

@dlech
Copy link
Owner

dlech commented Jun 2, 2018

Where are you setting "require user notification"? The global options or on a per-entry basis?

@r2evans
Copy link
Author

r2evans commented Jun 2, 2018

Honestly, I didn't know that there was a per-entry basis for "require user notification". Is that what "Use confirm constraint" means on the single-entry KeeAgent tab?

(I was referring to the global config here.)

@dlech
Copy link
Owner

dlech commented Jun 2, 2018

Is that what "Use confirm constraint" means on the single-entry KeeAgent tab?

yes

@dlech
Copy link
Owner

dlech commented Jun 3, 2018

FWIW, the internal workings are a bit different, but it basically has the same effect.

dlech added a commit that referenced this issue Jun 5, 2018
dlech added a commit that referenced this issue Jun 5, 2018
@dlech
Copy link
Owner

dlech commented Jun 5, 2018

@dlech dlech closed this as completed Jun 5, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants