Immutable signals

client/stillbox/src/app/login/auth.service.ts

@@ -1,4 +1,11 @@
-import { Injectable, signal, computed, effect, inject, DestroyRef } from '@angular/core';
+import {
+  Injectable,
+  signal,
+  computed,
+  effect,
+  inject,
+  DestroyRef,
+} from '@angular/core';
 import { HttpClient, HttpResponse } from '@angular/common/http';
 import { Router } from '@angular/router';
 import { Observable, Subject } from 'rxjs';
@@ -52,11 +59,11 @@ export class AuthService {
       .pipe(takeUntilDestroyed(this.destroyed))
       .subscribe({
         next: (res) => {
-            let state = <AuthState>{
+          let state = <AuthState>{
-              user: username,
+            user: username,
-              token: res.jwt,
+            token: res.jwt,
-              is_auth: true,
+            is_auth: true,
-            };
+          };
           this._state.set(state);
           this.loginFailed.update(() => false);
           this._router.navigateByUrl('/');
@@ -68,25 +75,17 @@ export class AuthService {
   }
 
   _clearState() {
-this._state.update((state) => {
+    this._state.set(<AuthState>{});
-            state.is_auth = false;
-            state.token = null;
-            return state;
-          });
-
   }
 
   logout() {
-    console.log("logout!");
+    this.http.get('/api/logout', { withCredentials: true }).subscribe({
-    this.http
+      next: (event) => {
-      .get('/api/logout', { withCredentials: true})
+        this._clearState();
-      .subscribe({
+      },
-        next: (event) => {
-          this._clearState();
-                },
       error: (err) => {
-          this._clearState();
+        this._clearState();
-      }
+      },
     });
     this._router.navigateByUrl('/login');
   }
@@ -97,12 +96,14 @@ this._state.update((state) => {
       .pipe(
         tap((event) => {
           if (event.status == 200) {
+            let ost = this._state();
             let tok = event.body?.jwt.toString();
-            this._state.update((state) => {
+            let state = <AuthState>{
-              state.is_auth = true;
+              user: ost.user,
-              state.token = tok ? tok : null;
+              token: tok ? tok : null,
-              return state;
+              is_auth: true,
-            });
+            };
+            this._state.set(state);
           }
         }),
       );