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

Add application/yaml MediaType #1272

Open
Pakisan opened this issue Jun 24, 2024 · 7 comments · May be fixed by #1273
Open

Add application/yaml MediaType #1272

Pakisan opened this issue Jun 24, 2024 · 7 comments · May be fixed by #1273

Comments

@Pakisan
Copy link

Pakisan commented Jun 24, 2024

Hi!

I'm working on service, which is working with YAML, so I decided to propose to you to add application/yaml MediaType

RFC: https://www.rfc-editor.org/rfc/rfc9512.html#name-media-type-application-yaml
IANA: https://www.iana.org/assignments/media-types/media-types.xhtml

@Pakisan Pakisan linked a pull request Jun 24, 2024 that will close this issue
@jamezp
Copy link
Contributor

jamezp commented Jun 24, 2024

YAML is definitely being used more these days so I don't object to this if others are okay with it. That said, I don't think we need to add every media type. It's really simple to define your own that can be used in your own code.

@Pakisan
Copy link
Author

Pakisan commented Jun 24, 2024

That said, I don't think we need to add every media type. It's really simple to define your own that can be used in your own code.

Yep, agree with you.

I have next point:

Here is three well-known formats:

  • application/yaml
  • application/json
  • application/xml

I'm sure that will be better add YAML, to receive it's support out of the box in frameworks, like Quarkus, for example and not to create and pack own-crafted MediaType

@mkarg
Copy link
Contributor

mkarg commented Jun 25, 2024

Just adding a string constant does not make it supported "out of the box", as JAX-RS does not mandate any implementation to provide Entity Providers for YAML.

@Pakisan
Copy link
Author

Pakisan commented Jun 25, 2024

Just adding a string constant does not make it supported "out of the box", as JAX-RS does not mandate any implementation to provide Entity Providers for YAML.

No problem, I'm ready to implement it. Where to dig?

@mkarg
Copy link
Contributor

mkarg commented Jun 25, 2024

Just adding a string constant does not make it supported "out of the box", as JAX-RS does not mandate any implementation to provide Entity Providers for YAML.

No problem, I'm ready to implement it. Where to dig?

In the repo of every vendor implementing JAX-RS, as here at JAX-RS we only provide interfaces. 😅

@Pakisan
Copy link
Author

Pakisan commented Jun 25, 2024

Just adding a string constant does not make it supported "out of the box", as JAX-RS does not mandate any implementation to provide Entity Providers for YAML.

No problem, I'm ready to implement it. Where to dig?

In the repo of every vendor implementing JAX-RS, as here at JAX-RS we only provide interfaces. 😅

Oookay, understood 😅

@mkarg
Copy link
Contributor

mkarg commented Jun 25, 2024

No problem, I'm ready to implement it. Where to dig?

In the repo of every vendor implementing JAX-RS, as here at JAX-RS we only provide interfaces. 😅

Oookay, understood 😅

A good starting point is Jersey. For long time it was the Reference Implementation of JAX-RS, and still many people are using it.

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 a pull request may close this issue.

3 participants