JsonParser null

[StillLutto]

Bug Report

Hi, it's me again 👋 (recently made a similar post)

Current Behavior

Simply put, the JsonParser gotten from the ClientOptions, the reactive API returns null or even try to copy-paste the default JSON_PARSER from ClientOptions.Builder:

// Attempt #1
println(client.options.jsonParser) // null

// Attempt #2
println(connection.reactive().jsonParser) // null

// Attempt #3
val services: Iterator<JsonParser> = ServiceLoader.load(JsonParser::class.java).iterator()
val result = if (services.hasNext()) services.next() else null
println(result) // null

I'm not well versed in Java's SPI, but the issue seems to be related to that. Futhermore, this is an issue that started happening recently, so I doubt that's the my Gradle environment is the problem, although it could still be. Any ideas?

Environment

Given that I use all of these with Lettuce, this should all be somewhat useful:

api("io.lettuce:lettuce-core:6.8.1.RELEASE")
api("com.fasterxml.jackson.core:jackson-databind:2.20.0")
runtimeOnly("org.jetbrains.kotlinx:kotlinx-coroutines-core-jvm:1.10.2")
api("org.jetbrains.kotlinx:kotlinx-coroutines-core:1.10.2")
api("org.jetbrains.kotlinx:kotlinx-coroutines-reactor:1.10.2")

With a Redis version of 8.2.0-alpine (docker).

[tishun]

Hey @StillLutto ,

The RedisParser is not loaded as a service, right now, but provided as an instance.

Have you consulted the RediJSON guide?

[StillLutto]

@tishun, thanks for the very quick reply. The last attempt is a snippet copied from the Lettuce source code as seen here (in the ClientOptions class):

The issue mostly lies when I use methods that throw an exception since the JsonParser is null, which seems to be related to that block of code above. Here is a longer code snippet that's causing the issue:

val client = RedisClient.create(
    RedisURI.Builder.redis(env("redis.host"), env("redis.port").toInt())
        .withAuthentication(env("redis.username"), env("redis.password"))
        .build()
)
val connection = client.connect()

println(connection.reactive().jsonParser) // returning null

// where I was originally having the issue
RedisManager.commands.jsonGet(key, JsonPath.ROOT_PATH)

[tishun]

My bad, you are correct about the ServiceLoader.

Did you add the dependency to the Jackson library?

Unless you are using a custom parser you need that to parse the data.

[StillLutto]

@tishun, the original message should have all the important dependencies you need. If this is the one that is needed, then yes:

api("com.fasterxml.jackson.core:jackson-databind:2.20.0")

[StillLutto]

Update: manually setting the JsonParser with the DefaultJsonParser class seems to work fine (seems to be the class that is used anyway), although the service itself still seems to be broken.

client.options = ClientOptions.builder()
    .jsonParser { DefaultJsonParser() }
    .build()

[tishun]

@tishun, the original message should have all the important dependencies you need. If this is the one that is needed, then yes:

api("com.fasterxml.jackson.core:jackson-databind:2.20.0")

Oh, my bad. Seems I am very distracted.

Update: manually setting the JsonParser with the DefaultJsonParser class seems to work fine (seems to be the class that is used anyway), although the service itself still seems to be broken.

client.options = ClientOptions.builder()
.jsonParser { DefaultJsonParser() }
.build()

Thanks for getting back on this.

Is the code in #3466 enough to reproduce the issue? If not can you perhaps help with a reproducer?
Ideally we should not have to set the parser manually unless we are overriding it.

[github-actions]

If you would like us to look at this issue, please provide the requested information. If the information is not provided within the next 2 weeks this issue will be closed.