----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/121962/#review74295 -----------------------------------------------------------
Ping? - Luca Beltrame On Gen. 10, 2015, 10:02 a.m., Luca Beltrame wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/121962/ > ----------------------------------------------------------- > > (Updated Gen. 10, 2015, 10:02 a.m.) > > > Review request for Amarok. > > > Repository: amarok > > > Description > ------- > > The sha256() function provided to generate hashes for Ampache is incorrect > (generates wrong hashes), and thus will not generate a proper passphrase for > Ampache versions greater than 35, which will fail with an "incorrect > password" message. > > The solution is to use the hash functions provided by QCA, which work > correctly. However this adds a (conditional and optional) dependency to QCA > to build the Ampache plugin. > > > Diffs > ----- > > src/services/CMakeLists.txt 83ccdfb > src/services/ampache/AmpacheAccountLogin.cpp 26aebed > src/services/ampache/CMakeLists.txt dd298e8 > src/services/ampache/sha256/sha256.h 9bf7387 > src/services/ampache/sha256/sha256.cpp 01b3764 > > Diff: https://git.reviewboard.kde.org/r/121962/diff/ > > > Testing > ------- > > Compiles, a non-working login with Ampache (git master) now works correctly. > > > Thanks, > > Luca Beltrame > >
_______________________________________________ Amarok-devel mailing list Amarok-devel@kde.org https://mail.kde.org/mailman/listinfo/amarok-devel
