On 6/4/2016 6:00 AM, Fraser Tweedale wrote:

Hi Endi et al,

Attached patch changes ExternalProcessKeyRetriever to read JSON data (https://fedorahosted.org/pki/ticket/2351). Would be nice to get this into 10.2.2 because it will simplify IPA custodia retrieval helper.

I am using Jackson for JSON parsing. This is already an implicit dependency, but should I also add it spec file as explicit dependency?

Cheers, Fraser

I think in general we should declare direct dependency on something that we use explicitly because there's no guarantee the indirect dependency won't change. This can be added separately later. ACK. Pushed to master. -- Endi S. Dewata