-
-
Notifications
You must be signed in to change notification settings - Fork 492
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
[bugfix] Fix parsing of large secrets
The AKV secret type does rely on the bufio.Scanner and that uses a 64k limit by default. This change extends the limit to always match the supplied input data. This might exceed the available memory but not corrupt the output data. If we want to support payloads larger than the available memory we must make large changes to the parser. But since the issue is quite severe and we don't advertise to support huge playloads this seems to be a good compromise. Fixes #2594 Signed-off-by: Dominik Schulz <[email protected]>
- Loading branch information
1 parent
5b13814
commit 8a2872f
Showing
4 changed files
with
200 additions
and
20 deletions.
There are no files selected for viewing
This file contains 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
This file contains 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
This file contains 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
Oops, something went wrong.