pagure

#666 Minor changes in new issue UI, changed to_unicode to to_bytes to avoid Unicode error in bcrypt
Closed by vivekanand1101. Opened by vivekanand1101.
vivekanand1101/pagure too-many-in-one  into  master

Download 666.patch
no initial comment
  1. bcrypt.hashpw doesn't accept unicode, neither in password nor in salt. As suggested by @ralph, it should have been bytes.
  2. wtf.ValidationError throws error. It should have been wtforms.validators.ValidationError. here
  3. Even if field description was present (in the macro render_bootstrap_field), it wasn't displayed in case of boolean field.
Edited by vivekanand1101

Did you check the bcrypt or the py-bcrypt library? Both have a bcrypt module but they are two different modules and we use the later one.
Edited by pingou

Other than this question, the rest looks good
Metadata
None
No Tags
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.