Skip to content

Fixed Bypassed IPs not returning populated should_block_request decision structure#401

Merged
tudor-timcu merged 7 commits intomainfrom
bypass-should-block-request-nil
Mar 17, 2026
Merged

Fixed Bypassed IPs not returning populated should_block_request decision structure#401
tudor-timcu merged 7 commits intomainfrom
bypass-should-block-request-nil

Conversation

@tudor-timcu
Copy link
Copy Markdown
Collaborator

@tudor-timcu tudor-timcu commented Mar 17, 2026

Summary by Aikido

Security Issues: 0 Quality Issues: 0 Resolved Issues: 0

⚡ Enhancements

  • Updated agent and PHP extension versions to 1.5.3.

🐛 Bugfixes

  • Initialized returned object for bypassed IPs to prevent null decision.

🔧 Refactors

  • Consolidated and simplified bypass conditional checks for early return.

More info

- Introduced `env.json` for environment variables related to Aikido blocking.
- Created `index.php` to handle requests and block specific IPs based on Aikido's decision.
- Added `start_config.json` for initial configuration settings, including allowed IPs and rate limiting.
- Implemented `test.py` to validate that requests from bypassed IPs return the expected response without crashing.
- Consolidated the bypass check for Aikido into a single conditional statement.
- Ensured that the return value is initialized with default values to prevent unintended behavior when an IP is bypassed.
…ndling

- Updated the gRPC library version in both agent and request-processor modules.
- Adjusted test assertions to reflect changes in response handling for bypassed IPs.
- Added a new assertion to check that the response does not contain a null decision message.
@tudor-timcu tudor-timcu changed the title Fixed Bypassed IPs not returning populated should_block_request decisions structure Fixed Bypassed IPs not returning populated should_block_request decision structure Mar 17, 2026
- Simplified the conditional checks for Aikido bypass status.
- Ensured early return for bypassed IPs to prevent unnecessary processing.
- Maintained initialization of return value for consistent behavior.
@tudor-timcu tudor-timcu merged commit 96ecd3e into main Mar 17, 2026
240 of 243 checks passed
@tudor-timcu tudor-timcu deleted the bypass-should-block-request-nil branch March 17, 2026 13:53
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