auth code changes / misc

src/components/Login.jsx

@@ -2,6 +2,17 @@ import React, { useState, useRef, useEffect } from "react";
 import { toast } from "react-toastify";
 import { API_URL } from "@/config";
 
+function getCookie(name) {
+    const value = `; ${document.cookie}`;
+    const parts = value.split(`; ${name}=`);
+    if (parts.length === 2) return decodeURIComponent(parts.pop().split(';').shift());
+    return null;
+}
+
+function clearCookie(name) {
+    document.cookie = `${name}=; Max-Age=0; path=/;`;
+}
+
 export default function LoginPage() {
     const [username, setUsername] = useState("");
     const [password, setPassword] = useState("");
@@ -20,32 +31,24 @@ export default function LoginPage() {
         setLoading(true);
 
         try {
-            if (!username) {
+            if (!username || !password) {
                 setLoading(false);
-                if (!toast.isActive("login-username-required-toast")) {
-                    return toast.error("Username and password are required",
-                        {
-                            toastId: "login-missing-data-toast",
-                        });
-                }
-            }
-            if (!password) {
-                setLoading(false);
-                if (!toast.isActive("login-password-required-toast")) {
-                    return toast.error("Username and password are required",
-                        {
-                            toastId: "login-missing-data-toast",
-                        });
+                if (!toast.isActive("login-missing-data-toast")) {
+                    toast.error("Username and password are required", {
+                        toastId: "login-missing-data-toast",
+                    });
                 }
+                return;
             }
 
-            const formData = new URLSearchParams();
-            formData.append("username", username);
-            formData.append("password", password);
-            formData.append("grant_type", "password");
-            formData.append("scope", "");
-            formData.append("client_id", "");
-            formData.append("client_secret", "");
+            const formData = new URLSearchParams({
+                username,
+                password,
+                grant_type: "password",
+                scope: "",
+                client_id: "",
+                client_secret: "",
+            });
 
             const resp = await fetch(`${API_URL}/auth/login`, {
                 method: "POST",
@@ -55,52 +58,40 @@ export default function LoginPage() {
             });
 
             if (resp.status === 401) {
-                if (!toast.isActive("login-error-invalid-toast")) {
-                    toast.error("Invalid username or password", {
-                        toastId: "login-error-invalid-toast",
-                    });
-                }
+                toast.error("Invalid username or password", {
+                    toastId: "login-error-invalid-toast",
+                });
                 setLoading(false);
                 return;
             }
 
             if (!resp.ok) {
                 const data = await resp.json().catch(() => ({}));
-                if (!toast.isActive("login-error-failed-toast")) {
-                    toast.error(data.detail ? `Login failed: ${data.detail}` : "Login failed",
-                        {
-                            toastId: "login-error-failed-toast",
-                        });
-                }
+                toast.error(data.detail ? `Login failed: ${data.detail}` : "Login failed", {
+                    toastId: "login-error-failed-toast",
+                });
                 setLoading(false);
                 return;
             }
 
             const data = await resp.json();
             if (data.access_token) {
-                if (!toast.isActive("login-success-toast")) {
-                    toast.success("Login successful!",
-                        {
-                            toastId: "login-success-toast",
-                        });
-                }
-                window.location.href = "/TRip"; // TODO: fix, hardcoded
+                toast.success("Login successful!", {
+                    toastId: "login-success-toast",
+                });
+                const returnTo = getCookie("returnTo") || "/TRip";
+                clearCookie("returnTo");
+                window.location.href = returnTo;
             } else {
-                if (!toast.isActive("login-error-no-token-toast")) {
-                    toast.error("Login failed: no access token received",
-                        {
-                            toastId: "login-error-no-token-toast",
-                        });
-                    setLoading(false);
-                }
+                toast.error("Login failed: no access token received", {
+                    toastId: "login-error-no-token-toast",
+                });
+                setLoading(false);
             }
         } catch (error) {
-            if (!toast.isActive("login-error-network-toast")) {
-                toast.error("Network error during login",
-                    {
-                        toastId: "login-error-network-toast",
-                    });
-            }
+            toast.error("Network error during login", {
+                toastId: "login-error-network-toast",
+            });
             console.error("Login error:", error);
             setLoading(false);
         }

src/components/TRip/MediaRequestForm.jsx

@@ -283,7 +283,9 @@ export default function MediaRequestForm() {
                 link.href = url;
 
                 const sanitize = (str) => str.replace(/[\\/:*?"<>|]/g, "_");
-                const filename = `${sanitize(artist)} - ${sanitize(title)}.flac`;
+                const urlPath = new URL(data.stream_url).pathname;
+                const extension = urlPath.split('.').pop().split('?')[0] || 'flac';
+                const filename = `${sanitize(artist)} - ${sanitize(title)}.${extension}`;
 
                 link.download = filename;
                 document.body.appendChild(link);