Skip to content

Conversation

bdehamer
Copy link
Collaborator

When using the push-to-registry option some users have encountered header overflow issues when interacting with certain OCI registries.

By default, node limits the total size of headers received in a single response to 16KB. We have seen certain interactions with the Google Artifact Registry API return headers totaling 21KB.

This change updates the maximum allowed header size to 32KB in order to give us some more breathing room when interacting with these registries.

@Copilot Copilot AI review requested due to automatic review settings August 28, 2025 22:14
@bdehamer bdehamer requested a review from a team as a code owner August 28, 2025 22:14
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR increases the Node.js HTTP header size limit from the default 16KB to 32KB to resolve header overflow issues when interacting with OCI registries, particularly Google Artifact Registry which can return headers totaling 21KB.

  • Adds NODE_OPTIONS environment variable to set max-http-header-size to 32KB (32768 bytes)

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@bdehamer bdehamer changed the title custom node max-http-header-size Adjust node max-http-header-size setting Aug 28, 2025
Signed-off-by: Brian DeHamer <bdehamer@github.com>
@bdehamer bdehamer force-pushed the bdehamer/node-max-http-header-size branch from 5d94e5a to acf8f11 Compare August 28, 2025 22:19
@bdehamer bdehamer merged commit 24900ce into main Aug 28, 2025
20 checks passed
@bdehamer bdehamer deleted the bdehamer/node-max-http-header-size branch August 28, 2025 22:22
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.

2 participants