googleapis / nodejs-logging-winston

Node.js client integration between Stackdriver Logging and Winston.
https://cloud.google.com/logging/
Apache License 2.0
105 stars 50 forks source link

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

Open cindy-peng opened 10 months ago

cindy-peng commented 10 months ago

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