#678 Combining REST resources for managing certificate requests
Closed: migrated by dmoluguw. Opened by edewata.

Currently the operations to manage certificate request are handled by separate REST resources which are more RPC-like. They should be combined into a single resource which is the certificate request itself. This way the URL will not change suppose the operations change in the future.

Here is the current interface:

public interface CertRequestResource {
    @POST
    @Path("agent/certrequests/{id}/approve")
    public void approveRequest(@PathParam("id") RequestId id,
        CertReviewResponse data);
    @POST
    @Path("agent/certrequests/{id}/reject")
    public void rejectRequest(@PathParam("id") RequestId id,
        CertReviewResponse data);
    @POST
    @Path("agent/certrequests/{id}/cancel")
    public void cancelRequest(@PathParam("id") RequestId id,
        CertReviewResponse data);
    @POST
    @Path("agent/certrequests/{id}/update")
    public void updateRequest(@PathParam("id") RequestId id,
        CertReviewResponse data);
    @POST
    @Path("agent/certrequests/{id}/validate")
    public void validateRequest(@PathParam("id") RequestId id,
        CertReviewResponse data);
    @POST
    @Path("agent/certrequests/{id}/unassign")
    public void unassignRequest(@PathParam("id") RequestId id,
        CertReviewResponse data);
    @POST
    @Path("agent/certrequests/{id}/assign")
    public void assignRequest(@PathParam("id") RequestId id,
        CertReviewResponse data);
}

Proposed interface:

public interface CertRequestResource {
    @POST
    @Path("agent/certrequests/{id}")
    public void processRequest(@PathParam("id") RequestId id,
        @QueryParam String action,
        CertReviewResponse data);
}

Metadata Update from @edewata:
- Issue set to the milestone: UNTRIAGED

Dogtag PKI is moving from Pagure issues to GitHub issues. This means that existing or new
issues will be reported and tracked through Dogtag PKI's GitHub Issue tracker.

This issue has been cloned to GitHub and is available here:
https://github.com/dogtagpki/pki/issues/1247

If you want to receive further updates on the issue, please navigate to the
GitHub issue and click on Subscribe button.

Thank you for understanding, and we apologize for any inconvenience.

Metadata Update from @dmoluguw:
- Issue close_status updated to: migrated
- Issue status updated to: Closed (was: Open)

Metadata