Makes untrusted domain error a warning.

pierreozoux · pierreozoux · commit e6d9ef2e38da · 2022-01-25T10:33:31.000+01:00
It sends a 400 to the client, so I could even argue that it should be an error.

But currently as an admin, I'm quiet surprised that I get a 400 in the UI, and nothing in the log with the default level.

I saw this commit that explains the reason why info. But I disagree.

Feel free to close the PR if you don't agree with it.

Signed-off-by: Pierre Ozoux &lt;pierre@ozoux.net&gt;
diff --git a/lib/base.php b/lib/base.php
@@ -775,7 +775,7 @@ public static function init() {
 			if (!$isScssRequest) {
 				http_response_code(400);
 
-				\OC::$server->getLogger()->info(
+				\OC::$server->getLogger()->warning(
 					'Trusted domain error. "{remoteAddress}" tried to access using "{host}" as host.',
 					[
 						'app' => 'core',

Original file line number	Diff line number	Diff line change
`@@ -775,7 +775,7 @@ public static function init() {`
`775`	`775`	`if (!$isScssRequest) {`
`776`	`776`	`http_response_code(400);`
`777`	`777`
`778`		`- \OC::$server->getLogger()->info(`
	`778`	`+ \OC::$server->getLogger()->warning(`
`779`	`779`	`'Trusted domain error. "{remoteAddress}" tried to access using "{host}" as host.',`
`780`	`780`	`[`
`781`	`781`	`'app' => 'core',`