From 7bbbfe20b9c6cfd6ba8e74926c43a61d783fb240 Mon Sep 17 00:00:00 2001
From: Shane Kilkelly <shane@kilkelly.me>
Date: Wed, 21 Dec 2016 13:50:13 +0000
Subject: [PATCH] If external auth is used, remove `/register` items from
 header nav.

(logic moved from docker-image settings file)
---
 .../app/coffee/infrastructure/ExpressLocals.coffee    | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/services/web/app/coffee/infrastructure/ExpressLocals.coffee b/services/web/app/coffee/infrastructure/ExpressLocals.coffee
index e469df9422..d976a1d23a 100644
--- a/services/web/app/coffee/infrastructure/ExpressLocals.coffee
+++ b/services/web/app/coffee/infrastructure/ExpressLocals.coffee
@@ -244,6 +244,17 @@ module.exports = (app, webRouter, apiRouter)->
 		for key, value of Settings.nav
 			res.locals.nav[key] = _.clone(Settings.nav[key])
 		res.locals.templates = Settings.templateLinks
+		try
+			externalAuth = res.locals.externalAuthenticationSystemUsed()
+			if externalAuth and res.locals.nav.header?
+				# filter out '/register' link
+				res.locals.nav.header = _.filter(
+					res.locals.nav.header,
+					(h) ->
+						h.url != '/register'
+				)
+		catch error
+			logger.error {error}, "error while trying to filter out '/register' links from header"
 		next()
 
 	webRouter.use (req, res, next) ->