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

Jump to bottom

Refresh usage of Windows Credential manager to properly store credentials #12803

Closed
Tracked in #12805
AliveDevil opened this issue Jan 20, 2022 · 0 comments · Fixed by #12918
Closed
Tracked in #12805

Refresh usage of Windows Credential manager to properly store credentials #12803

AliveDevil opened this issue Jan 20, 2022 · 0 comments · Fixed by #12918
Assignees
@AliveDevil
Labels
enhancement
Milestone
8.4.0

Comments

@AliveDevil
Copy link
Contributor

@AliveDevil AliveDevil commented Jan 20, 2022

Current implementation lacks compatibility and is prone to errors due to length limitations of password blobs in Windows Credential Manager.
Review implementation to include "attributes"/pages for increased payload size of up to 64 KiB - should be enough for a password.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@AliveDevil AliveDevil

Labels
enhancement
Projects
None yet
Milestone
8.4.0
Development

Successfully merging a pull request may close this issue.

Modernize PInvoke with CSWin32 iterate-ch/cyberduck
2 participants
@dkocher @AliveDevil