search

cnblogs / EnyimMemcachedCore

.NET Memcached client. Available on https://www.nuget.org/packages/EnyimMemcachedCore
Apache License 2.0
162 stars 45 forks source link

Fix PerformStore log message #210

Closed andrew-vant closed 9 months ago

andrew-vant commented 9 months ago

The arguments to this LogError call are out of order. The resulting message outputs "PerformStore" with no cause or trace. Someone previously reported this in issue #61, but it was closed for some reason.

Note, I'm not set up to build enyim myself, so I can't test the fix. You can reproduce the offending message by tripping an exception in this block. One way to do that is to try to store an object that is not marked [serializable] using the BinaryFormatterTranscoder.

I notice tags for 2.x and 3.x. I'm not sure if 2.x is still being maintained, but I based off 2.7.0 since that's what we're using.

cnblogs-dudu commented 9 months ago

Released 3.0.1 and 2.7.1 for this fix.