Skip to content

feat(api-gateway): add user blacklist feature#1798

Merged
deshmukhmayur merged 1 commit into
masterfrom
feat-api-gateway-blacklist
Jun 3, 2026
Merged

feat(api-gateway): add user blacklist feature#1798
deshmukhmayur merged 1 commit into
masterfrom
feat-api-gateway-blacklist

Conversation

@deshmukhmayur

Copy link
Copy Markdown
Member

Explain the feature/fix

  • Introduced a user blacklist functionality that loads blocked user identifiers from a specified file at startup.
  • Added support for a new environment variable BLACKLIST_FILE_PATH to configure the blacklist file location.
  • Implemented related functions for managing the blacklist, including loading from file and checking user claims against the blacklist.
  • Updated README with instructions on using the blacklist feature and provided an example blacklist file.
  • Enhanced test coverage for blacklist functionality.

Assisted-By: Cursor cursoragent@cursor.com

Does this PR introduce a breaking change

No.

Screenshots

N/A

Ready-for-merge Checklist

  • Expected files: all files in this pull request are related to one feature request or issue (no stragglers)?
  • Does the change have appropriate unit tests?
  • Did tests pass?
  • Did you update or add any necessary documentation (README.md, WHY.md, etc.)?
  • Was this feature demoed and the design review approved?

Assisted-By: Cursor <cursoragent@cursor.com>
Signed-off-by: Mayur Deshmukh <deshmukhmayur@outlook.com>
@deshmukhmayur deshmukhmayur self-assigned this Jun 3, 2026

@yashoswalyo yashoswalyo left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@deshmukhmayur deshmukhmayur merged commit 4abf36c into master Jun 3, 2026
3 checks passed
@deshmukhmayur deshmukhmayur deleted the feat-api-gateway-blacklist branch June 3, 2026 09:57
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.

2 participants