Skip to content

Conversation

@kastov
Copy link
Collaborator

@kastov kastov commented Aug 15, 2025

No description provided.

kastov added 2 commits August 16, 2025 00:14
…pdate start:prod command to increase max HTTP header size
@kastov kastov merged commit 556643f into main Aug 15, 2025
4 checks passed
Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

Greptile Summary

This PR represents a maintenance release bumping the version from 2.1.1 to 2.1.2. The primary change addresses a production runtime issue by adding the --max-http-header-size=65536 flag to the production start script, increasing Node.js's default HTTP header size limit from 8KB to 64KB. This change suggests the application was encountering "Request header too large" errors (HTTP 431) in production, likely due to large authentication tokens, session data, or custom headers common in VPN/proxy infrastructure applications like Remnawave.

The PR also includes routine dependency updates across several NestJS packages and development tools, bringing them to their latest minor versions. These updates appear to be standard maintenance to incorporate bug fixes and security patches. The version bump follows semantic versioning practices as a patch release, indicating backward-compatible bug fixes.

This change integrates with the existing NestJS-based architecture by modifying only the production runtime configuration without affecting the application's core functionality or API contracts.

Confidence score: 5/5

  • This PR is safe to merge with minimal risk as it addresses a known production issue with a standard Node.js configuration flag
  • Score reflects a straightforward configuration change and routine dependency updates that follow established patterns
  • No files require special attention as the changes are limited to package configuration

1 file reviewed, no comments

Edit Code Review Bot Settings | Greptile

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