HttpInterceptor pipe error in Angular 10 in catchError() function

1
votes

Argument of type '(error: any) => void' is not assignable to parameter of type '(err: any, caught: Observable<HttpEvent>) => ObservableInput'. Type 'void' is not assignable to type 'ObservableInput'.

enter image description here

Code

import { HttpErrorResponse, HttpEvent, HttpHandler, HttpInterceptor, HttpRequest, HttpResponse } from '@angular/common/http';
import { Injectable } from '@angular/core';
import { Observable, throwError } from 'rxjs';
import { catchError, map, skip, retry, tap } from 'rxjs/operators';


@Injectable()
export class AuthenticationInterceptor implements HttpInterceptor {

    API_SERVER_BASE_URL = "http://localhost:5000/api/v1";
    urlsToNotUse: Array<string>;

    constructor() {
        this.urlsToNotUse = [
            '/user/login',
            .....
        ];
    }


    intercept(req: HttpRequest<any>, next: HttpHandler): Observable<HttpEvent<any>> {
        const ACCESS_TOKEN = localStorage.getItem("accessToken");
        // if accesstoken is not null
        if (ACCESS_TOKEN && this.isValidRequestForInterceptor(req.url)) {
         .........
         .........
        } else {
            const reqWithoutAuth = req.clone({
                url: this.API_SERVER_BASE_URL + req.url,
            });
            return next.handle(reqWithoutAuth)
                .pipe(
                    tap(evt => { }),
                    catchError(error => {
                        console.log(error);
                     })
                );
        }
    }


    private isValidRequestForInterceptor(requestUrl: string): boolean {
        return !this.urlsToNotUse.includes(requestUrl);
    }

}
2
angularrxjsangular-http-interceptors
Can you share the code since it's not possible to see through picture :) – Bozhinovski
you have to return an observable (like return throwError(...) at line 41 in your picture. – griFlo

2 Answers

1
votes

Make sure you import throwError

import { throwError } from 'rxjs';

and in the end you return the Observable. After console.log() write

return .pipe(
                tap(evt => { }),
                catchError(error => {
                    console.log(error);
                    return throwError(error); // add this line
                 })
            );
0
votes
import { throwError } from 'rxjs';
import { catchError } from 'rxjs/operators';

Also dont forget the catchError N.B ~tap~ has been depricated, use map insted

return [Observable].pipe(
      // We use the map operator to change the data from the  response
      map(value => {
      }),
      catchError(error => {
        return throwError(error)
      }
})