Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add warnings for modifying things that invalidate DS certs #3776

Closed
2 of 16 tasks
ocket8888 opened this issue Aug 5, 2019 · 1 comment
Closed
2 of 16 tasks

Add warnings for modifying things that invalidate DS certs #3776

ocket8888 opened this issue Aug 5, 2019 · 1 comment
Labels
new feature A new feature, capability or behavior Traffic Ops related to Traffic Ops

Comments

@ocket8888
Copy link
Contributor

I'm submitting a ...

  • bug report
  • new feature / enhancement request
  • improvement request (usability, performance, tech debt, etc.)
  • other

Traffic Control components affected ...

  • CDN in a Box
  • Documentation
  • Grove
  • Traffic Control Client
  • Traffic Monitor
  • Traffic Ops
  • Traffic Ops ORT
  • Traffic Portal
  • Traffic Router
  • Traffic Stats
  • Traffic Vault
  • unknown

Current behavior:

There are several things you can edit that will cause stored SSL certificates for a Delivery Service to become invalid. These include - but may not be limited to - :

  • HOST_REGEXP in position 0
  • routing name
  • xmlid

Expected / new behavior:

Pursuant to #887 , it seems like we can't really do anything but issue a warning when such changes are made. Because of the number of ways in which a cert could be generated or imported, it's impossible to regenerate them in a generic way with the current data model.

Minimal reproduction of the problem with instructions:

Create a DS, generate SSL keys, change the xmlid.

@ocket8888 ocket8888 added new feature A new feature, capability or behavior Traffic Ops related to Traffic Ops labels Aug 5, 2019
@srijeet0406
Copy link
Contributor

Host regexp has been made immutable in #5297
CDN/ Routing Name have been made immutable in #5480
@ocket8888 This can be closed IMO

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new feature A new feature, capability or behavior Traffic Ops related to Traffic Ops
Projects
None yet
Development

No branches or pull requests

2 participants