Skip to content

Read the parsed body in Net Core#2501

Merged
anna-git merged 1 commit into
appsec/parsed-body-commonfrom
appsec/anna/aspnetcore-readingbody
Feb 24, 2022
Merged

Read the parsed body in Net Core#2501
anna-git merged 1 commit into
appsec/parsed-body-commonfrom
appsec/anna/aspnetcore-readingbody

Conversation

@anna-git

Copy link
Copy Markdown
Contributor

Summary of changes

Reason for change

Implementation details

Test coverage

Other details

@anna-git
anna-git requested a review from robertpi February 24, 2022 14:27
@anna-git
anna-git requested a review from a team as a code owner February 24, 2022 14:27
@anna-git
anna-git removed the request for review from a team February 24, 2022 14:27
@anna-git
anna-git merged commit f61ac89 into appsec/parsed-body-common Feb 24, 2022
@anna-git
anna-git deleted the appsec/anna/aspnetcore-readingbody branch February 24, 2022 14:43
anna-git added a commit that referenced this pull request Feb 28, 2022
anna-git added a commit that referenced this pull request Feb 28, 2022
anna-git added a commit that referenced this pull request Mar 2, 2022
anna-git added a commit that referenced this pull request Mar 3, 2022
anna-git added a commit that referenced this pull request Mar 3, 2022
anna-git added a commit that referenced this pull request Mar 7, 2022
anna-git added a commit that referenced this pull request Mar 8, 2022
anna-git added a commit that referenced this pull request Mar 10, 2022
anna-git added a commit that referenced this pull request Mar 16, 2022
Adding a more complex model

Removing tests with block enabled

Removing tests with block enabled Renaming samples wth security prefix to avoid nuke confusion Changing sln accordingly
anna-git added a commit that referenced this pull request Mar 23, 2022
Adding a more complex model

Removing tests with block enabled

Removing tests with block enabled Renaming samples wth security prefix to avoid nuke confusion Changing sln accordingly
robertpi pushed a commit that referenced this pull request Mar 29, 2022
Adding a more complex model

Removing tests with block enabled

Removing tests with block enabled Renaming samples wth security prefix to avoid nuke confusion Changing sln accordingly
anna-git added a commit that referenced this pull request Mar 31, 2022
Adding a more complex model

Removing tests with block enabled

Removing tests with block enabled Renaming samples wth security prefix to avoid nuke confusion Changing sln accordingly
anna-git added a commit that referenced this pull request Mar 31, 2022
* Failing test for framework

* Adding extractor for body model and event

* Subscribing to body available in security

* add content type for post request

* Read the parsed body in .NET Framework

Includes:
* Win Forms
* WebApi
* MVC

* Read the parsed body in Net Core (#2501)

Adding a more complex model

Removing tests with block enabled

Removing tests with block enabled Renaming samples wth security prefix to avoid nuke confusion Changing sln accordingly

* Adapt build steps for linux and rename integration right

* Changing snapshot files to match new ressponse content length

* Publish views dll for linux cause we use the publish folder for inte test

fix build

* Start of unit tests for parsed body

* Unit tests and handle some serialization corner cases

* Move code from IG into BodyExtractor.cs

* Better detection of primatives
Detect object graphs with cycles
Only look at instance properties

* Use fields instead of property to avoid side effects

Use fields instead of property to avoid side effects

Fix possible null ref

* [Appsec]Benchmarks (#2537)

* first appsec benchmark

* Remove required filter and add more benchmarks

* Fix benchmarks for .NET framework

* add a more complex object

* fix build steps

* Env variables

* fix benchmarks

* Init httpcontext outside of benchmark

* remove global setup

Co-authored-by: Robert Pickering <[email protected]>

* Adding enum to body extractor

* Remove unneeded code containing a TODO

Adding verified files

* First response to comments

* Fixing duck typing

Factorize peek and pop with Func

remove duck type on real type

* Repsonse to comments

Response to comments

* Fix integration tests

* Fix called several times if 2 binding sources with different id , fix derived attribute

* Adding instrumentation scenario filter to instrumentations

* Readonlydictionary and reuse the empty one

Co-authored-by: Anna <[email protected]>
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.

1 participant