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

(2.8) Add new JsonValueFormat value BASE64 #1164

Closed
cowtowncoder opened this issue Mar 15, 2016 · 1 comment
Closed

(2.8) Add new JsonValueFormat value BASE64 #1164

cowtowncoder opened this issue Mar 15, 2016 · 1 comment

Comments

@cowtowncoder
Copy link
Member

(note: see FasterXML/jackson-dataformat-avro#39 for background information)

Currently there is no good way to indicate type of byte[] encoded as Base64, since the most straight-forward type (from JSON perspective at least) of indicating String with format of base64 can not be used because of hard-coding of format values in JsonValueFormat enum.

While hard-coding itself is unfortunate and should be resolved somehow, for short-term needs addition of one more enum value makes sense.

@cowtowncoder
Copy link
Member Author

Actually it looks like this can be implemented without additional value, by detecting type passed into expectArrayFormat() method, so will NOT add new type.

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

No branches or pull requests

1 participant