-
Notifications
You must be signed in to change notification settings - Fork 481
Analyzer for misusage of MaxResponseHeadersLength #6796
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from 1 commit
Commits
Show all changes
20 commits
Select commit
Hold shift + click to select a range
284fe29
tests + unshipped markdown
amiru3f f30af52
conflict resolution
amiru3f 6017fc3
conflict removal
amiru3f 5e4490e
using Symbol comparison
amiru3f 7e4bec5
warning removal
amiru3f 1d1af89
remove conflicts
amiru3f f714877
Symbol checking in CompilationStart
amiru3f 3e130b6
warning message title
amiru3f 1e350ea
Unshipped.md update and cleanup
58b5170
rename
amiru3f dd2b34a
conflict fixture
amiru3f f77ff23
const rename
amiru3f 00eec51
avoid emitting warning in case of limit disabling
amiru3f 13cdf2e
supporting SocketsHttpHandler
amiru3f b456423
texts improved
amiru3f 974fb85
pack
amiru3f bb47cde
Binary operation fix
amiru3f b62ba45
fix: mistakenly generated analyzer release file
amiru3f 873acf1
Update src/NetAnalyzers/Core/Microsoft.NetCore.Analyzers/MicrosoftNet…
amiru3f 6b9c383
regenerated stuff with msbuild pack
amiru3f File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
conflict fixture
- Loading branch information
commit dd2b34a4e74bcc691ab40cb0e1728a464ef7ec4c
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
These analyzers shipped with 8.0, looks the PR need to pull the latest and run
msbuild /t:packThere 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.
Weird, it was mistakenly (auto) generated by
msbuildpacking.Fixed.