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

Support path parameters in platform-http #382

Closed
davsclaus opened this issue Nov 8, 2019 · 5 comments
Closed

Support path parameters in platform-http #382

davsclaus opened this issue Nov 8, 2019 · 5 comments
Assignees
Labels
bug Something isn't working
Milestone

Comments

@davsclaus
Copy link
Contributor

I wonder if its in the rest-platform that we dont support rest-dsl with path placeholders

@ApplicationScoped
public class MyCamel extends RouteBuilder {

    @Override
    public void configure() throws Exception {
        rest("/camel/").produces("text/plain")
            .get("{id}").to("bean:myBean");
    }
}

You can then not do localhost:8080/camel/123

This works in regular Camel

@ppalaga
Copy link
Contributor

ppalaga commented Nov 8, 2019

This probably needs to get fixed in QuarkusPlatformHttpConsumer. I remember I ignored the path params because I had no idea how to use them from Camel.

@ppalaga
Copy link
Contributor

ppalaga commented Nov 8, 2019

I can fix this at some point unless somebody is faster than me.

@ppalaga
Copy link
Contributor

ppalaga commented Nov 8, 2019

Could you please rename this issue to e.g. "Support path parameters in platform-http"?

@davsclaus davsclaus changed the title rest-platform Support path parameters in platform-http Nov 8, 2019
@lburgazzoli lburgazzoli added the bug Something isn't working label Nov 11, 2019
@lburgazzoli lburgazzoli added this to the quarkus 1.0.0 milestone Nov 11, 2019
@lburgazzoli
Copy link
Contributor

lburgazzoli commented Nov 13, 2019

@ppalaga any chance you can look at this so we can cut the next release ?

@ppalaga
Copy link
Contributor

ppalaga commented Nov 13, 2019

Yes, I can start looking today :)

ppalaga added a commit to ppalaga/camel-quarkus that referenced this issue Nov 13, 2019
ppalaga added a commit to ppalaga/camel-quarkus that referenced this issue Nov 13, 2019
ppalaga added a commit to ppalaga/camel-quarkus that referenced this issue Nov 13, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants