llaske / sugarizer

Sugarizer is a web implementation of the Sugar platform to run on any device or browser
https://sugarizer.org
Apache License 2.0
198 stars 411 forks source link

When password is mainly based on numbers, it expects minimum of 7 images instead of 4 #1380

Closed raghchawla131 closed 6 months ago

raghchawla131 commented 1 year ago

the issue happens when it's whole/mainly based on number (i.e. images with a shortcut as number like 1234). Not happen with only letter images.

llaske commented 1 year ago

It's depending of the server settings (see here). On which server did you try?

raghchawla131 commented 1 year ago

I accessed the website using the latest version of Google Chrome and visited the public URL directly without running it locally on any server. The issue occured when i was creating a password, i was only able to create a password with minimum of 7 images and not 4

llaske commented 1 year ago

Hmmm, just tested to create an user on the public Sugarizer Server on https://try.sugarizer.org/ and it works with 4 images. Process:

Did you follow this process? Are you able to do some screen capture?

raghchawla131 commented 1 year ago

https://github.com/llaske/sugarizer/assets/112266194/ec217ed0-23a8-40fe-bb5b-a3bcc05b22c6

llaske commented 1 year ago

Oh funny enough, it seems to depends of the password: the issue happens when it's whole/mainly based on number (i.e. images with a shortcut as number like 1234). Not happen with only letter images like aaaa. Thanks to identify this issue :+1:

raghchawla131 commented 1 year ago

I’m glad for the opportunity to help sir. I new to open source, and i am unable to run sugarizer web application locally on my system, could you please help me. I tried following the readMe but I was only able to run it's desktop application.

llaske commented 1 year ago

i am unable to run sugarizer web application locally on my system

Running Sugarizer Web Application could mean two things:

llaske commented 1 year ago

Fixed in https://github.com/llaske/sugarizer/commit/78cd33243bfc9c232ac5163702e052e1c772d78f You could test on https://dev.sugarizer.org