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

Increase sharingKey complexity #883

Open
VirtualGit opened this issue Apr 10, 2024 · 5 comments
Open

Increase sharingKey complexity #883

VirtualGit opened this issue Apr 10, 2024 · 5 comments

Comments

@VirtualGit
Copy link

Is your feature request related to a problem? Please describe.
Sharing an album is possible without password thanks to a sharing key. That's exactly my need. But I find the sharing key complexity absolutely too much weak. It is currently composed of 8 characters including [0-9a-f]. From my point of view this is too weak.

Describe the solution you'd like
I'm interested in having a much more secure sharing key. At least 32 characters, maybe 64, instead of 8. Even better, characters could be taken from [0-9A-Za-z], which increase number of combinations. Maybe the sharing key length could be a setting ?

Describe alternatives you've considered (optional)
No alternative.

I can work on it and push a MR if you want. I'm not familiar with open-source collaboration, but there's a first time for everything :) If so, tell me the direction you'd prefer.

@VirtualGit
Copy link
Author

VirtualGit commented Apr 10, 2024

FYI, I've done it here :
VirtualGit@e305215
I need to do more tests, but it's looks good.

  • Still missing translation entries (not sure how to update xlf files)
  • Long keys are not wrapped in the share list (config page) ; I should maybe enable text ellipsis or something like that. Will think about it soon.

@bpatrik
Copy link
Owner

bpatrik commented Apr 11, 2024

Hi,
That commit looks good to me. Happy to merge it if you send a PR.

@VirtualGit
Copy link
Author

Thanks. I'll send you a PR when I fix this :
Copie d'écran_20240425_085016
Expect a few days/weeks.

@VirtualGit
Copy link
Author

Is this ok for you :
(I'm not CSS expert !)
VirtualGit@edf03fe

That leads to :
screenshot_20240502_220320
and :
screenshot_20240502_220416

@bpatrik
Copy link
Owner

bpatrik commented May 5, 2024 via email

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