From baf68889f98b48c25924aae9ddc0275e88796399 Mon Sep 17 00:00:00 2001
From: Ruben Verborgh <ruben@verborgh.org>
Date: Sun, 29 Nov 2020 10:53:43 +0100
Subject: [PATCH] change: Do not warn in canHandle.

---
 src/ldp/http/BasicResponseWriter.ts               | 1 -
 src/ldp/http/ErrorResponseWriter.ts               | 1 -
 src/ldp/http/SparqlUpdateBodyParser.ts            | 1 -
 src/ldp/permissions/MethodPermissionsExtractor.ts | 4 ----
 4 files changed, 7 deletions(-)

diff --git a/src/ldp/http/BasicResponseWriter.ts b/src/ldp/http/BasicResponseWriter.ts
index eef69a2d87..8bcddad135 100644
--- a/src/ldp/http/BasicResponseWriter.ts
+++ b/src/ldp/http/BasicResponseWriter.ts
@@ -21,7 +21,6 @@ export class BasicResponseWriter extends ResponseWriter {
 
   public async canHandle(input: { response: HttpResponse; result: ResponseDescription | Error }): Promise<void> {
     if (input.result instanceof Error || input.result.metadata?.contentType === INTERNAL_QUADS) {
-      this.logger.warn('This writer only supports binary ResponseDescriptions');
       throw new NotImplementedHttpError('Only successful binary responses are supported');
     }
   }
diff --git a/src/ldp/http/ErrorResponseWriter.ts b/src/ldp/http/ErrorResponseWriter.ts
index 226f1a52bd..34e5c8aae8 100644
--- a/src/ldp/http/ErrorResponseWriter.ts
+++ b/src/ldp/http/ErrorResponseWriter.ts
@@ -13,7 +13,6 @@ export class ErrorResponseWriter extends ResponseWriter {
 
   public async canHandle(input: { response: HttpResponse; result: ResponseDescription | Error }): Promise<void> {
     if (!(input.result instanceof Error)) {
-      this.logger.warn('This writer can only write errors');
       throw new NotImplementedHttpError('Only errors are supported');
     }
   }
diff --git a/src/ldp/http/SparqlUpdateBodyParser.ts b/src/ldp/http/SparqlUpdateBodyParser.ts
index 8f5244b013..45b2501a3c 100644
--- a/src/ldp/http/SparqlUpdateBodyParser.ts
+++ b/src/ldp/http/SparqlUpdateBodyParser.ts
@@ -20,7 +20,6 @@ export class SparqlUpdateBodyParser extends BodyParser {
   public async canHandle({ request }: BodyParserArgs): Promise<void> {
     const contentType = request.headers['content-type'];
     if (contentType !== APPLICATION_SPARQL_UPDATE) {
-      this.logger.debug(`Unsupported content type: ${contentType}`);
       throw new UnsupportedMediaTypeHttpError('This parser only supports SPARQL UPDATE data.');
     }
   }
diff --git a/src/ldp/permissions/MethodPermissionsExtractor.ts b/src/ldp/permissions/MethodPermissionsExtractor.ts
index 59e69e6493..2a06c3db8b 100644
--- a/src/ldp/permissions/MethodPermissionsExtractor.ts
+++ b/src/ldp/permissions/MethodPermissionsExtractor.ts
@@ -1,4 +1,3 @@
-import { getLoggerFor } from '../../logging/LogUtil';
 import { NotImplementedHttpError } from '../../util/errors/NotImplementedHttpError';
 import type { Operation } from '../operations/Operation';
 import type { PermissionSet } from './PermissionSet';
@@ -13,11 +12,8 @@ const SUPPORTED_METHODS = new Set([ ...READ_METHODS, ...WRITE_METHODS ]);
  * Specifically: GET, HEAD, POST, PUT and DELETE.
  */
 export class MethodPermissionsExtractor extends PermissionsExtractor {
-  protected readonly logger = getLoggerFor(this);
-
   public async canHandle({ method }: Operation): Promise<void> {
     if (!SUPPORTED_METHODS.has(method)) {
-      this.logger.warn(`Unrecognized method ${method}`);
       throw new NotImplementedHttpError(`Cannot determine permissions of ${method}`);
     }
   }