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

3.0.x back ported 3331 and 3380#3406

Merged
dg4prez merged 4 commits intoapache:3.0.xfrom
ajschmidt:3.0.x
Mar 15, 2019
Merged

3.0.x back ported 3331 and 3380#3406
dg4prez merged 4 commits intoapache:3.0.xfrom
ajschmidt:3.0.x

Conversation

@ajschmidt
Copy link
Copy Markdown
Contributor

Which issue is fixed by this PR? If not related to an existing issue, what does this PR do?

Fixes #(replace_this_text_with_issue_number)

Which TC components are affected by this PR?

  • Documentation
  • Grove
  • Traffic Analytics
  • Traffic Monitor
  • Traffic Ops
  • Traffic Ops ORT
  • Traffic Portal
  • Traffic Router
  • Traffic Stats
  • Traffic Vault
  • Other _________

What is the best way to verify this PR? Please include manual steps or automated tests.

(If no tests are part of this PR, please provide explanation as to why no tests are included.)

curl a bunch of HTTPS delivery service URLs, including mixed case and some bogus DS urls. Also, add an expired SSL cert through TP and verify that it does not get used by TR

Check all that apply

  • This PR includes tests
  • This PR includes documentation updates
  • This PR includes an update to CHANGELOG.md
  • This PR includes all required license headers
  • This PR includes a database migration (ensure that migration sequence is correct)
  • This PR fixes a serious security flaw. Read more: www.apache.org/security

* Fix case-sensitive SSL cert lookup
* Set SSL certificate key names to lower case
* Set requested SNI to lower case

(cherry picked from commit 6504e7f)
* updated tr docker config so it would work

* added start and stop scripts for TR docker containers

* fixed bug in Tomcat which wasn't unregistering old SslHostConfigs

* Added validation checks of SSL certs while being loaded by CertificateRegistry

* Changed error message for Certificate Expired

* Corrected a parsing error in certificate validation checking
Set the startup/shutdown scripts for the TR docker container to executable

* Corrected some code formatting and one NullPointerException

* Added info to CHANGELOG.md

(cherry picked from commit 0d2560d)
@rawlinp rawlinp added backport an upstream change merged into an older release Traffic Router related to Traffic Router labels Mar 14, 2019
@asfgit
Copy link
Copy Markdown
Contributor

asfgit commented Mar 14, 2019

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/3343/
Test PASSed.

@rawlinp rawlinp added this to the 3.0.1 milestone Mar 14, 2019
@asfgit
Copy link
Copy Markdown
Contributor

asfgit commented Mar 14, 2019

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/3344/
Test PASSed.

@asfgit
Copy link
Copy Markdown
Contributor

asfgit commented Mar 14, 2019

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/3345/
Test PASSed.

Copy link
Copy Markdown
Contributor

@rawlinp rawlinp left a comment

Choose a reason for hiding this comment

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

LGTM

@dg4prez dg4prez merged commit cf62217 into apache:3.0.x Mar 15, 2019
@mitchell852 mitchell852 added the new feature A new feature, capability or behavior label Oct 24, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

backport an upstream change merged into an older release new feature A new feature, capability or behavior Traffic Router related to Traffic Router

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants