Začištění použití remote-user hlavičky

server/src/auth.ts

@@ -6,13 +6,16 @@ import jwt from 'jsonwebtoken';
  * @param login přihlašovací jméno uživatele
  * @returns JWT token
  */
-export function generateToken(login: string): string {
+export function generateToken(login?: string): string {
     if (!process.env.JWT_SECRET) {
         throw Error("Není vyplněna proměnná prostředí JWT_SECRET");
     }
     if (process.env.JWT_SECRET.length < 32) {
         throw Error("Proměnná prostředí JWT_SECRET musí být minimálně 32 znaků");
     }
+    if (!login || login.trim().length === 0) {
+        throw Error("Nebyl předán login");
+    }
     return jwt.sign({ login }, process.env.JWT_SECRET);
 }
 

server/src/index.ts

@@ -38,29 +38,30 @@ const parseToken = (req: any) => {
 
 // ----------- Metody nevyžadující token --------------
 
-app.get("/api/whoami",(req,res)=>{
+app.get("/api/whoami", (req, res) => {
     res.send(req.header('remote-user'));
 })
 
 app.post("/api/login", (req, res) => {
-    if (!req.body?.login) {
+    // Autentizace pomocí trusted headers
+    const remoteUser = req.header('remote-user');
+    if (remoteUser && remoteUser.length > 0) {
+        res.status(200).json(generateToken(remoteUser));
+        return;
+    }
+    // Klasická autentizace loginem
+    if (!req.body?.login || req.body.login.trim().length === 0) {
         throw Error("Nebyl předán login");
     }
-    // TODO: je tohle hnusny?... bude to fungovat? lol
+    // TODO zavést podmínky pro délku loginu (min i max)
-    if (req.header('remote-user')){
+    res.status(200).json(generateToken(req.body.login));
-        let username = req.header('remote-user') || "jmenonemahlavicku"
-        res.status(200).json(generateToken(username));
-    } else {
-        // TODO zavést podmínky pro délku loginu (min i max)
-        res.status(200).json(generateToken(req.body.login));
-    }
 });
 
 // ----------------------------------------------------
 
 /** Middleware ověřující JWT token */
 app.use((req, res, next) => {
-    if (req.header('remote-user')){
+    if (req.header('remote-user')) {
         console.log("Tvuj username: %s.", req.header('remote-user'));
     }
     if (!req.headers.authorization) {