خطأ في أحدث إصدار من discourse-migratepassword

fhe · 13 مارس 2019، 7:37م

أود لفت الانتباه إلى خلل في discourse-migratepassword يمنع المستخدمين الذين تم استيراد كلمات مرورهم من إعادة تعيين كلمة المرور في ظروف معينة.

ظهر هذا الخلل مع إضافة دعم Question2Answer، وقد قمت بتقديم طلب سحب (PR) لإصلاح المشكلة هنا:

github.com/communiteq/discourse-migratepassword

Fixed no conversion of nil to String in check_q2a (#10)

master ← fheidenreich:fix-q2a

merged 07:42PM - 13 Mar 19 UTC

fheidenreich

+3 -1

I've noticed users that haven't visited by community for a while to be unable to… login and even send password resets over the last couple of days. I was able to track down the cause to the newly added support for Q2A. A string `m0OO&:dfd8942a0affeb062438be8de6ae4600` gives a salt that is shorter than 8 characters, so that `salt[-8..-1]` returns `nil` which triggers a `TypeError (no implicit conversion of nil into String)`.

michaeld · 13 مارس 2019، 7:44م

Good catch! Thank you!

fhe · 13 مارس 2019، 7:44م

Thanks for the quick merge!

الموضوع		الردود	مرات العرض
Importer support for migrated passwords Development	7	1453	12 مارس 2018
Cannot login to dev instance with imported user Development	5	1568	7 فبراير 2018
"password too short" on vbulletin import due to migratepasssword plugin Development	1	842	16 أغسطس 2019
Migrated password hashes support Plugin	32	11964	16 يناير 2023
Q2a import error Migration	5	573	2 نوفمبر 2019

خطأ في أحدث إصدار من discourse-migratepassword

الموضوعات ذات الصلة