Some auth route guard and link refactoring.

angular.json

@@ -46,7 +46,13 @@
                         "development": {
                             "optimization": false,
                             "extractLicenses": false,
-                            "sourceMap": true
+                            "sourceMap": true,
+                            "fileReplacements": [
+                              {
+                                "replace": "src/environments/environment.ts",
+                                "with": "src/environments/environment.development.ts"
+                              }
+                            ]
                         }
                     },
                     "defaultConfiguration": "production"

src/app/app.routes.ts

@@ -3,6 +3,7 @@ import { RecipePage } from './pages/recipe-page/recipe-page';
 import { RecipesPage } from './pages/recipes-page/recipes-page';
 import { RecipesSearchPage } from './pages/recipes-search-page/recipes-search-page';
 import { RecipeUploadPage } from './pages/recipe-upload-page/recipe-upload-page';
+import { authGuard } from './shared/guards/auth-guard';
 
 export const routes: Routes = [
     {
@@ -16,6 +17,7 @@ export const routes: Routes = [
     {
         path: 'recipe-upload',
         component: RecipeUploadPage,
+        canActivate: [authGuard]
     },
     {
         path: 'recipes/:username/:slug',

src/app/endpoints.ts

@@ -1,3 +1,6 @@
 export const Endpoints = {
+    authLogin: 'auth/login',
+    authLogout: 'auth/logout',
+    authRefresh: 'auth/refresh',
     recipes: 'recipes',
 };

src/app/shared/components/nav/NavLinkConfig.ts

@@ -0,0 +1,5 @@
+export interface NavLinkConfig {
+    relativeUrl: string;
+    title: string;
+    disabled?: boolean;
+}

src/app/shared/components/nav/nav.html

@@ -1,8 +1,10 @@
 <nav>
     <h2>Nav</h2>
     <ul>
-        <li><a [routerLink]="'/'">Browse Recipes</a></li>
-        <li><a [routerLink]="'/recipes-search'">Search Recipes</a></li>
-        <li><a [routerLink]="'/recipe-upload'">Upload Recipe</a></li>
+        @for (link of links(); track link.relativeUrl) {
+            @if (!link.disabled) {
+                <li><a [routerLink]="link.relativeUrl">{{ link.title }}</a></li>
+            }
+        }
     </ul>
 </nav>

src/app/shared/components/nav/nav.ts

@@ -1,5 +1,7 @@
-import { Component } from '@angular/core';
+import { Component, computed, inject } from '@angular/core';
 import { RouterLink } from '@angular/router';
+import { AuthService } from '../../services/AuthService';
+import { NavLinkConfig } from './NavLinkConfig';
 
 @Component({
     selector: 'app-nav',
@@ -7,4 +9,26 @@ import { RouterLink } from '@angular/router';
     templateUrl: './nav.html',
     styleUrl: './nav.css',
 })
-export class Nav {}
+export class Nav {
+
+    private readonly authService = inject(AuthService);
+
+    protected readonly links = computed<NavLinkConfig[]>(() => {
+        return [
+            {
+                relativeUrl: '/',
+                title: 'Browse Recipes',
+            },
+            {
+                relativeUrl: '/recipes-search',
+                title: 'Search Recipes',
+            },
+            {
+                relativeUrl: '/recipe-upload',
+                title: 'Upload Recipe',
+                disabled: this.authService.accessToken() === null
+            }
+        ];
+    });
+
+}

src/app/shared/guards/auth-guard.spec.ts

@@ -0,0 +1,17 @@
+import { TestBed } from '@angular/core/testing';
+import { CanActivateFn } from '@angular/router';
+
+import { authGuard } from './auth-guard';
+
+describe('authGuardGuard', () => {
+  const executeGuard: CanActivateFn = (...guardParameters) =>
+      TestBed.runInInjectionContext(() => authGuard(...guardParameters));
+
+  beforeEach(() => {
+    TestBed.configureTestingModule({});
+  });
+
+  it('should be created', () => {
+    expect(executeGuard).toBeTruthy();
+  });
+});

src/app/shared/guards/auth-guard.ts

@@ -0,0 +1,8 @@
+import { CanActivateFn } from '@angular/router';
+import { inject } from '@angular/core';
+import { AuthService } from '../services/AuthService';
+
+export const authGuard: CanActivateFn = (route, state) => {
+    const authService = inject(AuthService);
+    return authService.accessToken() !== null;
+};

src/app/shared/services/AuthService.ts

@@ -4,6 +4,7 @@ import { LoginView } from '../models/LoginView.model';
 import { firstValueFrom, Observable, tap } from 'rxjs';
 import { QueryClient } from '@tanstack/angular-query-experimental';
 import { Router } from '@angular/router';
+import { EndpointService } from './EndpointService';
 
 @Injectable({
     providedIn: 'root',
@@ -18,11 +19,12 @@ export class AuthService {
     private readonly http = inject(HttpClient);
     private readonly queryClient = inject(QueryClient);
     private readonly router = inject(Router);
+    private readonly endpointService = inject(EndpointService);
 
     public async login(username: string, password: string): Promise<LoginView> {
         const loginView = await firstValueFrom(
             this.http.post<LoginView>(
-                'http://localhost:8080/auth/login',
+                this.endpointService.getUrl('authLogin'),
                 { username, password },
                 {
                     withCredentials: true,
@@ -36,7 +38,7 @@ export class AuthService {
     }
 
     public async logout(): Promise<void> {
-        await firstValueFrom(this.http.post('http://localhost:8080/auth/logout', null));
+        await firstValueFrom(this.http.post(this.endpointService.getUrl('authLogout'), null));
         this._username.set(null);
         this._accessToken.set(null);
         await this.router.navigate(['/']);
@@ -47,7 +49,7 @@ export class AuthService {
         this._accessToken.set(null);
         this._username.set(null);
         return this.http
-            .post<LoginView>('http://localhost:8080/auth/refresh', null, {
+            .post<LoginView>(this.endpointService.getUrl('authRefresh'), null, {
                 withCredentials: true,
             })
             .pipe(

src/app/shared/services/EndpointService.ts

@@ -1,12 +1,13 @@
 import { Injectable } from '@angular/core';
 import { Endpoints } from '../../endpoints';
 import { QueryParams } from '../models/Query.model';
+import { environment } from '../../../environments/environment';
 
 @Injectable({
     providedIn: 'root',
 })
 export class EndpointService {
-    public getUrl(endpoint: keyof typeof Endpoints, pathParams?: string[], queryParams?: QueryParams): string {
+    public getUrl(endpoint: keyof typeof Endpoints, pathParts?: string[], queryParams?: QueryParams): string {
         const urlSearchParams = new URLSearchParams();
         if (queryParams?.page !== undefined) {
             urlSearchParams.set('page', queryParams.page.toString());
@@ -29,7 +30,7 @@ export class EndpointService {
             }
         });
 
-        let pathString = pathParams?.join('/') || '';
+        let pathString = pathParts?.join('/') || '';
         if (pathString?.length) {
             pathString = '/' + pathString;
         }
@@ -39,6 +40,6 @@ export class EndpointService {
             queryString = '?' + queryString;
         }
 
-        return `http://localhost:8080/${Endpoints[endpoint]}${pathString}${queryString}`;
+        return environment.apiBaseUrl + '/' + Endpoints[endpoint] + pathString + queryString;
     }
 }

src/app/shared/services/RecipeService.ts

@@ -19,24 +19,29 @@ export class RecipeService {
 
     public getRecipes(): Promise<Recipe[]> {
         return firstValueFrom(
-            this.http.get<RecipeInfoViews>('http://localhost:8080/recipes').pipe(map((res) => res.content)),
+            this.http.get<RecipeInfoViews>(this.endpointService.getUrl('recipes'))
+                .pipe(map((res) => res.content))
         );
     }
 
     public getRecipeView(username: string, slug: string): Promise<RecipeView> {
-        return firstValueFrom(this.http.get<RecipeView>(`http://localhost:8080/recipes/${username}/${slug}`));
+        return firstValueFrom(this.http.get<RecipeView>(
+            this.endpointService.getUrl('recipes', [username, slug])
+        ));
     }
 
-    private getRecipeUrl(recipeView: RecipeView): string {
-        return `http://localhost:8080/recipes/${recipeView.recipe.owner.username}/${recipeView.recipe.slug}`;
+    private getRecipeBaseUrl(recipeView: RecipeView): string {
+        return this.endpointService.getUrl(
+            'recipes', [recipeView.recipe.owner.username, recipeView.recipe.slug]
+        );
     }
 
     public async toggleStar(recipeView: RecipeView): Promise<void> {
         if (this.authService.accessToken()) {
             if (recipeView.isStarred) {
-                await lastValueFrom(this.http.delete(this.getRecipeUrl(recipeView) + '/star'));
+                await lastValueFrom(this.http.delete(this.getRecipeBaseUrl(recipeView) + '/star'));
             } else {
-                await lastValueFrom(this.http.post(this.getRecipeUrl(recipeView) + '/star', null));
+                await lastValueFrom(this.http.post(this.getRecipeBaseUrl(recipeView) + '/star', null));
             }
             await this.queryClient.invalidateQueries({
                 queryKey: ['recipe', recipeView.recipe.owner.username, recipeView.recipe.slug],
@@ -56,9 +61,10 @@ export class RecipeService {
 
     public async addComment(username: string, slug: string, commentText: string): Promise<RecipeComment> {
         const comment = await firstValueFrom(
-            this.http.post<RecipeComment>(`http://localhost:8080/recipes/${username}/${slug}/comments`, {
-                text: commentText,
-            }),
+            this.http.post<RecipeComment>(
+                this.endpointService.getUrl('recipes', [username, slug, 'comments']),
+                { text: commentText }
+            )
         );
         await this.queryClient.invalidateQueries({
             queryKey: ['recipeComments', username, slug],

src/environments/environment.development.ts

@@ -0,0 +1,3 @@
+export const environment = {
+    apiBaseUrl: 'http://localhost:8080'
+};

src/environments/environment.ts

@@ -0,0 +1,3 @@
+export const environment = {
+    apiBaseUrl: 'http://localhost:8080'
+};