Lib.AspNetCore.Security icon indicating copy to clipboard operation
Lib.AspNetCore.Security copied to clipboard

Feedback

Open tpeczek opened this issue 8 years ago • 4 comments

This issue is for general feedback regarding this project.

tpeczek avatar Aug 06 '17 06:08 tpeczek

Looking great so far!

And it would be even better to have the ability to extend the behavior of IsCspReportRequest method in order to cut some requests earlier (before they get deserialized by ContentSecurityPolicyViolationReportJsonDeserializer.DeserializeAsync)

https://github.com/tpeczek/Lib.AspNetCore.Security/blob/6d6c11eee5f105e341924c04440b4669b18a31f7/Lib.AspNetCore.Security/ContentSecurityPolicyReportingMiddleware.cs#L38-L44

Den-dp avatar Feb 09 '23 12:02 Den-dp

Hi @Den-dp,

Can you elaborate a little bit more on what would you had in mind?

tpeczek avatar Feb 09 '23 20:02 tpeczek

@tpeczek sure! I'm thinking of something like that:

.MapContentSecurityPolicyReporting(ReportRoute, (context) => context.User.Identity.IsAuthenticated);

And thanks for the quick feedback! 🚀

Den-dp avatar Feb 13 '23 19:02 Den-dp

@Den-dp Thanks for the sample. I think it makes sense, let me think about it.

tpeczek avatar Feb 15 '23 14:02 tpeczek