Skip to content

Add Origin header test for CORS not being set#480

Merged
hummeltech merged 1 commit intoopenstreetmap:masterfrom
hummeltech:CORSMaster
Apr 15, 2025
Merged

Add Origin header test for CORS not being set#480
hummeltech merged 1 commit intoopenstreetmap:masterfrom
hummeltech:CORSMaster

Conversation

@hummeltech
Copy link
Collaborator

Also moved cors = NULL fix from #479 into _add_tile_config instead as that fix only applied to configs added via AddTileConfig in httpd.conf files, not all methods.

@codecov-commenter
Copy link

codecov-commenter commented Apr 14, 2025

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 81.52%. Comparing base (c4e3e62) to head (1d4e1d4).
Report is 17 commits behind head on master.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #480      +/-   ##
==========================================
- Coverage   81.54%   81.52%   -0.02%     
==========================================
  Files          24       24              
  Lines        5645     5668      +23     
==========================================
+ Hits         4603     4621      +18     
- Misses       1042     1047       +5     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@hummeltech hummeltech changed the title Add test for CORS not being set Add Origin header test for CORS not being set Apr 14, 2025
@hummeltech hummeltech force-pushed the CORSMaster branch 2 times, most recently from ae3385f to dc91364 Compare April 15, 2025 04:24
@hummeltech hummeltech merged commit ba27f1b into openstreetmap:master Apr 15, 2025
1 check passed
@hummeltech hummeltech deleted the CORSMaster branch April 15, 2025 16:28
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