moon
/
enigma-bbs
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Some logging updates

This commit is contained in:
Bryan Ashby 2018-12-17 11:20:14 -07:00
parent e0f6847581
commit b1eea4f4b7
1 changed files with 9 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
core/servers/content/nntp.js
View File

@ -578,7 +578,13 @@ class NNTPServer extends NNTPServerBase {
});
if(!value) {
this.log.trace(`No value for requested header field "${field}"`);
//
// Clients will check some headers just to see if they exist.
// Don't spam logs with these. For others, it's good to know.
//
if(!['references', 'xref'].includes(field)) {
this.log.trace(`No value for requested header field "${field}"`);
}
}
return value;
@ -588,8 +594,9 @@ class NNTPServer extends NNTPServerBase {
return super._getOverviewFmt(session);
}
_getNewNews(/*session, time, wildmat*/) {
_getNewNews(session, time, wildmat) {
// Currently seems pointless to implement. No semi-modern clients seem to use it anyway.
this.log.debug( { time, wildmat }, 'Request made using unsupported NEWNEWS command');
throw new Errors.Invalid('NEWNEWS is not enabled on this server');
}