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

[Bug] If the Username contains /... #435

Open
wubinworks opened this issue Dec 27, 2023 · 0 comments
Open

[Bug] If the Username contains /... #435

wubinworks opened this issue Dec 27, 2023 · 0 comments

Comments

@wubinworks
Copy link

Describe the bug
Cannot change Username if Username contains /
/ is invalid char for Username.

To Reproduce
Steps to reproduce the behavior:

  1. Login as admin and create a new user has abcd as Username.
  2. Update the Username to ab/cd.
  3. Now, you cannot change ab/cd to anything else. (Error message: Not found)

Environment

  • FileGator Version: 7.9.3
  • Server: Apache 2.4 on Ubuntu
  • PHP Version: PHP 8.2
  • Browser: Firefox

Additional context

Cause:

http://localhost/?r=/updateuser/ab/cd
the last / caused routing issue.

Suggestion:

As Username is the ID in this system, I suggest limiting the valid char to A-Za-z0-9-_.@.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant