Skip to content

fix: rate limit migration assistant routes#198

Merged
Jozsef Damokos (jozsefdamokos) merged 4 commits into
trunkfrom
fix/migration-assistant-rate-limits
Jun 17, 2026
Merged

fix: rate limit migration assistant routes#198
Jozsef Damokos (jozsefdamokos) merged 4 commits into
trunkfrom
fix/migration-assistant-rate-limits

Conversation

@jozsefdamokos

@jozsefdamokos Jozsef Damokos (jozsefdamokos) commented Jun 10, 2026

Copy link
Copy Markdown
Member
  • add dedicated rate limiting to migration assistant log access and download endpoints
  • keep the limiter definitions inside the plugin by registering them through a compiler pass
  • cover the new limiter behavior with focused controller and unit tests

Fixes: shopware/shopware#16571

@jozsefdamokos Jozsef Damokos (jozsefdamokos) marked this pull request as ready for review June 11, 2026 10:46
Comment thread src/RateLimiter/MigrationApiRateLimiter.php Outdated
@jozsefdamokos Jozsef Damokos (jozsefdamokos) merged commit 9de8b84 into trunk Jun 17, 2026
8 checks passed
@jozsefdamokos Jozsef Damokos (jozsefdamokos) deleted the fix/migration-assistant-rate-limits branch June 17, 2026 07:42
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.

Add rate limits to migration assistant routes

3 participants