[V3] Leaking dependencies through public API

Azure / azure-cosmosdb-java

Java Async SDK for SQL API of Azure Cosmos DB

MIT License

54 stars 61 forks source link

[V3] Leaking dependencies through public API #153

Closed christopheranderson closed 5 years ago

christopheranderson commented 5 years ago

All classes extending from com.microsoft.azure.cosmosdb.JsonSerializable expose a constructor that leaks com.fasterxml.jackson.databind.ObjectMapper as a parameter type. Similarly, there are methods on this class, inherited by all Resource sub-types, that expose Jackson types through the public API.

christopheranderson commented 5 years ago

This is a TON of work and I'm not sure it's the right path for ObjectMapper. For now I'm going to make those APIs package private and we'll deal with this in 3.1.

christopheranderson commented 5 years ago

done