Skip to content
This repository was archived by the owner on Nov 24, 2025. It is now read-only.

Conversation

@zrhoffman
Copy link
Member

@zrhoffman zrhoffman commented Jun 6, 2023

This PR fixes #7560 by making the Delivery Service SSL Keys form refresh after the certificate gets update, either by submitting the form or by fetching a new ACME certificate.


Which Traffic Control components are affected by this PR?

  • Traffic Portal

What is the best way to verify this PR?

  1. Generate a cert for the DS
  2. Go to the DS's SSL keys page and change it to a cert that expires in a year
  3. Submit the form
  4. Verify that the human-readable Expiration date field updated without you refreshing the page

PR submission checklist

@zrhoffman zrhoffman added Traffic Portal v1 related to Traffic Portal version 1 improvement The functionality exists but it could be improved in some way. labels Jun 6, 2023
Copy link
Contributor

@rimashah25 rimashah25 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code changes LGTM. Tested on local for the following scenario: I generate a cert for an existing DS and see the timestamp change when a cert is updated. But I can't generate cert through ACME to confirm a date change.

Copy link
Contributor

@rimashah25 rimashah25 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TP tests are still failing but are not related to this PR.

@mitchell852 mitchell852 merged commit 669c714 into apache:master Jun 7, 2023
@zrhoffman zrhoffman deleted the refresh-ds-ssl-keys-form branch June 7, 2023 14:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

improvement The functionality exists but it could be improved in some way. Traffic Portal v1 related to Traffic Portal version 1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Delivery Service SSL Keys form does not update human-readable fields when form submits

3 participants