Skip to content

Align Config endpoint Map signature with base class#619

Merged
ziagham merged 1 commit intoflowsynx:masterfrom
mohin-io:fix/611-config-rate-limit-name
Oct 31, 2025
Merged

Align Config endpoint Map signature with base class#619
ziagham merged 1 commit intoflowsynx:masterfrom
mohin-io:fix/611-config-rate-limit-name

Conversation

@mohin-io
Copy link
Copy Markdown
Contributor

@mohin-io mohin-io commented Oct 31, 2025

Summary

  • Rename the Config endpoint's Map parameter to
    ateLimitPolicyName to mirror EndpointGroupBase
  • Update the rate limiting registration to use the renamed parameter
  • Document the endpoint registration to match the conventions used by other endpoint groups

Testing

  • Not run: dotnet test (dotnet CLI is not installed in the current environment)

Fixes #611

@mohin-io mohin-io requested review from a team as code owners October 31, 2025 22:25
@sonarqubecloud
Copy link
Copy Markdown

@ziagham ziagham merged commit 321911c into flowsynx:master Oct 31, 2025
4 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.

Rename parameter rateLimitPolicy to rateLimitPolicyName in Config endpoint

2 participants