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

[security] Manual user creation with password does not work as expected #178

Open
tukanos opened this issue Mar 2, 2018 · 2 comments
Open

Comments

@tukanos
Copy link
Contributor

tukanos commented Mar 2, 2018

Hi Ethan,

I have just tried to create a user manually (no mailouts yet) and set him a password from within the Decko system, but to no avail.

I was unable to login as the user. Is there a trick to do so?

@ethn
Copy link
Member

ethn commented Mar 2, 2018

Can you confirm that:

  1. The user has a user card (type is "User")
  2. that user card has a +*account card
  3. that +*account card has reasonable looking content for the +*email, +*password, and +*status fields? (password will just show the word "encrypted")

@tukanos
Copy link
Contributor Author

tukanos commented Mar 5, 2018

I tried that with the following screen cast.

ad 1) yes the user card is type "User"
ad 2) the user card has a +*account card
ad 3) looks reasonable and the status field I changed from pending to active manually

unable_to_change_password_v2.zip

You can see it also on the attached video:

Decko changing password

I have even encountered an error during a search. When a search string is tukan+*account+*status then the regexp engine returns an error (displayed in the screencast).

The error:

Error message (visible to admin only)

PG::InvalidRegularExpression: ERROR: invalid regular expression: quantifier operand invalid : /* *search / SELECT DISTINCT c1., c1.updated_at FROM cards c1 WHERE ((replace(c1.name,'+',' ') ~* '[[:<:]]tukan[[:>:]]' OR c1.db_content ~* '[[:<:]]tukan[[:>:]]') AND (replace(c1.name,'+',' ') ~* '[[:<:]]*account[[:>:]]' OR c1.db_content ~* '[[:<:]]*account[[:>:]]') AND (replace(c1.name,'+',' ') ~* '[[:<:]]*status[[:>:]]' OR c1.db_content ~* '[[:<:]]*status[[:>:]]')) AND c1.trash is false ORDER BY c1.updated_at desc LIMIT 20 OFFSET 0

@tukanos tukanos changed the title [security] Manual user creationg with password does not work as expected [security] Manual user creation with password does not work as expected Mar 6, 2018
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

2 participants