Skip to content

feat: Add :metadata option to Logger middleware#829

Merged
yordis merged 1 commit intoelixir-tesla:masterfrom
balexand:logger_middleware_metadata
Feb 23, 2026
Merged

feat: Add :metadata option to Logger middleware#829
yordis merged 1 commit intoelixir-tesla:masterfrom
balexand:logger_middleware_metadata

Conversation

@balexand
Copy link
Contributor

@balexand balexand commented Feb 21, 2026

This makes it possible to include custom metadata in log messages.

@yordis
Copy link
Member

yordis commented Feb 22, 2026

@balexand hey, could you squash into 1 PR and follow SemConv 🙏🏻

@balexand balexand force-pushed the logger_middleware_metadata branch 2 times, most recently from c931cd9 to dab0f80 Compare February 23, 2026 20:26
@balexand balexand changed the title Add :metadata option to Logger middleware feat: Add :metadata option to Logger middleware Feb 23, 2026
@balexand balexand force-pushed the logger_middleware_metadata branch from dab0f80 to 8ef5709 Compare February 23, 2026 20:55
@balexand balexand force-pushed the logger_middleware_metadata branch from 8ef5709 to dd2e4c0 Compare February 23, 2026 20:59
@balexand
Copy link
Contributor Author

Thanks @yordis! I'm assuming that you mean squash into 1 commit. I've done this.

I also removed the test for the debug log message since this was failing on certain Elixir versions in the CI matrix. I don't think it is necessary or worth the trouble of conditionally asserting based on the Elixir version.

@yordis yordis merged commit 38e209a into elixir-tesla:master Feb 23, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants