Project

Profile

Help

Refactor #4593

closed

rename certguard type to x509

Added by amacdona@redhat.com over 5 years ago. Updated about 4 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Assignee:
Sprint/Milestone:
Start date:
Due date:
% Done:

100%

Estimated time:
Platform Release:
Groomed:
Yes
Sprint Candidate:
No
Tags:
Sprint:
Sprint 51
Quarter:

Description

IMO the call to make a new x509 certguard would make more sense if it was:

http://localhost:8000/pulp/api/v3/contentguards/certguard/x509/ 

instead of

http://localhost:8000/pulp/api/v3/contentguards/certguard/certguard/
Actions #1

Updated by bmbouter over 5 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to bmbouter
  • Sprint set to Sprint 51

+1 to this story @asmacdo.

Actions #2

Updated by bmbouter over 5 years ago

  • Groomed changed from No to Yes
Actions #3

Updated by bmbouter over 5 years ago

  • Subject changed from rename certguard type to x509 to rename certguard type to X.509
  • Description updated (diff)

Switchiing to its formal name: X.509

Actions #4

Updated by bmbouter over 5 years ago

  • Subject changed from rename certguard type to X.509 to rename certguard type to x509
  • Description updated (diff)

Actually openssl uses this shorthand so I'll use it too.

Actions #5

Updated by bmbouter over 5 years ago

  • Status changed from ASSIGNED to POST
Actions #6

Updated by bmbouter over 5 years ago

  • Description updated (diff)

This PR is ready, but the problem with the URL that ends in 'certguard/x509/' is that it still reserves the model TYPE as 'certguard' when really that is x509 also. This means the sane URL to use (and not occupy the generic term certguard correctly) is 'x509/x509'. Overall the master detail URL structure I think is causing this to not be as awesome, but I believe 'x509/x509' is better than 'certguard/certguard' so I'm +1 to this change.

Rewriting description because of it.

Actions #7

Updated by bmbouter over 5 years ago

  • Description updated (diff)

Actually the name is not determined by the TYPE, it's the app name so it will be certguard/x509 which is great.

Added by bmbouter over 5 years ago

Revision bd173cac | View on GitHub

Updates x509 certguard URL

The name being x509 makes more sense than certguard.

https://pulp.plan.io/issues/4593 closes #4593

Actions #8

Updated by bmbouter over 5 years ago

  • Status changed from POST to MODIFIED
  • % Done changed from 0 to 100
Actions #9

Updated by bmbouter over 5 years ago

  • Sprint/Milestone set to 1.0.0 Release
Actions #10

Updated by bmbouter over 5 years ago

  • Tags deleted (Pulp 3)
Actions #11

Updated by bmbouter about 4 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Also available in: Atom PDF