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

Conversation

@kdamichie
Copy link
Contributor

Removed warn log when no cookie


Which Traffic Control components are affected by this PR?

  • Traffic Ops

What is the best way to verify this PR?

Verify that there is no warn log when cookie is missing or empty

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

PR submission checklist

@codecov
Copy link

codecov bot commented May 10, 2023

Codecov Report

Merging #7513 (554168a) into master (e7dbab8) will decrease coverage by 37.91%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             master    #7513       +/-   ##
=============================================
- Coverage     65.51%   27.60%   -37.91%     
  Complexity       98       98               
=============================================
  Files           309      686      +377     
  Lines         11822    77933    +66111     
  Branches        815       90      -725     
=============================================
+ Hits           7745    21515    +13770     
- Misses         3727    54398    +50671     
- Partials        350     2020     +1670     
Flag Coverage Δ
golib_unit 48.58% <ø> (?)
grove_unit 4.60% <ø> (?)
t3c_unit 5.32% <ø> (?)
traffic_monitor_unit 21.28% <ø> (?)
traffic_ops_integration 69.42% <ø> (ø)
traffic_ops_unit 23.14% <ø> (?)
traffic_portal_v2 ?
traffic_stats_unit 10.14% <ø> (?)
unit_tests 24.24% <ø> (-51.87%) ⬇️
v3 57.79% <ø> (ø)
v4 79.18% <ø> (ø)
v5 78.63% <ø> (ø)

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

Impacted Files Coverage Δ
traffic_ops/traffic_ops_golang/tocookie/cookie.go 58.62% <ø> (ø)

... and 583 files with indirect coverage changes

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

@srijeet0406 srijeet0406 merged commit 1c11f7f into apache:master May 10, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants