From 518090f900b4183fb65f84c0309f3661e1e8390a Mon Sep 17 00:00:00 2001
From: LEGO Technix <109212476+lego-technix@users.noreply.github.com>
Date: Mon, 20 Feb 2023 11:03:29 +0100
Subject: [PATCH 1/2] doc: document new nginx env variables introduced in
 pix-7084-add-geolocation-endpoint

---
 README.md | 26 ++++++++++++++++++++++++++
 1 file changed, 26 insertions(+)

diff --git a/README.md b/README.md
index 45e238481..b47611001 100755
--- a/README.md
+++ b/README.md
@@ -74,6 +74,32 @@ If not present, Metabase cards will not be loaded.
 - type: String
 - default: none
 
+---
+
+`NGINX_GEOAPI_UPSTREAM_HOST`
+Host name of the GEOAPI service that is providing the "/me" endpoint.
+This variable is used only in the nginx configuration.
+If not present, nginx will start and stop in error with message "nginx: [emerg] no host in upstream".
+
+- presence: required
+- type: String
+- default: none
+
+`NGINX_GEOAPI_UPSTREAM_MAX_FAILS`
+This variable is used only in the nginx configuration.
+
+- presence: optional
+- type: Number
+- default: 3
+
+`NGINX_GEOAPI_UPSTREAM_FAIL_TIMEOUT`
+This variable is used only in the nginx configuration.
+
+- presence: optional
+- type: String
+- default: 5s
+
+
 ## Build Setup
 
 En dev, copier le fichier `sample.env` vers un `.env` et remplacer les valeurs.

From aadbb8ec0394bba0a4d433e533f57d3b8a64f49c Mon Sep 17 00:00:00 2001
From: LEGO Technix <109212476+lego-technix@users.noreply.github.com>
Date: Mon, 20 Feb 2023 11:29:01 +0100
Subject: [PATCH 2/2] Documentation addition provided by the captains

Co-authored-by: Mickael Alibert <mickaelalibert@users.noreply.github.com>
---
 README.md | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/README.md b/README.md
index b47611001..673fee8e6 100755
--- a/README.md
+++ b/README.md
@@ -86,6 +86,7 @@ If not present, nginx will start and stop in error with message "nginx: [emerg]
 - default: none
 
 `NGINX_GEOAPI_UPSTREAM_MAX_FAILS`
+Number of failed request to upstream before putting it in quarantine.
 This variable is used only in the nginx configuration.
 
 - presence: optional
@@ -93,6 +94,7 @@ This variable is used only in the nginx configuration.
 - default: 3
 
 `NGINX_GEOAPI_UPSTREAM_FAIL_TIMEOUT`
+Duration of the quarantine for a failed upstream server.
 This variable is used only in the nginx configuration.
 
 - presence: optional