temporalio / temporalio/temporal
Expose zap logger encoding to config
@Ardagan is already working on this.
Since Jul 3, 2021.
- Dominant language
- Go
- Stars
- 23.2k
- Forks
- 1.9k
- Avg merge
- 2d 8h
- Merged PRs (30d)
- 228
Description
Is your feature request related to a problem? Please describe.
Temporal is hard coded to always output logs in json. Our infrastructure handles json-encoded logs but in a specific way that has certain constraints (e.g. low cardinality of field names), therefore these logs are somewhat incompatible with our infra.
Describe the solution you'd like
Would like to be able to use the zap "console" encoder which I assume outputs regular log lines.
Describe alternatives you've considered
It may be possible to get our infra to treat the json encoded logs as plain text but they are not very user friendly.
Additional context
None.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.