qthttpserver/tests/auto/qhttpserverresponder/qhttpserverresponder.pro

CONFIG += testcase
TARGET = tst_qhttpserverresponder
SOURCES  += tst_qhttpserverresponder.cpp

QT = httpserver testlib

TESTDATA += *.html
Introduce QHttpServerResponder It encapsulates the socket and gives an API to answer the received requests. Change-Id: Ic95db2c50224a650a02b206faca9a0ff8d1cc62b Reviewed-by: Mårten Nordheim <marten.nordheim@qt.io> Reviewed-by: Ryan Chu <ryan.chu@qt.io> Reviewed-by: Edward Welbourne <edward.welbourne@qt.io> 2018-03-13 15:20:47 +01:00			`CONFIG += testcase`
			`TARGET = tst_qhttpserverresponder`
			`SOURCES += tst_qhttpserverresponder.cpp`

			`QT = httpserver testlib`
Refactor QHttpServerResponder::write(QIODevice, ...) The current implementation has several problems. 1. The function takes an ownership the QIODevice and puts it into a smartpointer. Also we conntected socket's destroyed signal to lambda which has captured the smartpointer. So if responder does not find the file, the smartpointer will call deleteLater which then will call socket::destroyed and then lambda will be called and try to access the smartpointer which does not exist anymore. 2. The function takes an ownership the file(QIODevice) and puts it into a smartpointer. Also we conntected the QTemporaryFile's aboutToClose signal to lambda which has captured the smartpointer. So when the QTemporaryFile calls destructor it will emit aboutToClose signal which will call the lambda and this lambda will try to access the smartpointer which does not exist anymore. 3. If we send a file smaller than chunksize, IOChunkedTransfer we will never be deleted. 4. Does not check a socket connection type (keep-alive). 5. Does not send anything if file is not found or opened in wrong mode. Change-Id: I699e7d5a462c4b8d195908747bf0386132b19973 Reviewed-by: Jesus Fernandez <Jesus.Fernandez@qt.io> 2018-10-03 10:14:42 +02:00
			`TESTDATA += *.html`