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

Make ObjectMapper configurable #76

Open
FrelliBB opened this issue Jan 13, 2017 · 1 comment
Open

Make ObjectMapper configurable #76

FrelliBB opened this issue Jan 13, 2017 · 1 comment
Milestone

Comments

@FrelliBB
Copy link
Member

Since we are using ObjectMapper for JSON de/serialization we should provide a way for Bastion users to configure the ObjectMapper we use for Bastion; ideally through Bastion.config().

Therefore we should also make sure that we use the same instance of ObjectMapper whenever we de/serialize.

I think we can have Bastion.config().objectMapper() provide an instance which can be configured.

@KPull KPull added this to the 0.6-BETA milestone Jan 14, 2017
@KPull
Copy link
Member

KPull commented Jan 28, 2017

There are some left overs of Google's GSON in the code. For anyone tackling this task, it would be great if we remove any references to Google GSON and use Jackson's ObjectMapper instead.

@KPull KPull modified the milestones: 0.6-BETA, 1.0 Mar 31, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants