aporeto-inc / trireme-lib

Simple, scalable and secure application segmentation
https://trireme.io
Apache License 2.0
300 stars 51 forks source link

Fix tcp option parsing (#966) #970

Closed amitlimaye closed 4 years ago

amitlimaye commented 4 years ago

Description

Changes proposed in this pull request.

Test plan

Outline the test plan used to test this change before merging it.

Fixes #.

amitlimaye commented 4 years ago

/build - automatically fired by gogo with following PRs and commit SHAs v1.0.0

[
  {
    "project": "",
    "component": "trireme-lib",
    "pr-id": "970",
    "commit-sha": "911958a2609282c0d658f622d17e459c3d84cedf",
    "pipeline": "master"
  },
  {
    "project": "",
    "component": "enforcerd",
    "pr-id": "1578",
    "commit-sha": "eeafa00a136851bd20b42909ccf53d26e943d10e",
    "pipeline": "master"
  }
]
codecov[bot] commented 4 years ago

Codecov Report

Merging #970 into master will decrease coverage by 0.15%. The diff coverage is 63.63%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #970      +/-   ##
==========================================
- Coverage   54.46%   54.31%   -0.16%     
==========================================
  Files         122      122              
  Lines       11732    11737       +5     
==========================================
- Hits         6390     6375      -15     
- Misses       4714     4739      +25     
+ Partials      628      623       -5
Impacted Files Coverage Δ
controller/pkg/packet/packet.go 52.08% <63.63%> (+0.13%) :arrow_up:
monitor/extractors/linux.go 48.78% <0%> (-21.96%) :arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 9d04acb...911958a. Read the comment docs.