search

dogtagpki / pki

The Dogtag Certificate System is an enterprise-class Certificate Authority (CA) which supports all aspects of certificate lifecycle management, including key archival, OCSP and smartcard management.
https://www.dogtagpki.org
GNU General Public License v2.0
358 stars 134 forks source link

CertRequestInfo has incorrect URLs #2567

Closed pki-bot closed 3 years ago

pki-bot commented 3 years ago

This issue was migrated from Pagure Issue #2447. Originally filed by ftweedal (@frasertweedale) on 2016-08-26 05:07:35:

The CertRequestInfos in the enrollment response contains invalid URLs:

{"total":1
,"entries":[
  {"requestType":"enrollment"
  ,"requestStatus":"complete"
  ,"requestURL":"https://f24b-0.ipa.local:8443/ca/rest/50"
  ,"certId":"0x2d"
  ,"certURL":"https://f24b-0.ipa.local:8443/ca/rest/45"
  ,"certRequestType":"pkcs10"
  ,"operationResult":"success"
  }
 ]
,"Link":[]
}
pki-bot commented 3 years ago

Comment from ftweedal (@frasertweedale) at 2016-08-26 05:30:31

Also occurs for GET /ca/rest/certrequests/<id>. Maybe occurs in other places too?

pki-bot commented 3 years ago

Comment from mharmsen (@mharmsen) at 2016-08-31 21:22:23

Per PKI Bug Council of 08/31/2016: 10.3.6

pki-bot commented 3 years ago

Comment from vakwetu (@vakwetu) at 2016-09-02 22:26:40

alee@aleeredhat pki]$ git push origin master Counting objects: 36, done. Delta compression using up to 8 threads. Compressing objects: 100% (27/27), done. Writing objects: 100% (36/36), 3.00 KiB | 0 bytes/s, done. Total 36 (delta 21), reused 0 (delta 0) To ssh://vakwetu@git.fedorahosted.org/git/pki.git bc65e12..7a93dbe master -> master

Checked into master: 7a93dbeae18407e28437f4affc31ddc24a2c42f2

pki-bot commented 3 years ago

Comment from mharmsen (@mharmsen) at 2016-09-07 23:25:53

Cherry-picked into DOGTAG_10_3_BRANCH:

From 7baa7e60b708c5b4c79d6dd963321d34958cc81b Mon Sep 17 00:00:00 2001 From: Ade Lee alee@redhat.com Date: Fri, 2 Sep 2016 16:08:02 -0400 Subject: [PATCH 04/10] Fix CertRequestInfo URLs

The URLs were generated by a UriBuilder that referred to the resource's annotated path. This top-level path changed though, even if the underlying paths did not. Replace this with a reference to the getX methods instead.

Also fixed a few eclipse flagged warnings (unused imports etc).

Ticket 2447

(cherry picked from commit 7a93dbeae18407e28437f4affc31ddc24a2c42f2)

pki-bot commented 3 years ago

Comment from ftweedal (@frasertweedale) at 2017-02-27 14:05:41

Metadata Update from @frasertweedale: