Skip to content

Use bufio.Reader instead of bufio.Scanner for lines.Send

username-removed-506061 requested to merge use-reader-instead-of-scanner-for-lines-send into master

Conversation: https://gitlab.com/gitlab-org/gitaly/issues/384

Fixes https://gitlab.com/gitlab-org/gitaly/issues/486.

bufio.Scanner has a limit on how long a token can be, which would make parsing commits with a very long message (e.g. this commit) fail.

Edited by Andrew Newdigate

Merge request reports