From c973cd2f38bf305c4f08cebdf361fdd08e434fa2 Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Thu, 13 Nov 2025 13:46:33 +0000
Subject: [PATCH 1/2] Initial plan


From b7e44aef1816b010a2c60be88a37dcfb8c971ccf Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Thu, 13 Nov 2025 13:52:51 +0000
Subject: [PATCH 2/2] Add configurable redirect URI field to fix INVALID_CLIENT
 error

Co-authored-by: raphaelbleier <75416341+raphaelbleier@users.noreply.github.com>
---
 app/routes/web.py        |  3 +++
 app/templates/index.html |  9 +++++++++
 tests/test_config.py     | 11 +++++++++++
 3 files changed, 23 insertions(+)

diff --git a/app/routes/web.py b/app/routes/web.py
index 1f046ad..2516c1c 100644
--- a/app/routes/web.py
+++ b/app/routes/web.py
@@ -30,6 +30,7 @@ def index():
             color_method=status.get('color_extraction_method', 'vibrant'),
             spotify_client_id=config.get('SPOTIFY_CLIENT_ID', ''),
             spotify_client_secret=config.get('SPOTIFY_CLIENT_SECRET', ''),
+            spotify_redirect_uri=config.get('SPOTIFY_REDIRECT_URI', 'http://localhost:5000/callback'),
             refresh_interval=config.get('REFRESH_INTERVAL', 30),
             wled_ips=config.get('WLED_IPS', []),
             spotify_authenticated=status.get('spotify_authenticated', False)
@@ -72,6 +73,7 @@ def api_config_update():
         try:
             client_id = request.form.get('client_id', '').strip()
             client_secret = request.form.get('client_secret', '').strip()
+            redirect_uri = request.form.get('redirect_uri', '').strip()
             
             # Validate and convert refresh interval with proper error handling
             try:
@@ -85,6 +87,7 @@ def api_config_update():
             
             config.set('SPOTIFY_CLIENT_ID', client_id)
             config.set('SPOTIFY_CLIENT_SECRET', client_secret)
+            config.set('SPOTIFY_REDIRECT_URI', redirect_uri)
             config.set('REFRESH_INTERVAL', refresh_interval)
             
             config.save()
diff --git a/app/templates/index.html b/app/templates/index.html
index 8919e69..e356153 100644
--- a/app/templates/index.html
+++ b/app/templates/index.html
@@ -138,6 +138,15 @@ <h4 id="trackName" class="mb-2">{{ current_track.name }}</h4>
                         <input type="password" name="client_secret" class="form-control" 
                                value="{{ spotify_client_secret }}" required>
                     </div>
+                    <div class="mb-3">
+                        <label class="form-label">Redirect URI</label>
+                        <input type="text" name="redirect_uri" class="form-control" 
+                               value="{{ spotify_redirect_uri }}" required>
+                        <small class="form-text text-muted">
+                            Must match the redirect URI in your <a href="https://developer.spotify.com/dashboard" target="_blank">Spotify Dashboard</a>. 
+                            Example: <code>http://your-server-ip:5000/callback</code>
+                        </small>
+                    </div>
                     <div class="mb-3">
                         <label class="form-label">Refresh Interval (seconds)</label>
                         <input type="number" name="refresh_interval" class="form-control" 
diff --git a/tests/test_config.py b/tests/test_config.py
index 78fefe0..a88c783 100644
--- a/tests/test_config.py
+++ b/tests/test_config.py
@@ -57,6 +57,17 @@ def test_validation_success(self):
         self.assertTrue(is_valid)
         self.assertEqual(len(errors), 0)
     
+    def test_custom_redirect_uri(self):
+        """Test custom redirect URI configuration"""
+        custom_uri = 'http://192.168.1.50:5000/callback'
+        self.config.set('SPOTIFY_REDIRECT_URI', custom_uri)
+        self.assertEqual(self.config.get('SPOTIFY_REDIRECT_URI'), custom_uri)
+        
+        # Save and reload to verify persistence
+        self.config.save()
+        new_config = Config(config_path=self.temp_file.name)
+        self.assertEqual(new_config.get('SPOTIFY_REDIRECT_URI'), custom_uri)
+    
     def test_validation_missing_credentials(self):
         """Test validation with missing Spotify credentials"""
         self.config.set('SPOTIFY_CLIENT_ID', '')