Skip to content

fix(divide): account for header names and framing in headerMaxSize (#… - #7821

Open
harshkha3417 wants to merge 1 commit into
apache:masterfrom
harshkha3417:fix/7818-divide-header-max-size
Open

harshkha3417 wants to merge 1 commit into
apache:masterfrom
harshkha3417:fix/7818-divide-header-max-size

Conversation

@harshkha3417

Copy link
Copy Markdown

Fixes #7818.

Motivation

In DividePlugin, headerMaxSize previously summed only the byte lengths of request header values:

long headerSize = exchange.getRequest().getHeaders().values()
        .stream()
        .flatMap(Collection::stream)
        .mapToLong(header -> header.getBytes(StandardCharsets.UTF_8).length)
        .sum();

This branch has not been deployed

No deployments
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.

[BUG] Divide headerMaxSize counts only values and omits header names

2 participants