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.

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

Create a camel-websocket-jsr356 extension #613

Closed
lburgazzoli opened this issue Jan 9, 2020 · 6 comments · Fixed by #833
Closed

Create a camel-websocket-jsr356 extension #613

lburgazzoli opened this issue Jan 9, 2020 · 6 comments · Fixed by #833

Comments

@lburgazzoli
Copy link
Contributor

No description provided.

@jamesnetherton
Copy link
Contributor

I can take this one. I actually had a tentative look into it this morning.

@jamesnetherton jamesnetherton self-assigned this Jan 10, 2020
@jamesnetherton
Copy link
Contributor

As the producer part of this component is currently broken, is it worth me implementing an extension for camel-ahc-ws until the fix is available?

@lburgazzoli
Copy link
Contributor Author

@jamesnetherton no hack possible ?

@jamesnetherton
Copy link
Contributor

It is possible. Would need to override and extend some bits of the component internals.

@lburgazzoli
Copy link
Contributor Author

if the workaround is not too complex I'd go for the workaround and oyu are free to implement also camel-ahc-ws :)

@jamesnetherton
Copy link
Contributor

I think this component in its current form is going to need too many hacks to get it working properly.

I opened #619 to implement ahc-ws. I'll revisit this when I'm done fixing up the websocket-jsr356 component on the Camel side.

jamesnetherton added a commit to jamesnetherton/camel-quarkus that referenced this issue Mar 2, 2020
jamesnetherton added a commit to jamesnetherton/camel-quarkus that referenced this issue Mar 2, 2020
jamesnetherton added a commit to jamesnetherton/camel-quarkus that referenced this issue Mar 2, 2020
jamesnetherton added a commit that referenced this issue Mar 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants