Merge pull request #12 from lyft/nosigpipe

Handle SIGPIPE on the UDP socket

Author: Reflejo

CHANGELOG.md

@@ -2,6 +2,10 @@
 All notable changes to this project will be documented in this file.
 `Kronos` adheres to [Semantic Versioning](http://semver.org/).
 
+## [0.2.2](https://github.com/lyft/Kronos/releases/tag/0.2.1)
+- Add NOSIGPIPE to socket so the app doesn't crash on the event of a
+PIPE signal
+
 ## [0.2.1](https://github.com/lyft/Kronos/releases/tag/0.2.1)
 - Fix crash on DNS timeout after Swift 3 integration
 

Kronos.podspec

@@ -1,6 +1,6 @@
 Pod::Spec.new do |s|
   s.name = 'Kronos'
-  s.version = '0.2.1'
+  s.version = '0.2.2'
   s.license = { :type => 'Apache License, Version 2.0', :file => 'LICENSE' }
   s.summary = 'Elegant NTP client in Swift'
   s.homepage = 'https://github.com/lyft/Kronos'

Sources/NTPClient.swift

@@ -179,6 +179,9 @@ final class NTPClient {
 
         let runLoopSource = CFSocketCreateRunLoopSource(kCFAllocatorDefault, socket, 0)
         CFRunLoopAddSource(CFRunLoopGetMain(), runLoopSource, CFRunLoopMode.commonModes)
+
+        var noSIGPIPE: UInt32 = 1
+        setsockopt(CFSocketGetNative(socket), SOL_SOCKET, SO_NOSIGPIPE, &noSIGPIPE, 4)
         CFSocketConnectToAddress(socket, ip.addressData(withPort: port), timeout)
         return (runLoopSource!, socket)
     }