- Notifications
You must be signed in to change notification settings - Fork1.7k
C#: Make web.config match case insensitive (with change note)#20065
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Pull Request Overview
This PR updates the C# CodeQL library to make web.config file matching case insensitive. Previously, only files with exact case matches (e.g., "Web.config") were recognized, but now files with any case variation (e.g., "web.config", "WEB.CONFIG") will be detected.
- Modifies web.config file detection to use case-insensitive matching
- Updates both standard web.config and web.release.config file patterns
- Adds a change note documenting the fix and its impact on existing queries
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| csharp/ql/src/change-notes/2025-07-16-web-config.md | Documents the case insensitive matching fix and query impact |
| csharp/ql/lib/semmle/code/asp/WebConfig.qll | Implements case insensitive file name matching using toLowerCase() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Excellent - thank you!
d6a3b2e intogithub:mainUh oh!
There was an error while loading.Please reload this page.
Variant of#20061 with a change note