Skip to content

Cleaned up logging #46

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Cleaned up logging #46

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
66d9b0b
cleaned logging
abaevbog Jun 6, 2023
155b2ca
updated dependencies + removed Promise.delay for to run tests
abaevbog Jun 6, 2023
4d5a622
remaining dependencies updated
abaevbog Jun 7, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 11 additions & 3 deletions lib/citeServer.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -109,7 +109,10 @@ config = argv;
// Set up debug/logging output
log.level = config.logLevel;
log.verbose("npmlog initialized");
log.verbose("Configuration: %j", config);
for (let [param, val] of Object.entries(config) ) {
log.verbose(`Config param: ${param}`, val);
}


var requestCount = 0;

Expand Down Expand Up @@ -168,10 +171,15 @@ var server = http.createServer(function (request, response) {
Promise.resolve().then(function(){
// Parse url from request object, and merge it with default config
parsedQuery = querystring.parse(url.parse(request.url).query);
log.verbose("parsedQuery", parsedQuery);
for (let [param, val] of Object.entries(parsedQuery)) {
log.verbose(`parsedQuery param: ${param}`, val);
}
requestConfig = _.extend({}, defaultRequestConfig, parsedQuery);
requestConfig.locale = localeManager.chooseLocale(requestConfig.locale);
log.verbose("Request configuration:", requestConfig);
for (let [param, val] of Object.entries(requestConfig)) {
log.verbose(`Request configuration: ${param}`, val);
}


//make just memoryUsage response if requested
// FIXME: this should use GET
Expand Down
2 changes: 1 addition & 1 deletion lib/styles.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -147,7 +147,7 @@ exports.CslLoader.prototype.normalizeStyleIdentifier = function(style){
log.verbose("CslLoader", "processStyleIdentifier");

let urlObj = url.parse(style);
log.verbose("urlObj ", urlObj);
log.verbose("urlPath ", urlObj.href);
if(!urlObj.host){
log.verbose("CslLoader", "short name only");
//short name, treat as a zotero.org/styles url
Expand Down
Loading