Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

HTTP producer throws StackOverflowError in native mode when extracting the response body as stream #6896

Closed
jamesnetherton opened this issue Jan 9, 2025 · 0 comments
Closed

HTTP producer throws StackOverflowError in native mode when extracting the response body as stream #6896

jamesnetherton opened this issue Jan 9, 2025 · 0 comments
Assignees
@jamesnetherton
Labels
area/http area/native bug Something isn't working native
Milestone
3.18.0

Comments

@jamesnetherton
Copy link
Contributor

Bug description

Turns out I screwed up the brotli removal quite badly in #6778. There are 2 issues:

  1. BrotliAbsentBooleanSupplier checks for the presence of BrotliInputStream. Which is a bit silly given that class is being generated at build time.

  2. The generated BrotliInputStream class recursively invokes its own constructor. It needs to invoke the super class InputStream constructor.
@jamesnetherton jamesnetherton added the bug Something isn't working label Jan 9, 2025
@jamesnetherton jamesnetherton added this to the 3.18.0 milestone Jan 9, 2025
@jamesnetherton jamesnetherton self-assigned this Jan 9, 2025
jamesnetherton added a commit to jamesnetherton/camel-quarkus that referenced this issue Jan 9, 2025
@jamesnetherton
Fix generated BrotliInputStream constructor
ccec32c 
Fixes apache#6896
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jamesnetherton jamesnetherton

Labels
area/http area/native bug Something isn't working native
Projects
None yet
Milestone
3.18.0
Development

No branches or pull requests
1 participant
@jamesnetherton