Possible Bug: `.setPropertyNamingStrategy(PropertyNamingStrategies.SNAKE_CASE)` doesnt respect the naming strategy

Describe the bug

I am declaring a global ObjectMapper as such:

val OBJECT_MAPPER = ObjectMapper()
    .setPropertyNamingStrategy(PropertyNamingStrategies.SNAKE_CASE)
    .registerKotlinModule()
    .registerModule(JavaTimeModule())
    .registerModule(ProtobufModule())
    .configure(DeserializationFeature.FAIL_ON_UNKNOWN_PROPERTIES, false)
    .setSerializationInclusion(JsonInclude.Include.NON_NULL)
    .disable(SerializationFeature.WRITE_DATES_AS_TIMESTAMPS)

This worked prior to upgrade to 2.12.5 using

val OBJECT_MAPPER = ObjectMapper()
    .setPropertyNamingStrategy(PropertyNamingStrategy.SNAKE_CASE)  <---- notice the deprecated usage
    ...

However, now when I use something like fun GeneratedMessageV3.toProcedureObject() = OBJECT_MAPPER.writeValueAsString(this), the naming strategy is no longer respected. It is using camelCase instead.

I cant figure out why; Ive tracked it to where it shows the naming strategy (which shows correctly), but thats as far I was able to get.

Version information

2.12.5, upgrading from 2.11.4

To Reproduce Using the above setup, it doesnt serialize correctly.

Additional context

Kotlin 1.5.31

[minxylynx]

@michielboekhoff I agree. I opened up an issue to release out jackson-datatype-protobuf with the latest of whats in there.

[michielboekhoff]

@minxylynx -

I’m pretty sure this is an issue with the ProtobufModule, which is something we ran into as well. Specifically, support for the new PropertyNamingStrategies has been implemented in master (see also here) however, it has not been released to mvncentral yet.

Therefore, I think this issue can actually be closed & an issue should be opened against the repo for jackson-datatype-protobuf.