Skip to content

BYNT-1275: CSRF rate limiting#49

Merged
tarekio merged 5 commits intomainfrom
csrf-rate-limiting
Jan 14, 2025
Merged

BYNT-1275: CSRF rate limiting#49
tarekio merged 5 commits intomainfrom
csrf-rate-limiting

Conversation

@cango91
Copy link
Contributor

@cango91 cango91 commented Jan 3, 2025

Jira Issue

  1. BYNT-1275

Description

We were discussing whether to keep databases separate so as not to corrupt production db. Otherwise we'll need to keep track of all keys added during test and remove them after tests are done.

Checklist

  • Tests added/updated
  • Documentation updated (if needed)
  • New strings prepared for translations

API Changes (if applicable)

  • Permissions checked
  • Endpoint tests added

Additional Notes

[Any other relevant information]

@cango91 cango91 added the enhancement New feature or request label Jan 3, 2025
@cango91 cango91 self-assigned this Jan 3, 2025
@cango91 cango91 requested a review from level09 January 3, 2025 10:15
Copy link
Collaborator

@level09 level09 left a comment

Choose a reason for hiding this comment

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

All good. it might be better to move lookup proxy headers to settings but we can do it in a later iteration.

@tarekio tarekio merged commit 0fb321f into main Jan 14, 2025
@tarekio tarekio deleted the csrf-rate-limiting branch January 14, 2025 17:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants