blob: 119f7f670df7d67209a9896e82eb85865b1c6882 [file] [log] [blame]
Frédéric Lécailleb083c832018-08-23 18:06:35 +02001# commit 7b6cc52784526c32efda44b873a4258d3ae0b8c7
2# BUG/MINOR: lua: Bad HTTP client request duration.
3#
4# HTTP LUA applet callback should not update the date on which the HTTP client requests
5# arrive. This was done just after the LUA applet has completed its job.
6#
Ilya Shipitsin47d17182020-06-21 21:42:57 +05007# This patch simply removes the affected statement. The same fix has been applied
Frédéric Lécailleb083c832018-08-23 18:06:35 +02008# to TCP LUA applet callback.
9#
10# To reproduce this issue, as reported by Patrick Hemmer, implement an HTTP LUA applet
11# which sleeps a bit before replying:
12#
13# core.register_service("foo", "http", function(applet)
14# core.msleep(100)
15# applet:set_status(200)
16# applet:start_response()
17# end)
18#
Ilya Shipitsin01881082021-08-07 14:41:56 +050019# This had as a consequence to log %TR field with approximately the same value as
Frédéric Lécailleb083c832018-08-23 18:06:35 +020020# the LUA sleep time.
21
22varnishtest "LUA bug"
Christopher Fauletfdafd9a2018-12-19 11:50:17 +010023#REQUIRE_OPTIONS=LUA
Frédéric Lécailledc1a3bd2019-03-29 15:07:24 +010024#REGTEST_TYPE=bug
Frédéric Lécailleb083c832018-08-23 18:06:35 +020025
26feature ignore_unknown_macro
27
28syslog Slog {
Frédéric Lécailleb083c832018-08-23 18:06:35 +020029 recv info
Willy Tarreau9c27ea02018-11-16 15:54:23 +010030 expect ~ "[^:\\[ ]\\[[0-9]*\\]: Ta=[0-9]* Tc=[0-9]* Td=[0-9]* Th=[0-9]* Ti=[0-9]* Tq=[0-9]* TR=[0-9]* Tr=[0-9]* Tt=[0-9]* Tw=[0-9]*$"
Frédéric Lécailleb083c832018-08-23 18:06:35 +020031
32 recv info
Willy Tarreau9c27ea02018-11-16 15:54:23 +010033 expect ~ "[^:\\[ ]\\[[0-9]*\\]: Tc=[0-9]* Td=[0-9]* Th=[0-9]* Tt=[0-9]* Tw=[0-9]*$"
Frédéric Lécailleb083c832018-08-23 18:06:35 +020034} -start
35
36haproxy h1 -conf {
37 global
Frédéric Lécailleb894f922019-03-29 16:13:48 +010038 lua-load ${testdir}/bad_http_clt_req_duration.lua
Frédéric Lécailleb083c832018-08-23 18:06:35 +020039
40 defaults
41 timeout client 1s
42 timeout server 1s
43 timeout connect 1s
44
45 frontend f1
46 mode http
47 bind "fd@${f1}"
48 log ${Slog_addr}:${Slog_port} daemon
49 log-format Ta=%Ta\ Tc=%Tc\ Td=%Td\ Th=%Th\ Ti=%Ti\ Tq=%Tq\ TR=%TR\ Tr=%Tr\ Tt=%Tt\ Tw=%Tw
50 default_backend b1
51
52 backend b1
53 mode http
Frédéric Lécailleb083c832018-08-23 18:06:35 +020054 http-request use-service lua.foo.http
55
56 frontend f2
57 mode tcp
58 bind "fd@${f2}"
59 log ${Slog_addr}:${Slog_port} daemon
60 log-format Tc=%Tc\ Td=%Td\ Th=%Th\ Tt=%Tt\ Tw=%Tw
61
62 tcp-request inspect-delay 1s
63 tcp-request content use-service lua.foo.tcp
64} -start
65
66client c1 -connect "${h1_f1_sock}" {
67 txreq
68 rxresp
69} -run
70
71client c2 -connect "${h1_f2_sock}" {
72 txreq
73 rxresp
74} -run
75
76syslog Slog -wait