PR Comments

Author: anishfyle

src/app/app.module.ts

@@ -18,7 +18,6 @@ import { SharedModule } from './shared/shared.module';
 import { RippleModule } from 'primeng/ripple';
 import { BrandingService } from './core/services/common/branding.service';
 import { Sage300ConfigurationModule } from './integrations/sage300/sage300-main/sage300-configuration/sage300-configuration.module';
-import { HttpErrorInterceptor } from './core/interceptor/http-error.interceptor';
 
 @NgModule({
   declarations: [
@@ -47,11 +46,6 @@ import { HttpErrorInterceptor } from './core/interceptor/http-error.interceptor'
       useClass: JwtInterceptor,
       multi: true
     },
-    {
-      provide: HTTP_INTERCEPTORS,
-      useClass: HttpErrorInterceptor,
-      multi: true
-    },
     {
       provide: ErrorHandler,
       useClass: GlobalErrorHandler

src/app/core/interceptor/http-error.interceptor.spec.ts

@@ -26,12 +26,18 @@ export class ApiService {
   }
 
   private handleError(error: HttpErrorResponse, httpMethod: string, url: string) {
-    if (error.error instanceof ErrorEvent) {
-      console.error('An error occurred:', error.error.message);
+    if (httpMethod === 'POST') {
+      if (error.status >= 500) {
+        console.error(`POST error ${error.status}: ${error.message}`);
+      }
+    } else if (httpMethod === 'GET') {
+      if (error.status !== 404) {
+        console.error(`GET error ${error.status}: ${error.message}`);
+      }
+      // Else (404 GET error) - do not log
     } else {
-      console.error(
-        `Backend returned code ${error.status}, url was: ${url} method was: ${httpMethod}, body was: ${JSON.stringify(error.error)}`
-      );
+      // For other HTTP methods, you can add custom logic or just log the errors
+      console.error(`HTTP ${httpMethod} error ${error.status}: ${error.message}`);
     }
     return throwError(error);
   }