Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

harmonization of gorouter -> java app communication#1078

Merged
pivotal-david-osullivan merged 1 commit intocloudfoundry:maincloudfoundry/java-buildpack:mainfrom
h0nIg:patch-1h0nIg/java-buildpack:patch-1Copy head branch name to clipboard
Oct 23, 2024
Merged

harmonization of gorouter -> java app communication#1078
pivotal-david-osullivan merged 1 commit intocloudfoundry:maincloudfoundry/java-buildpack:mainfrom
h0nIg:patch-1h0nIg/java-buildpack:patch-1Copy head branch name to clipboard

Conversation

@h0nIg
Copy link
Contributor

@h0nIg h0nIg commented May 29, 2024

fix for #881

it is a known issue that gorouter -> java applications do not work properly together, once keepalive is used: https://knowledge.broadcom.com/external/article/298104/intermittent-502-eof-gorouter-errors-for.html

The fix is described in the link, but not implemented so far for the java buildpack

@h0nIg
Copy link
Contributor Author

h0nIg commented Sep 6, 2024

@dmikusa @anthonydahanne @kevin-ortega @pivotal-david-osullivan any chance to get this merged?

@anthonydahanne
Copy link
Contributor

@pivotal-david-osullivan can you please review / approve too? thanks!

@h0nIg
Copy link
Contributor Author

h0nIg commented Oct 10, 2024

ping @anthonydahanne @pivotal-david-osullivan @Gerg

I don't understand why an additional approval is required for an existing knowledge article. Can you finally merge this please?

@anthonydahanne
Copy link
Contributor

@h0nIg h0nIg deleted the patch-1 branch November 11, 2024 07:58
ramonskie pushed a commit that referenced this pull request Dec 4, 2025
harmonization of gorouter -> java app communication
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.

3 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.