Clean up full errors being logged (#174440)

## Summary

Fix a few logging calls where the full `Error` object was being logged,
either as string representation in the message, or as log meta.
This commit is contained in:
Pierre Gayvallet 2024-01-09 11:35:12 +01:00 committed by GitHub
parent de961a54a7
commit 8186fc7300
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
5 changed files with 11 additions and 9 deletions

View file

@ -285,8 +285,10 @@ describe('getPayloadSize', () => {
} as unknown as Response,
logger
);
expect(logger.warn.mock.calls[0][0]).toMatchInlineSnapshot(
`"Failed to calculate response payload bytes."`
);
expect(logger.warn.mock.calls[0][0]).toMatchInlineSnapshot(`
"Failed to calculate response payload bytes: Converting circular structure to JSON
--> starting at object with constructor 'Object'
--- property 'circular' closes the circle"
`);
});
});

View file

@ -81,7 +81,7 @@ export function getResponsePayloadBytes(response: Response, log: Logger): number
// We intentionally swallow any errors as this information is
// only a nicety for logging purposes, and should not cause the
// server to crash if it cannot be determined.
log.warn('Failed to calculate response payload bytes.', e);
log.warn(`Failed to calculate response payload bytes: ${e.message}`);
}
return undefined;

View file

@ -97,7 +97,7 @@ export const convertModelVersionTransformFn = ({
const result = modelTransformFn(doc, context);
return { transformedDoc: result.document, additionalDocs: [] };
} catch (error) {
log.error(error);
log.error(`Error trying to transform document: ${error.message}`);
throw new TransformSavedObjectDocumentError(error, virtualVersion);
}
};

View file

@ -56,7 +56,7 @@ export function convertMigrationFunction(
return { transformedDoc: result, additionalDocs: [] };
} catch (error) {
log.error(error);
log.error(`Error trying to transform document: ${error.message}`);
throw new TransformSavedObjectDocumentError(error, version);
}
};

View file

@ -37,7 +37,7 @@ export async function bootstrap() {
server = new Server({ config: configService, logger });
serverStart = await server.start();
} catch (e) {
log.error(`Failed to start Server: ${e}`);
log.error(`Failed to start Server: ${e.message}`);
process.exit(1);
}
@ -46,7 +46,7 @@ export async function bootstrap() {
kibanaService = new KibanaService({ config: configService, logger });
await kibanaService.start({ server: serverStart });
} catch (e) {
log.error(`Failed to start Kibana service: ${e}`);
log.error(`Failed to start Kibana service: ${e.message}`);
process.exit(1);
}
@ -58,7 +58,7 @@ export async function bootstrap() {
};
process.on('unhandledRejection', async (err: Error) => {
log.error(err);
log.error(`Unhandled rejection: ${err.message}`);
await attemptGracefulShutdown(1);
});