Skip to content

Remove api rate limiting plugin#2993

Closed
marcaurele wants to merge 1 commit intoapache:mainfrom
marcaurele:remove-api-rate-limiting
Closed

Remove api rate limiting plugin#2993
marcaurele wants to merge 1 commit intoapache:mainfrom
marcaurele:remove-api-rate-limiting

Conversation

@marcaurele
Copy link
Member

Description

Remove the API rate limiting plugin. This will ease the removal of ehcache in CS completely, and this feature can be configured on a HTTP frontend like Nginx to perform the same job.

Discussion in the dev mailing list.

Link to initial discussion about ehcache: #2913.

The PR can be closed if people still want this feature in the code.

Types of changes

  • Breaking change (fix or feature that would cause existing functionality to change)
  • New feature (non-breaking change which adds functionality)
  • Bug fix (non-breaking change which fixes an issue)
  • Enhancement (improves an existing feature and functionality)
  • Cleanup (Code refactoring and cleanup, that may add test cases)

@yadvr
Copy link
Member

yadvr commented Nov 12, 2018

Looks like this may be in use by users, we can deprecate this over time but that will be a long process than simply remove it.

@marcaurele
Copy link
Member Author

I would have changed the documentation to state the alternative way with Nginx and its rate-limiting plugin. It would be equivalent.

@marcaurele marcaurele closed this Jan 3, 2022
Tonitzpp pushed a commit to scclouds/cloudstack that referenced this pull request Dec 18, 2025
…cclouds'

Incremento nos eventos gerados ao reverter um _snapshot_

Closes apache#2993

See merge request scclouds/scclouds!1261
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants