There is still lot to fix. One main headache is related to how QByteArray's are handled in Qt6:
https://doc.qt.io/qt-6/qstring.html#QString-9. This breaks Base32 among other classes.

Note: : any null ('\0') bytes in the byte array will be included in this string, converted to Unicode null characters (U+0000). This behavior is different from Qt 5.x.

The easiest way to deal this is to replace QString(byteArray) calls with QString(byteArray.toStdString().c_str()) but it's not very convenient. Although it's mostly used in tests, we should make sure there are no places where this is actually used with some relevant data.

If we need to convert a non-string byte array (as in actual raw bytes) to qstring for comparison or printing, then we should convert it to hex or base64 first. Ideally we would just compare with the desired bytestring though.

If we need to convert a non-string byte array (as in actual raw bytes) to qstring for comparison or printing, then we should convert it to hex or base64 first. Ideally we would just compare with the desired bytestring though.

QString::fromUtf8() is the most problematic one here. It's used in multiple relevant places, like in KeePassReader classes. Even after changing those, the tests fail. There's also something strange about the Endian functions. Currently I'm out of ideas.

Is this still gradually moving or got stuck?

Want to merge other major refactoring first

any work that can be carved off here into other tickets?, I'm willing to submit PRs, failing tests, etc.

I think its the CMake cleanup first, then this one. Wrapping up 2.7.10 development first which should carry us through next year for 2.8.0. Gotta do all this prep to work on the big new features like entry layouts and FIDO auth.

pinging as this is important

Yes agree, was looking at this earlier this week. We do have an EOS date for qt 5.15, and we are pretty strict about that stuff.

@varjolintu can you rebase this onto #11003?

Yes agree, was looking at this earlier this week. We do have an EOS date for qt 5.15, and we are pretty strict about that stuff.

@varjolintu can you rebase this onto #11003?

Maybe that should be rebased as well first?

Good point

@varjolintu rebase done

FYI: Still working on this. Started with KPXC_MINIMAL build. macOS side already works and all tests pass. When everything seems fine with Linux and Windows as well, I'll move to a full build.

Work continues in #11651.