HTTP/2: allow to read the body #584

honfika · 2019-04-25T18:39:56Z

Doneness:

Build is okay - I made sure that this change is building successfully.
No Bugs - I made sure that this change is working properly as expected. It doesn't have any bugs that you are aware of.
Branching - If this is not a hotfix, I am making this request against the master branch

(still not able to modify header or body data)

…which was already there)

justcoding121 · 2019-04-25T20:08:51Z

@honfika You can merge when ready.

honfika · 2019-04-25T21:49:10Z

Could you please review the last commit, too?

honfika added 5 commits April 25, 2019 20:34

Allow to read HTTP/2 request/response body

06afb6f

(still not able to modify header or body data)

set properties to null to allog collect them by GC

425c7ff

.net 4.5 build fix

6555adc

remove the old common decoder parameter (different instances needed, …

2afbd3c

…which was already there)

nullpointer fix in race conditions

42ac058

justcoding121 previously approved these changes Apr 25, 2019

View reviewed changes

another race condition fix

e6a201e

honfika dismissed justcoding121’s stale review via e6a201e April 25, 2019 20:32

justcoding121 approved these changes Apr 25, 2019

View reviewed changes

honfika merged commit 66c4113 into justcoding121:master Apr 26, 2019

Provide feedback