#295 express payload type via an enum (based on #287)
Closed by rcritten. Opened by ftweedal.
ftweedal/certmonger pr-287-plus-enum  into  master

Download 295.patch

The proposed enum for payload type (XML or JSON). Suggest to merge #287 first,
then I can update/rebase this PR if needed and review it separately.

I still need to test this but in general it looks good.

I have my paint bucket out. Might I suggest RPC_PROTOCOL instead of PAYLOAD_FORMAT?

8 new commits added

  • express payload type via an enum
  • Pass all=True to certprofile-find to retrieve all profiles
  • Suppress unused arguments to the LDAP SASL interactive callback
  • Fix directive argument is NULL errors (-Wformat-overflow)
  • Fix format-overflow warning when converting time
  • Drop unused cm_submit_d parameters
  • Add tests for the expanded dogtag requests through IPA API
  • Use the IPA API by default for dogtag requests

bikeshed colour change approved. PR updated.

Merged via https://pagure.io/certmonger/pull-request/296

Pull-Request has been closed by rcritten

Metadata