Route format mime type should be used

[ralpha]

Currently the format tag is ignored in routes.

#[openapi]
#[post("/create", data = "<data>", format = "text/plain")]
fn something(...) -> ...{
}

This should not be ignored but used.

This issue is also related to #10

[ncpenke]

Hi there, I took a stab at fixing this here: ncpenke@58e16cf. Please let me know if you're interested in taking it. I can open a PR and incorporate any further feedback.

Below is a screenshot with a modification to the json-web-api example. Thanks!

[lukeb2e]

I just encountered the same issue.
If you have the invalid format type in the openapi spec, the api call is not working and returns a 404.
This is because the search now seems to check the format line, but the generated swagger page is then invalid.

PUT /v1/data/asdf application/octet-stream:
   >> No matching routes for PUT /v1/data/asdf application/octet-stream.
   >> No 404 catcher registered. Using Rocket default.
   >> Response succeeded.

If I manually set the format to application/octet-stream then my code is working as expected.

[lukeb2e]

I reproduced this issue in a small example application.

https://github.com/lukeb2e/rocket-openapi-format-swaggergen-bug

Even though the endpoint is configured to #[put("/<key>", data = "<data>", format = "text/plain")] the swagger only shows formatting for application/octet-stream. If this call is executed a 404 error is produced by the webserver. See screenshot below for generated swagger page, or check the example.

The fix from @ncpenke looks good to me, but then I only started to use rust a few days ago. ;)
So ping @GREsau ?

[jmdaly]

I'm definitely being bitten by this one too. I'd like to POST some text as text/plain, but I can't get the Swagger interface to show anything other than application/octet-stream, which results in a 404 as others have mentioned above.

[jason-rietzke]

I also run in this unfortunate behaviour which makes it difficult to provide an easy to use API UI to test the interface with. Are there any plans to use the format annotation as described in this issue?
I think this would match the expected behaviour 🙂

[ralpha]

It would just depend on when I have time. And currently I'm not actively using this project much, so just doing maintenance. But if someone creates a MR I can review it and get it merged. So feel free to create one.