Reject request if NUL is present in the request line by normanmaurer · Pull Request #14180 · netty/netty (original) (raw)

franz1981

@normanmaurer

Motivation:

We should not allow extra whitespaces or control chars in the request line.

Modifications:

Result:

Fix #14166

violetagg

bryce-anderson

@normanmaurer @bryce-anderson

…tDecoder.java

Co-authored-by: Bryce Anderson bryce_anderson@apple.com

@normanmaurer

@normanmaurer

normanmaurer added a commit that referenced this pull request

Jul 15, 2024

@normanmaurer @bryce-anderson

Motivation:

We should not allow extra whitespaces or control chars in the request line.

Modifications:

Result:

Fix #14166


Co-authored-by: Bryce Anderson bryce_anderson@apple.com

normanmaurer added a commit that referenced this pull request

Jul 15, 2024

@normanmaurer @bryce-anderson

Motivation:

We should not allow extra whitespaces or control chars in the request line.

Modifications:

Result:

Fix #14166


Co-authored-by: Bryce Anderson bryce_anderson@apple.com

normanmaurer added a commit that referenced this pull request

Jul 16, 2024

@normanmaurer

Motivation:

#14180 introduced some changes which broke the handling of HTTP responses without a reason-phrase. This change fixes it by correctly following the RFC

Modifications:

Correctly handle responses without reason-phrase

Result:

Fixes regression introduced by #14180

normanmaurer added a commit that referenced this pull request

Jul 16, 2024

@normanmaurer

Motivation:

#14180 introduced some changes which broke the handling of HTTP responses without a reason-phrase. This change fixes it by correctly following the RFC

Modifications:

Correctly handle responses without reason-phrase

Result:

Fixes regression introduced by #14180

normanmaurer added a commit that referenced this pull request

Jul 16, 2024

@normanmaurer

Motivation:

#14180 introduced some changes which broke the handling of HTTP responses without a reason-phrase. This change fixes it by correctly following the RFC

Modifications:

Correctly handle responses without reason-phrase

Result:

Fixes regression introduced by #14180

normanmaurer added a commit that referenced this pull request

Jul 16, 2024

@normanmaurer

Motivation:

#14180 introduced some changes which broke the handling of HTTP responses without a reason-phrase. This change fixes it by correctly following the RFC

Modifications:

Correctly handle responses without reason-phrase

Result:

Fixes regression introduced by #14180

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

[ Show hidden characters]({{ revealButtonHref }})