qiurui
/
qthttpserver
mirror of https://github.com/qt/qthttpserver.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Shorten local socket path in autotest 

The limitation on Unix domain socket paths on Linux is 107 characters.
After c617cc95934ae3c0896082d61a88487b34cf96be the temp directory
handling on Android must have changed to go past that limitation. When
testing this the socket path on my device was

/data/user/0/org.qtproject.example.tst_qhttpservermultithreaded/cache/
tst_qhttpservermultithreaded_local11757

...which is too long and therefore the listening on the socket will
fail.

The simple quick solution to get everything working is to just shorten
the socket name a bit. Combined with the application PID it should still
be unique and not cause any other issues.

Pick-to: 6.10 6.9
Fixes: QTBUG-137849
Change-Id: Ib606ea315d7a929a37a0639c07ed08b474b54812
Reviewed-by: Assam Boudjelthia <assam.boudjelthia@qt.io>
Reviewed-by: Øystein Heskestad <oystein.heskestad@qt.io>
This commit is contained in:
Olli Vuolteenaho 2025-06-24 14:38:16 +03:00
parent d9cad3e9f6
commit cc96264b62
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
tests/auto/qhttpservermultithreaded/tst_qhttpservermultithreaded.cpp
View File

@ -136,7 +136,7 @@ static int port = 0;
#if QT_CONFIG(ssl)
static int sslPort = 0;
#endif
static const QString local = u"tst_qhttpservermultithreaded_local"_s + QString::number(QCoreApplication::applicationPid());
static const QString local = u"tst_qhttpservermultithreaded"_s + QString::number(QCoreApplication::applicationPid());
// Simple HTTP 1.1 client
class LocalHttpClient
{