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

vertx-websocket consumer can be configured with host + port without any impact on the exposed endpoint #5118

Closed
llowinge opened this issue Jul 25, 2023 · 0 comments
Closed

vertx-websocket consumer can be configured with host + port without any impact on the exposed endpoint #5118

llowinge opened this issue Jul 25, 2023 · 0 comments
Assignees
@jamesnetherton
Labels
area/vertx-websocket enhancement New feature or request
Milestone
3.0.0-RC2

Comments

@llowinge
Copy link
Contributor

llowinge commented Jul 25, 2023 •
edited
Loading

This test shows that it is possible to configure from("vertx-websocket:redundant.host:9999/test/default/host/port/applied"), but the host redundant.host and port 9999 are not used at all.

It would be better to not allow such configuration at all.
@llowinge llowinge added the enhancement New feature or request label Jul 25, 2023
@jamesnetherton jamesnetherton self-assigned this Jul 25, 2023
@jamesnetherton jamesnetherton changed the title vertx-websocket can be configured with host + port without any impact on the exposed endpoint vertx-websocket consumer can be configured with host + port without any impact on the exposed endpoint Jul 27, 2023
jamesnetherton added a commit to jamesnetherton/camel-quarkus that referenced this issue Jul 27, 2023
@jamesnetherton
Disallow invalid host/port values on the vertx-websocket server consumer
55cb5a7 
Fixes apache#5118
jamesnetherton added a commit to jamesnetherton/camel-quarkus that referenced this issue Jul 27, 2023
@jamesnetherton
Disallow invalid host/port values on the vertx-websocket server consumer
2018966 
Fixes apache#5118
jamesnetherton added a commit to jamesnetherton/camel-quarkus that referenced this issue Jul 27, 2023
@jamesnetherton
Disallow invalid host/port values on the vertx-websocket server consumer
749cc73 
Fixes apache#5118
@github-actions github-actions bot added this to the 3.0.0-RC2 milestone Jul 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jamesnetherton jamesnetherton

Labels
area/vertx-websocket enhancement New feature or request
Projects
None yet
Milestone
3.0.0-RC2
Development

No branches or pull requests
2 participants
@jamesnetherton @llowinge