PR#157: dogtag: use POST for profileProcess requests -

certmonger

#157 dogtag: use POST for profileProcess requests

Merged Jun 23, 2020 by rcritten. Opened Jun 18, 2020 by ftweedal.

ftweedal/certmonger fix/ipa8373-profileProcess-POST into master

dogtag: use POST for profileProcess requests

Fraser Tweedale • Jun 18, 2020

1fa94e536b1f0b3dfed3732680d8a930ae86ebfc

ftweedal commented Jun 18, 2020

An upcoming change to Dogtag requires HTTP POST method for
profileProcess operations. Update certmonger to use POST for these
operations.

Fixes: https://pagure.io/freeipa/issue/8373

Metadata Update from @rcritten:
- Request assigned

Jun 18, 2020

rcritten commented Jun 18, 2020

This looks good. What is the timeline for the dogtag change?

I tested this against current dogtag vIa freeIPA in F31 and it installs fine and was able to renew a couple of certs just fine.

ftweedal commented Jun 19, 2020

@rcritten the change was reverted from the active PR (which has now been merged) and will be deferred until after this certmonger change lands and sees a release. (See https://github.com/dogtagpki/pki/pull/446 ). @dmoluguw can best answer what the desired timeframe is.

rcritten commented Jun 22, 2020

I just want to be sure that if I commit and ship this I'm not going to break existing installations and will future-proof for the upcoming dogtag.

ftweedal commented Jun 23, 2020

@rcritten I've tested with current and patched Dogtag. I am not worried about breakage.

rcritten commented Jun 23, 2020

Ok, ack.

Pull-Request has been merged by rcritten

Jun 23, 2020