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

adding sslkey_expirations methods in clients #7388

Merged
merged 4 commits into from
Mar 13, 2023

Conversation

ragen14
Copy link
Contributor

@ragen14 ragen14 commented Mar 8, 2023

This PR adds sslkey_expirations methodology to v4 and v5 clients
Closes: #7387


Which Traffic Control components are affected by this PR?

  • Traffic Control Client (go)

What is the best way to verify this PR?

to verify this PR, call GetExipiringCerts using TC client

If this is a bugfix, which Traffic Control versions contained the bug?

PR submission checklist

@codecov
Copy link

codecov bot commented Mar 8, 2023

Codecov Report

Merging #7388 (c55da03) into master (34a601f) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master    #7388      +/-   ##
============================================
+ Coverage     26.12%   26.13%   +0.01%     
  Complexity       98       98              
============================================
  Files           648      650       +2     
  Lines         76280    76294      +14     
  Branches         90       90              
============================================
+ Hits          19925    19939      +14     
  Misses        54520    54520              
  Partials       1835     1835              
Flag Coverage Δ
golib_unit 52.47% <ø> (ø)
grove_unit 4.60% <ø> (ø)
t3c_unit 5.33% <ø> (ø)
traffic_monitor_unit 20.43% <ø> (ø)
traffic_ops_integration 69.42% <100.00%> (+0.07%) ⬆️
traffic_ops_unit 19.70% <ø> (ø)
traffic_stats_unit 10.14% <ø> (ø)
unit_tests 22.55% <ø> (ø)
v3 57.79% <ø> (ø)
v4 79.18% <100.00%> (+0.09%) ⬆️
v5 78.63% <100.00%> (+0.09%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
traffic_ops/v4-client/sslkey_expirations.go 100.00% <100.00%> (ø)
traffic_ops/v5-client/sslkey_expirations.go 100.00% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@srijeet0406 srijeet0406 self-requested a review March 8, 2023 21:10
@srijeet0406 srijeet0406 added Traffic Ops related to Traffic Ops TO Client (Go) related to the Go implementation of a TC client improvement The functionality exists but it could be improved in some way. labels Mar 8, 2023
Copy link
Contributor

@srijeet0406 srijeet0406 left a comment

Choose a reason for hiding this comment

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

Code looks good. Just a couple of minor comments.

* under the License.
*/

type SSLKeyExpirationGetResponse struct {
Copy link
Contributor

Choose a reason for hiding this comment

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

Needs a GoDoc comment for this struct

"github.com/apache/trafficcontrol/traffic_ops/toclientlib"
)

func (to *Session) GetExpiringCerts(opts RequestOptions) (tc.SSLKeyExpirationGetResponse, toclientlib.ReqInf, error) {
Copy link
Contributor

Choose a reason for hiding this comment

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

Needs a GoDoc comment

"github.com/apache/trafficcontrol/traffic_ops/toclientlib"
)

func (to *Session) GetExpiringCerts(opts RequestOptions) (tc.SSLKeyExpirationGetResponse, toclientlib.ReqInf, error) {
Copy link
Contributor

Choose a reason for hiding this comment

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

same

@ocket8888 ocket8888 added new feature A new feature, capability or behavior low impact affects only a small portion of a CDN, and cannot itself break one and removed improvement The functionality exists but it could be improved in some way. labels Mar 9, 2023
Copy link
Contributor

@srijeet0406 srijeet0406 left a comment

Choose a reason for hiding this comment

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

Code looks good. All manual and automated tests pass. LGTM!

@srijeet0406 srijeet0406 merged commit 5e1bca7 into apache:master Mar 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
low impact affects only a small portion of a CDN, and cannot itself break one new feature A new feature, capability or behavior TO Client (Go) related to the Go implementation of a TC client Traffic Ops related to Traffic Ops
Projects
None yet
Development

Successfully merging this pull request may close these issues.

sslkey_expirations endpoint is not implemented in v4/v5 TC go Client
3 participants