BUG/MEDIUM: resolvers: Make 1st server of a template take part to SRV resolution
authorChristopher Faulet <cfaulet@haproxy.com>
Tue, 29 Jun 2021 18:52:35 +0000 (20:52 +0200)
committerChristopher Faulet <cfaulet@haproxy.com>
Tue, 29 Jun 2021 18:52:37 +0000 (20:52 +0200)
commit81ba74ae5061bc55b3460f01a99371281de8f7f3
treedb5d2bf54610816a44c57c76a0a12c6f28464ef4
parent0de0becf0b024b6beca9c5f7e1ed8a08120d781c
BUG/MEDIUM: resolvers: Make 1st server of a template take part to SRV resolution

The commit 3406766d5 ("MEDIUM: resolvers: add a ref between servers and srv
request or used SRV record") introduced a regression. The first server of a
template based on SRV record is no longer resolved. The same bug exists for
a normal server based on a SRV record.

In fact, the server used during parsing (used as reference when a
server-template line is parsed) is never attached to the corresponding srvrq
object. Thus with following lines, no resolution is performed because
"srvrq->attached_servers" is empty:

  server-template test 1 _http.domain.tld resolvers dns ...
  server test1 _http.domain.tld resolvers dns ...

This patch should fix the issue #1295 (but not confirmed yet it is the same
bug). It must be backported everywhere the above commit is.
src/server.c