Skip to content

Next

  • Projects
  • Groups
  • Snippets
  • Help
    • Loading...
    • Help
    • Submit feedback
    • Contribute to GitLab
    • Switch to GitLab Next
  • Sign in / Register
Minds Backend - Engine
Minds Backend - Engine
  • Project
    • Project
    • Details
    • Activity
    • Releases
    • Dependency List
    • Cycle Analytics
    • Insights
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Charts
    • Locked Files
  • Issues 162
    • Issues 162
    • List
    • Boards
    • Labels
    • Service Desk
    • Milestones
  • Merge Requests 48
    • Merge Requests 48
  • CI / CD
    • CI / CD
    • Pipelines
    • Jobs
    • Schedules
    • Charts
  • Registry
    • Registry
  • Packages
    • Packages
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Members
    • Members
  • Collapse sidebar
  • Activity
  • Graph
  • Charts
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
  • Minds
  • Minds Backend - EngineMinds Backend - Engine
  • Issues
  • #560

Closed
Open
Opened 1 week ago by Ben Hayward@benhayward.ben
  • Report abuse
  • New issue
Report abuse New issue

(bug): Limited password validation on forgot password

Summary

When you forget your password, the password strength enforcement is different. Characters like @ are not allowed, but all lower case with no numbers and symbols is.

Steps to reproduce

  1. Log out
  2. Click on forgot my password on login form
  3. follow instructions
  4. try an all lower case password with an @
  5. try removing the @

Platform information

Very likely cross-platform but reproduced on S7 edge.

What is the current bug behavior?

Weak passwords are allowed.

What is the expected correct behavior?

Weak passwords should not be allowed.

Relevant logs and/or screenshots

(Paste any relevant logs - please use code blocks (```) to format console output, logs, and code as it's very hard to read otherwise.)

Possible fixes

(If you can, link to the line of code that might be responsible for the problem)

Please solve the reCAPTCHA

We want to be sure it is you, please confirm you are not a robot.

Edited 1 week ago by Ben Hayward

Related issues
0

    Related merge requests
    1
    • [Sprint/HipsterHedgehog](bug): Forgot password validation and refactor
      front !369
    • Ben Hayward @benhayward.ben added 1 - High P - Onboarding P - Platform T - Bug labels 1 week ago

      added 1 - High P - Onboarding P - Platform T - Bug labels

    • Ben Hayward @benhayward.ben changed title from (high): Limited password validation on forgot password to (bug): Limited password validation on forgot password 1 week ago

      changed title from (high): Limited password validation on forgot password to (bug): Limited password validation on forgot password

    • Ben Hayward @benhayward.ben assigned to @benhayward.ben 1 week ago

      assigned to @benhayward.ben

    • Ben Hayward @benhayward.ben changed weight to 3 1 week ago

      changed weight to 3

    • Ben Hayward @benhayward.ben changed milestone to %sprint: Hipster Hedgehog 1 week ago

      changed milestone to %sprint: Hipster Hedgehog

    • Ben Hayward @benhayward.ben added S - InProgress label 1 week ago

      added S - InProgress label

    • Ben Hayward @benhayward.ben added S - Review label and removed S - InProgress label 1 week ago

      added S - Review label and removed S - InProgress label

    • Ben Hayward @benhayward.ben changed weight to 5 1 week ago

      changed weight to 5

    • Ben Hayward @benhayward.ben changed milestone to %sprint: Hipster Hedgehog 1 week ago

      changed milestone to %sprint: Hipster Hedgehog

    • Ben Hayward @benhayward.ben changed weight to 5 1 week ago

      changed weight to 5

    • Ben Hayward @benhayward.ben moved from #513 (closed) 1 week ago

      moved from #513 (closed)

    • Ben Hayward @benhayward.ben mentioned in merge request front!369 (closed) 1 week ago

      mentioned in merge request front!369 (closed)

    • Mark Harding @markeharding removed S - Review label 3 days ago

      removed S - Review label

    • Mark Harding
      Mark Harding @markeharding · 3 days ago
      Owner

      This is for the backend to restrict, not the frontend

    • Ben Hayward
      Ben Hayward @benhayward.ben · 3 days ago
      Developer

      Good point, back-end is needed, my personal preference then would be to put front-end and back-end validation in, which allows us to handle incorrect passwords without throwing unneeded requests at the server.

      That being said I'm sure you've considered that already, so I'll re-do this for the back-end this sprint.

    • Ben Hayward @benhayward.ben added S - InProgress label 4 minutes ago

      added S - InProgress label

    • Ben Hayward @benhayward.ben changed milestone to %sprint: Hipster Hedgehog 3 minutes ago

      changed milestone to %sprint: Hipster Hedgehog

    • Ben Hayward @benhayward.ben changed weight to 5 3 minutes ago

      changed weight to 5

    • Ben Hayward @benhayward.ben moved from front#1442 (closed) 3 minutes ago

      moved from front#1442 (closed)

    • You're only seeing other activity in the feed. To add a comment, switch to one of the following options.
    Please register or sign in to reply
    Assignee
    Ben Hayward's avatar Ben Hayward @benhayward.ben
    Assign to
    None
    Epic
    None
    sprint: Hipster Hedgehog
    Milestone
    sprint: Hipster Hedgehog
    Assign milestone
    None
    Time tracking
    No estimate or time spent
    None
    Due date
    None
    5
    Labels
    1 - High P - Onboarding P - Platform S - InProgress T - Bug
    Assign labels
    • View project labels
    5
    Weight
    5
    Confidentiality
    Not confidential
    Lock issue
    Unlocked
    2
    2 participants
    user avatar
    Ben Hayward
    user avatar
    Mark Harding
    Reference: minds/engine#560