search

houseabsolute / Log-Dispatch

Dispatches messages to one or more outputs
https://metacpan.org/release/Log-Dispatch/
Other
12 stars 29 forks source link

2.60 throws an exception if the message is a non-empty string #38

Closed oschwald closed 7 years ago

oschwald commented 7 years ago

This is a reasonably significant change in behavior that caused a number of breakages for us. I don't think the behavior is necessarily wrong, but it is a breaking change without being mentioned in the Changes and without an appropriate version bump.

autarch commented 7 years ago

Ah, that wasn't really intentional. I'll change it to accept an empty string again.

oschwald commented 7 years ago

After looking more closely into the error, I think the equivalent ident change is what is actually causing the most issues, primarily in test code.

oschwald commented 7 years ago

Thanks for fixing that!