Skip to content

Conversation

@youngledo
Copy link
Contributor

Currently, the health check details displayed for Redis are as follows:

image

But, the health check will report an error when different Redis distributions return different information.

image image image image

To avoid this issue, when enabling "always" or "when_authorized" it's preferable to display all details rather than just the version number:

management.endpoint.health.show-details=always | when_authorized

This issue might also exist in older versions and may require a backport.
Similarly, other component distributions may also have similar issues.

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Nov 28, 2025
Signed-off-by: Huang Xiao <youngledo@qq.com>
@wilkinsona wilkinsona changed the title The health check will report an error when different Redis distributions return different information. Redis health check reports an error when redis_version is missing from the INFO response Nov 28, 2025
Signed-off-by: Huang Xiao <youngledo@qq.com>
@wilkinsona wilkinsona added type: bug A general bug and removed status: waiting-for-triage An issue we've not yet triaged labels Nov 28, 2025
@wilkinsona wilkinsona added this to the 3.4.x milestone Nov 28, 2025
@wilkinsona wilkinsona self-assigned this Nov 28, 2025
@wilkinsona
Copy link
Member

For future reference, from the docs for the INFO command:

Please note depending on the version of Redis some of the fields have been added or removed. A robust client application should therefore parse the result of this command by skipping unknown properties, and gracefully handle missing fields.

So it's reasonable for us to start gracefully handling the missing redis_version field, even though it appears it really should be there for Redis 2.4 and later.

@wilkinsona
Copy link
Member

Thanks very much, @youngledo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: bug A general bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants