configureRequestLog example code has a bug

[smackfu]

🐞 Bug Report

Describe the bug

In this example code, the log parameter is defaulted to an empty object, but then line 28 references clonedLog.request.metaData which is not defined.

one-app/prod-sample/sample-modules/frank-lloyd-root/0.0.0/src/config.js

Lines 24 to 28 in 7b9858c

	configureRequestLog: ({ req, log = {} }) => {
	const clonedLog = JSON.parse(JSON.stringify(log));
	const { cookies } = req;
	clonedLog.request.metaData = {

To Reproduce

If you write a test for this code and do not not pass the log variable, then you will get the following error:

TypeError: Cannot read property 'metaData' of undefined
    at Object.configureRequestLog

Expected behavior

Example code should be valid. Possible solutions:

1. Update the default value of log.
2. Add a check for log.request not being defined.
3. Remove the default value of log.

[oneamexbot]

Hey @smackfu! We really appreciate you taking the time to report an issue. The collaborators on this project attempt to help as many people as possible, but we're a limited number of volunteers, so it's possible this won't be addressed swiftly.

If you need any help, you can ask in our GitHub Discussions."

[github-actions]

This issue is stale because it has been open 30 days with no activity.

[10xLaCroixDrinker]

closed by #647