Mike Goodfellow

Results 33 comments of Mike Goodfellow

Latest: https://github.com/TurnerSoftware/CacheTower/blob/main/src/CacheTower.Serializers.Protobuf/ProtobufCacheSerializer.cs#L21 This is saying 1 - file data, 2 - expiry data? Pre PR it was: https://github.com/TurnerSoftware/CacheTower/pull/172/files#diff-972f6f8307717fcd20b6599842d83ab73931e2aa4b483e5076ab39d5f5109e0dL18 1 - Expiry, 2 - data It was swapped :(

> I'm not quite sure how this type serializes without the ProtoXXX attributes on the class, but magically it does Aha, now I know how that bit of magic works...

Oh no! ![image](https://user-images.githubusercontent.com/6501186/195344462-2cfade12-a414-49c4-bed0-e1bdab61bfb9.png) It's ... both?

I guess in the short term I could swap in my own version of the ProtobufSerializer which tries to deserialize one way, then tries the other if the first fails....

I had only just got around to pulling the repo by the time you had already written the test - thanks for that ❤️ - bittersweet to see the result...

Wow great spot - really good find! Strange it serializes as surely it wouldn't match the RuntimeConfig rather than just ignoring it entirely and serializing in the wrong positions. I...

🤯 So the setter is required for the RuntimeTypeModel configuration approach, but not required for the attribute approach it seems?

Very interesting! Thanks for the investigation, glad we have an answer for the strangeness now. There isn't any rush for the changes, I was only just getting around to upgrading...

Under "Frameworks & Extensions" use `No wrap - in ` or `No wrap - in ` instead of `onLoad`. Click Run again, and it works fine for me. If you...