From f447020b4769ca49fff826dbd7df580f88d60592 Mon Sep 17 00:00:00 2001
From: Keith Lawrence <keith.lawrence@digital.cabinet-office.gov.uk>
Date: Mon, 8 Jul 2024 16:39:05 +0100
Subject: [PATCH] Rename parameter variable

 - in tests, this worked fine. But in practise, there was some odd
   variable shadowing going on that led to the client
   logout_token method being called with a nil.
---
 app/controllers/oidc_events_controller.rb | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/app/controllers/oidc_events_controller.rb b/app/controllers/oidc_events_controller.rb
index b7087998..f961484f 100644
--- a/app/controllers/oidc_events_controller.rb
+++ b/app/controllers/oidc_events_controller.rb
@@ -1,6 +1,6 @@
 class OidcEventsController < ApplicationController
   def backchannel_logout
-    logout_token = oidc_client.logout_token(logout_token)
+    logout_token = oidc_client.logout_token(logout_token_jwt)
     if logout_token
       user_id = logout_token[:logout_token].sub
       LogoutNotice.new(user_id).persist
@@ -13,7 +13,7 @@ def backchannel_logout
 
 private
 
-  def logout_token
+  def logout_token_jwt
     params.require(:logout_token)
   end