Corrected the default health check endpoint name

[andyo-tyk]

Description

Corrected error in the config.go file which stated that the default name of the health check endpoint is /hello whereas it is /health (per

tyk-pump/server/server.go

Line 13 in ba10799

var defaultHealthEndpoint = "health"

)

Motivation and Context

Correcting an error in the documentation

How This Has Been Tested

Visual test - it's a code comment

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

[probelabs]

This pull request corrects an error in the code comments of the config.go file. The previous comment incorrectly stated that the default health check endpoint was /hello, when the actual default is /health. This change updates the comment to reflect the correct default value and improves the surrounding comments for better clarity.

Files Changed Analysis

• config.go: The only file changed. It involves 8 additions and 7 deletions, which are purely related to updating comments and reordering the HealthCheckEndpointPort field to be grouped with HealthCheckEndpointName for improved readability.

Architecture & Impact Assessment

• What this PR accomplishes: It fixes a documentation error in the main configuration file, ensuring that developers and operators have the correct information about the default health check endpoint.
• Key technical changes introduced: The changes are limited to code comments and the reordering of a struct field. There are no functional or logical changes to the code.
• Affected system components: This change has no impact on the runtime behavior or architecture of the system. It only affects the source code documentation, improving clarity for anyone reading the configuration file.

Scope Discovery & Context Expansion

• The scope of this PR is limited to a documentation fix within the config.go file.
• A review of tyk-pump/server/server.go confirms that the default endpoint is initialized as "health", validating the correction made in this PR.
• The main() function in tyk-pump/main.go shows that the SystemConfig.HealthCheckEndpointName value from the configuration is passed directly to the server.ServeHealthCheck function, which then applies the default if the configuration value is empty.

Metadata

• Review Effort: 1 / 5
• Primary Label: bug

---

Powered by Visor from Probelabs

Last updated: 2026-05-15T17:48:00.938Z | Triggered by: pr_opened | Commit: 9dfb81a

💡 TIP: You can chat with Visor using /visor ask <your question>

[probelabs]

✅ Security Check Passed

No security issues found – changes LGTM.

✅ Architecture Check Passed

No architecture issues found – changes LGTM.

✅ Performance Check Passed

No performance issues found – changes LGTM.

---

Powered by Visor from Probelabs

Last updated: 2026-05-15T17:47:55.075Z | Triggered by: pr_opened | Commit: 9dfb81a

💡 TIP: You can chat with Visor using /visor ask <your question>

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud