Skip to content
This repository has been archived by the owner on May 25, 2022. It is now read-only.

UDP / TCP input: check for nil interface before calling Close() #273

Merged
merged 3 commits into from
Sep 27, 2021
Merged

UDP / TCP input: check for nil interface before calling Close() #273

merged 3 commits into from
Sep 27, 2021

Conversation

jsirianni
Copy link
Member

@jsirianni jsirianni commented Sep 20, 2021

Resolves #272

@jsirianni jsirianni requested review from djaglowski and a team September 20, 2021 20:11
@codecov
Copy link

codecov bot commented Sep 20, 2021

Codecov Report

Merging #273 (4edbdf0) into main (48fd163) will increase coverage by 0.2%.
The diff coverage is 100.0%.

Impacted file tree graph

@@           Coverage Diff           @@
##            main    #273     +/-   ##
=======================================
+ Coverage   76.4%   76.6%   +0.2%     
=======================================
  Files         95      95             
  Lines       4454    4456      +2     
=======================================
+ Hits        3404    3416     +12     
+ Misses       719     714      -5     
+ Partials     331     326      -5     
Impacted Files Coverage Δ
operator/builtin/input/tcp/tcp.go 79.6% <100.0%> (+5.0%) ⬆️
operator/builtin/input/udp/udp.go 75.8% <100.0%> (+4.7%) ⬆️

@jsirianni jsirianni merged commit efb458e into open-telemetry:main Sep 27, 2021
djaglowski added a commit to djaglowski/opentelemetry-log-collection that referenced this pull request Nov 30, 2021
- `combine_with` setting to `recombine` operator, to allow for joining on custom delimiter ([PR315](open-telemetry#315))
- Issue where `force_flush_period` could cause line splitting to be skipped ([PR303](open-telemetry#303))
- Issue where `tcp_input` and `udp_input` could panic when stopping ([PR273](open-telemetry#273))
- Syslog severity mapping is now aligned with log specification ([PR300](open-telemetry#300))

- Improve error message when timezone database is not found ([PR289](open-telemetry#289))
djaglowski added a commit that referenced this pull request Nov 30, 2021
- `combine_with` setting to `recombine` operator, to allow for joining on custom delimiter ([PR315](#315))
- Issue where `force_flush_period` could cause line splitting to be skipped ([PR303](#303))
- Issue where `tcp_input` and `udp_input` could panic when stopping ([PR273](#273))
- Syslog severity mapping is now aligned with log specification ([PR300](#300))

- Improve error message when timezone database is not found ([PR289](#289))
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.

Check if connection is nil before calling Close() (TCP / UDP)
2 participants