Skip to content

Using a filename mask in the include directive of geo blocks#34

Open
jimf5 wants to merge 2 commits into
nginx:masterfrom
jimf5:dev/geo-include
Open

Using a filename mask in the include directive of geo blocks#34
jimf5 wants to merge 2 commits into
nginx:masterfrom
jimf5:dev/geo-include

Conversation

@jimf5
Copy link
Copy Markdown
Contributor

@jimf5 jimf5 commented Mar 13, 2026

Proposed changes

Adjust tests geo.t and stream_geo.t to use a filename mask in the include directive of geo blocks.
Remove the unsafe check in geo_binary.t and stream_geo_binary.t because it is no longer needed.
This PR validates nginx/nginx#1184

Checklist

Before creating a PR, run through this checklist and mark each as complete:

@jimf5 jimf5 requested a review from pluknet March 13, 2026 00:31
@jimf5 jimf5 self-assigned this Mar 13, 2026
@sindhushiv sindhushiv moved this from New to In Progress in NGINX OSS Unified Workspace Mar 16, 2026
@sindhushiv sindhushiv moved this from In Progress to In Review in NGINX OSS Unified Workspace Mar 16, 2026
@jimf5 jimf5 force-pushed the dev/geo-include branch from ef8bdb5 to b2f146e Compare March 24, 2026 18:14
@jimf5 jimf5 force-pushed the dev/geo-include branch from b2f146e to 6532db5 Compare April 2, 2026 23:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: In Review

Development

Successfully merging this pull request may close these issues.

2 participants