-
Notifications
You must be signed in to change notification settings - Fork 516
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
qt6: add postgresql #2957
base: master
Are you sure you want to change the base?
qt6: add postgresql #2957
Conversation
Qt6 supports Window's native schannel. Is there a strong case for building with openssl support too, given the extra dependencies? |
Actually, I wasn't aware of Qt's schannel support on Windows. I just noticed, that QNetworkManager was not able do establish https connections, and enabling openssl for Qt fixed my problem. But I quess schannel would be fine, too. |
I think schannel is enabled. At least it works for me. If we're going to enable the postgresql driver, that's going to drag in openssl anyway. Hmm. |
I'll try and let you know! |
I've been using Qt6 QNetworkRequest successfully for HTTPS without explicitly enabling OpenSSL. |
Enable openssl and the PostgreSQL QSql plugin for qt6, like it is also in qt5 fixes mxe#2950
2d82545
to
93d5404
Compare
Yes SSL works also without OpenSSL! (I had a mistake in my packager script, so the schannel plugin was in the wrong directory). So you are absolutely right, we don't need it on Windows. |
It is possible to build both plugins, but I think it is not easily possible to choose the back end at runtime. |
This adds a very heavy dependency to Qt permanently... I wouldn't want this for our build images, for example. |
Although Qt 6 supports schannel on Windows, it may not work for all use cases. Now both openssl and schannel should be available. #2957
Although Qt 6 supports schannel on Windows, it may not work for all use cases. Now both openssl and schannel should be available. mxe#2957
Enable openssl and the PostgreSQL QSql plugin for qt6, like it is also in qt5
fixes #2950