Skip to content

Remove flag "skipParentEntryForCloudRun" to skip writing RequestLogEntry for cloud run by default. #824

@cindy-peng

Description

@cindy-peng

Thanks for stopping by to let us know something could be better!

PLEASE READ: If you have a support contract with Google, please create an issue in the support console instead of filing on GitHub. This will ensure a timely response.

Is your feature request related to a problem? Please describe.
This is related to the fix for Possible replication of bug from nodejs-logging-bunyan.
We fixed the duplicate logging entry for cloud run in 821 with optional flag "skipParentEntryForCloudRun" enabled to avoid breaking changes. This request is to track removing optional flag "skipParentEntryForCloudRun" and make the fix default behavior for cloud run in the next major release.

Describe the solution you'd like
Remove optional parameter "skipParentEntryForCloudRun" and related unit tests for existing makeMiddleware function

Describe alternatives you've considered

Additional context
Similar issue also exists for repo nodejs-logging-bunyan mentioned in issue 626

Metadata

Metadata

Assignees

Labels

api: loggingIssues related to the googleapis/nodejs-logging-winston API.priority: p3Desirable enhancement or fix. May not be included in next release.type: feature request‘Nice-to-have’ improvement, new feature or different behavior or design.

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions