Skip to content

Commit 989e5e0

Browse files
No need to configure the LoggingContextFilter (#1410)
This hasn't been necessary for 5 years since matrix-org/synapse#8051 because we automatically configure this for you within Synapse itself. Spawning from seeing this fail after we tried to change the `LoggingContextFilter` constructor in element-hq/synapse#18868 (comment). Although this served as a decent canary of what people may have historically configured, I've now added the relevant context to that part of the code as part of element-hq/synapse#18868
1 parent 9fa1325 commit 989e5e0

File tree

1 file changed

+0
-8
lines changed

1 file changed

+0
-8
lines changed

lib/SyTest/Homeserver.pm

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -222,19 +222,11 @@ sub configure_logger
222222
format => "%(asctime)s - %(name)s - %(lineno)d - %(levelname)s - %(request)s - %(message)s"
223223
}
224224
},
225-
226-
filters => {
227-
context => {
228-
"()" => "synapse.logging.context.LoggingContextFilter",
229-
request => ""
230-
}
231-
},
232225
handlers => {
233226
file => {
234227
class => "logging.FileHandler",
235228
formatter => "precise",
236229
filename => "$hs_dir/$log_type.log",
237-
filters => ["context"],
238230
encoding => "utf8"
239231
}
240232
},

0 commit comments

Comments
 (0)