From ceb08f9334d4210e3ef03dd876442e069c20213e Mon Sep 17 00:00:00 2001
From: Frank Wall <fw@moov.de>
Date: Tue, 14 Apr 2020 23:58:41 +0200
Subject: [PATCH] net/haproxy: fix invalid use of option forwardfor

This resolves another infamous warning during config test:
'option forwardfor' ignored for frontend as it requires HTTP mode
---
 .../opnsense/service/templates/OPNsense/HAProxy/haproxy.conf    | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/net/haproxy/src/opnsense/service/templates/OPNsense/HAProxy/haproxy.conf b/net/haproxy/src/opnsense/service/templates/OPNsense/HAProxy/haproxy.conf
index 1e47bd478..d3d429689 100644
--- a/net/haproxy/src/opnsense/service/templates/OPNsense/HAProxy/haproxy.conf
+++ b/net/haproxy/src/opnsense/service/templates/OPNsense/HAProxy/haproxy.conf
@@ -1087,7 +1087,7 @@ frontend {{frontend.name}}
     http-request replace-header Cookie '^(.*?; )?({{backend_data.persistence_cookiename}}=)"([^;"]*)"(;.*)?$' \1\2\3\4
 {%         endif %}
 {%       endif %}
-{%       if frontend.forwardFor == '1' %}
+{%       if frontend.forwardFor == '1' and frontend.mode == 'http' %}
     option forwardfor
 {%       endif %}
     # tuning options