Skip to content

feat: allow configuring max header length for realtime#1785

Merged
sweatybridge merged 1 commit into
mainfrom
fix-rt
Dec 28, 2023
Merged

feat: allow configuring max header length for realtime#1785
sweatybridge merged 1 commit into
mainfrom
fix-rt

Conversation

@sweatybridge

@sweatybridge sweatybridge commented Dec 28, 2023

Copy link
Copy Markdown
Contributor

What kind of change does this PR introduce?

Bug fix #1784

What is the new behavior?

Allows configuring max header length in config.toml.

Additional context

Add any other context or screenshots.

@sweatybridge sweatybridge requested a review from a team as a code owner December 28, 2023 03:50
@sweatybridge sweatybridge changed the title fix(realtime): allow configuring max header length for local dev feat: allow configuring max header length for realtime Dec 28, 2023
@coveralls

coveralls commented Dec 28, 2023

Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 7344231896

  • 1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
  • 5 unchanged lines in 1 file lost coverage.
  • Overall coverage decreased (-0.03%) to 58.295%

Files with Coverage Reduction New Missed Lines %
internal/gen/keys/keys.go 5 11.76%
Totals Coverage Status
Change from base Build 7328565609: -0.03%
Covered Lines: 6005
Relevant Lines: 10301

💛 - Coveralls

@sweatybridge sweatybridge merged commit 0d351fc into main Dec 28, 2023
@sweatybridge sweatybridge deleted the fix-rt branch December 28, 2023 04:08
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.

Fix Realtime Bug By Overriding MAX_HEADER_LENGTH in config.toml for Realtime

2 participants