Skip to content

Conversation

@wescopeland
Copy link
Member

@wescopeland wescopeland commented Nov 23, 2025

This PR is a basic housekeeping PR to prepare the V2 Web API:

  • Changes internal-api.php config to a more general-purpose api.php config file.
  • Lifts up HealthController so it's used by both the Internal API and V2 Web API.

How to test this PR:

curl --request GET \
  --url http://localhost:64000/api/v2/health \
  --header 'X-API-Key: {redacted}'

Next steps:

I'm not going to open any documentation PRs over in api-docs for the first few commits into the V2 API. The first endpoint or two will likely be heavily subject to change.

@wescopeland wescopeland requested a review from a team November 23, 2025 23:01
Copy link
Member

@Jamiras Jamiras left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

internal/health and v2/health are both functional. the first is restricted properly.

@wescopeland wescopeland merged commit de00bfe into RetroAchievements:master Dec 1, 2025
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants