[Feature Proposal]: Disable autogeneration of `EventId` parameters

[ZzZombo]

Background and motivation

Now that #33 is fixed and I finally see the SG in action I also see that the package autofills the event ID of the logged methods. Please add a way to suppress that, leaving our original output intact.

Feature Proposal

I suppose a new configuration parameter would work.

Feature Usage

https://github.com/stbychkov/AutoLoggerMessage/wiki/Configuration.

Risks

None?

[stbychkov]

Hey, thanks for the suggestion!

Unfortunately, this isn't something I'm gonna implement because it follows the default behavior of LoggerMessage behind the scenes (see here). Also, LoggerMessage doesn't currently support dynamic event IDs, as discussed here.

While it would be possible to modify the imported LoggerMessage source generator, I'd prefer to keep its behavior aligned with the .NET implementation. That helps avoid maintaining a fork that could drift from upstream as new changes are introduced.

For now, the available options are to provide a static event ID or let it be generated automatically.

[ZzZombo]

Can't you pass a sort of an empty EventId so that it would output [0] in the log just as it was before? While passing meaningful event IDs is something I'd like to do, this is on a backburner for now.

I mean the autogenerated strings are just ugly and overly verbose while not really useful, so please look into letting users to opt out.

[ZzZombo]

@stbychkov, I'm sorry to nag, but please take into consideration my reply above. Thanks for your time.

[stbychkov]

@stbychkov, I'm sorry to nag, but please take into consideration my reply above. Thanks for your time.

Hey, @ZzZombo! No worries, I did see your previous comment.

I only have limited time to work on this project, so I usually get to it on weekends when I have some free time. If it's not some critical bug, the time can be extended, so no SLA and commitments here from me 🙂

If you'd like to help move this feature along faster, contributions are always welcome!

[ZzZombo]

I get it about time, no pressure from me here, just wasn't sure had you seen my message.

[stbychkov]

Hey, I've added 2 parameters where you can specify default event id. It's available in v1.0.17.

Hey! I've added two new configuration parameters that let you specify the default event ID used when one isn't explicitly provided:

• DefaultEventId
• DefaultEventId_Name

You can find the documentation here.

This is available starting with v1.0.17.

[ZzZombo]

Thank you, that's rad!