Skip to content

feat: add IP whitelist middleware for admin routes - #198

Merged
chizzy192 merged 1 commit into
StellarState:devfrom
Rayhab2000:dev
Jul 30, 2026
Merged

feat: add IP whitelist middleware for admin routes#198
chizzy192 merged 1 commit into
StellarState:devfrom
Rayhab2000:dev

Conversation

@Rayhab2000

Copy link
Copy Markdown
Contributor
  • Add ip-whitelist middleware that checks client IP against allowed CIDR blocks
  • Add admin routes (approve-kyc, reject-kyc) with IP whitelist enforcement
  • Add ADMIN_IP_WHITELIST env var and admin.ipWhitelist config
  • Add unit tests for IP whitelist middleware
  • Add ip-range-check dependency

Description

Closes #

Type of Change

  • 🐛 Bug fix (non-breaking change which fixes an issue)
  • ✨ New feature (non-breaking change which adds functionality)
  • 💥 Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • 📝 Documentation update
  • 🎨 UI/UX improvement
  • ♻️ Code refactoring
  • ✅ Test addition or update
  • 🔧 Configuration change

Checklist

  • All GitHub Actions workflows are green on this PR (required for merge)
  • Commit messages follow Conventional Commits (feat:, fix:, chore:, etc.) — enforced by CI
  • No secrets, API keys, .env, or credentials committed (see CONTRIBUTING.md)
  • My code follows the code style of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published

Testing

How to Test

  1. Step one
  2. Step two
  3. Step three

Test Coverage

  • Unit tests added/updated
  • Integration tests added/updated
  • E2E tests added/updated (if applicable)
  • Manual testing completed

Screenshots (if applicable)

Additional Notes

For Reviewers

  • Code quality and readability
  • Test coverage
  • Security implications
  • Performance impact
  • Breaking changes

Closes #172

- Add ip-whitelist middleware that checks client IP against allowed CIDR blocks
- Add admin routes (approve-kyc, reject-kyc) with IP whitelist enforcement
- Add ADMIN_IP_WHITELIST env var and admin.ipWhitelist config
- Add unit tests for IP whitelist middleware
- Add ip-range-check dependency
@drips-wave

drips-wave Bot commented Jul 29, 2026

Copy link
Copy Markdown

@Rayhab2000 Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@chizzy192
chizzy192 merged commit 56766f5 into StellarState:dev Jul 30, 2026
3 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.

[SECURITY] Add middleware for checking IP subnet whitelist on admin routes

2 participants