I am receiving the following strange dependency injection behavior when using custom HttpInterceptors in angular 5+.
The following simplified code works fine:
I got a similar issue with the same design of an auth service coupled with an interceptor.
@Injectable() AuthInterceptorService {
constructor (authApi: AuthApiService) {}
handle () {...do the job}
}
@Injectable() AuthApiService {
constructor () {
// ...do some setup with a request, such as to get current session
// that leads to an indirect circular between 2 constructors.
}
}
In my case, i found the cause is that I try to start a http request in the auth service's constructor. At that point the injector seems haven't completed the registration of the auth service's instance, while the http client captures the new request and tried to instantiaze the interceptor again, since the previous interceptor instance was stuck in its constructor on the call stack too!
That recursive invoke with two constructors, breaks the singleton pattern of the injector and leads to out of call stack.
you need to add Injector
into constructor and inject AuthService
via injector
export class AuthInterceptor implements HttpInterceptor {
constructor(private inj: Injector) {}
intercept(req: HttpRequest<any>, next: HttpHandler): Observable<HttpEvent<any>> {
const auth = this.inj.get(AuthService);
const token = this.auth.getToken();
return next.handle(req);
}
}
don't forget import
import {Injector} from '@angular/core';
For me @Benjineer answar helped to provide as a dependency in app.module.ts the required sercvice.
Although what is interensting, in the order you add as a dependency in app.module.ts you get it in the constructor of the service.
For Example: in the app module you provide in the order of AuthService, ErroMsgService
deps: [AuthService, ErrorMsgService]
in the HttpInterceptor constructor you have to create them in the same order
constructor(
private authService: AuthService,
private errorService: ErrorMsgService
) {}
If in case, service class is declared in the same file of Injector class then service class should be declared and defined first. Followed by the Injector class where the service class injected as a dependency.
Following above structure does resolves the problem in my case.
Angular Team resolved this issue in Angular 5.2.3 released 31 January 2018. After updating angular version you will be able to inject services that use HTTPClient as normal in constructor
Bug Fixes
common: allow HttpInterceptors to inject HttpClient (#19809) (ed2b717), closes #18224
from Angular changelog
So it turns out that if the service you inject into the Http Interceptor has a dependency on HttpClient
, this leads to a cyclic dependency.
Since my AuthService
was a mix of all different logics (login/out, routing the user, saving/loading tokens, making api calls), I separated the part needed for the interceptors into its own service (just the user credentials & tokens) and now injecting it successfully into the Interceptor.
export class AuthInterceptor implements HttpInterceptor {
constructor(private credentials: CredentialsService) {}
intercept(request: HttpRequest<any>, next: HttpHandler): Observable<HttpEvent<any>> {
const token = this.credentials.getToken();
const api_key = this.credentials.getApiKey();
}
}
export class CredentialsService {
token: string;
user: IUser;
constructor(private http: HttpClient) {
this.loadCredentialsFromStorage();
}
}
This seems to work fine. Hope this helps someone.