vertx-redis-client icon indicating copy to clipboard operation
vertx-redis-client copied to clipboard

Prune "null" support

Open pmlopes opened this issue 3 years ago • 1 comments

Redis doesn't allow null in requests. Currently for compatibility reasons we encode null as the String "null". In order to make the behavior consistent with jedis the official client, we should throw an IllegalArgumentException in that case.

In order to avoid breaking changes, for now we should have a JVM flag to enforce it as strict null:

-Dio.vertx.redis.request.null-as-string=true
  • deprecate all nullArg()
  • if the flag is false during the request encoding if a null is present throw IAE

pmlopes avatar Mar 12 '21 10:03 pmlopes

Initial step in commit: 05100c0f8ea4842e19992d5d52da55c4750db95d

pmlopes avatar May 20 '21 11:05 pmlopes