-
Notifications
You must be signed in to change notification settings - Fork 893
[Prometheus.AspNetCore] Support GZip #7274
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
Changes from 12 commits
0f018e2
9bdc57a
b7ff4db
5e90244
ca448e5
af91589
05b3a83
50b75f2
60aad7a
bf2ed3a
c6f5522
c04e031
d003819
a98605d
cc4019e
844990b
f848e61
2cfffe0
d2e5ec6
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -7,6 +7,7 @@ | |
| <PackageTags>$(PackageTags);prometheus;metrics</PackageTags> | ||
| <MinVerTagPrefix>coreunstable-</MinVerTagPrefix> | ||
| <DefineConstants>$(DefineConstants);PROMETHEUS_ASPNETCORE</DefineConstants> | ||
| <NoWarn>$(NoWarn);CA2007</NoWarn> | ||
|
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. AspNetCore has no synchronisation context so
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. True, I would consider keeping this calls just in case we need to share with httplistener similar code
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. At the moment the code reuse is one way from HttpListener to AspNetCore, so I don't think we need to worry about that in practice as if it needs to be shared it'll go in the other project which doesn't supress CA2007. |
||
| </PropertyGroup> | ||
|
|
||
| <ItemGroup Condition="'$(RunningDotNetPack)' != 'true'"> | ||
|
|
||
Uh oh!
There was an error while loading. Please reload this page.