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

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

Closed
raghchawla131 opened this issue Jul 29, 2023 · 8 comments
Labels
bug to be release Fixed, to be release
Milestone

Comments

@raghchawla131
Copy link

raghchawla131 commented Jul 29, 2023

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
Copy link
Owner

llaske commented Jul 29, 2023

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

@raghchawla131
Copy link
Author

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
Copy link
Owner

llaske commented Aug 3, 2023

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

  • Connect to https://try.sugarizer.org/
  • Click on New User button
  • Type a new name
  • Click on Next button
  • Choose 4 images
  • Click on Next button
  • Click on Next button
  • Click on Accept button

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

@raghchawla131
Copy link
Author

Series.of.images.for.password.is.of.at.least.7.images.Issue.1380.llaske_sugarizer.-.Google.Chrome.2023-08-04.20-45-07.mp4

@llaske
Copy link
Owner

llaske commented Aug 4, 2023

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 👍

@llaske llaske added the bug label Aug 4, 2023
@raghchawla131
Copy link
Author

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.

@raghchawla131 raghchawla131 changed the title Series of images for password is of at least 7 images When password is mainly based on numbers, it expects minimum of 7 images instead of 4 Aug 5, 2023
@llaske
Copy link
Owner

llaske commented Aug 5, 2023

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

Running Sugarizer Web Application could mean two things:

  • Running Sugarizer from the browser using local files. In short it means running Sugarizer as the desktop version but from the browser. You could do it following instruction Sugarizer Application into the Web Browser from here.
  • Running Sugarizer Server. It means executing a web server to host Sugarizer client and server features (sharing activities, remote storage, ...) and access it from a browser locally or remotely. You could do it following instructions here.

@llaske
Copy link
Owner

llaske commented Aug 5, 2023

Fixed in 78cd332
You could test on https://dev.sugarizer.org

@llaske llaske added the to be release Fixed, to be release label Aug 5, 2023
@llaske llaske added this to the v1.8 milestone Apr 10, 2024
@llaske llaske closed this as completed Apr 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug to be release Fixed, to be release
Projects
None yet
Development

No branches or pull requests

2 participants