Skip to content

Fix Unversioned Endpoints - #1203

Merged
commonsensesoftware merged 2 commits into
mainfrom
dev/css/fix-unversioned-endpoints
Jul 28, 2026
Merged

Fix Unversioned Endpoints#1203
commonsensesoftware merged 2 commits into
mainfrom
dev/css/fix-unversioned-endpoints

Conversation

@commonsensesoftware

Copy link
Copy Markdown
Collaborator

Fix Unversioned Endpoints

  • You've read the Contributor Guide and Code of Conduct.
  • You've included unit or integration tests for your change, where applicable.
  • You've included inline docs for your change, where applicable.
  • There's an open issue for the PR that you are making. If you'd like to propose a new feature or change, please open an issue to discuss the change or find an existing issue.

Description

Fixes the routing of unversioned endpoints. The ApiVersioningMatcherPolicy builds edges and a jump table based on versioned endpoints. An Endpoint is versioned if it has ApiVersionMetadata applied. The IEndpointSelectorPolicy.AppliesToEndpoints builds and enforces versioning policies when any Endpoint is versioned in a provided set. Before this change, it was possible that a versioned Endpoint could provide a route template with a fallback that would also include overlapping unversioned endpoints in the same set. As a result, any unversioned endpoints have versioning policies enforced and become unrouteable.

This change ensures that unversioned endpoints remain routeable without any versioning policies. If an endpoint really doesn't exist, 404 is still the final destination.

@commonsensesoftware
commonsensesoftware force-pushed the dev/css/fix-unversioned-endpoints branch from 92f0a65 to 2d80382 Compare July 28, 2026 00:45
@commonsensesoftware
commonsensesoftware marked this pull request as ready for review July 28, 2026 00:56
@commonsensesoftware
commonsensesoftware merged commit 7da98f3 into main Jul 28, 2026
6 checks passed
@commonsensesoftware
commonsensesoftware deleted the dev/css/fix-unversioned-endpoints branch July 28, 2026 01:05
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.

1 participant