Skip to content
This repository was archived by the owner on Apr 20, 2024. It is now read-only.

Commit 87c1e33

Browse files
Made the code safer by removing the force cast and modified the header dictonary building to use reduce(into:) so that the overall code is more idionatic Swift
1 parent 9eada43 commit 87c1e33

File tree

1 file changed

+6
-7
lines changed

1 file changed

+6
-7
lines changed

Sources/Bugsnag/BugsnagReporter.swift

Lines changed: 6 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -103,15 +103,14 @@ extension BugsnagReporter {
103103
eventRequestBody = nil
104104
}
105105

106-
var headerDict: [String : Any] = request.headers.reduce([:], { result, value in
107-
var copy = result
108-
copy[value.0] = value.1
109-
return copy
110-
})
106+
var headerDict: [String : Any] = request.headers.reduce(into: [:]) { result, value in
107+
result[value.0] = value.1
108+
}
111109
strip(keys: configuration.keyFilters, from: &headerDict)
112110

113-
let filteredHeaders: [(String, String)] = headerDict.map {
114-
k, v in (k, v as! String)
111+
let filteredHeaders: [(String, String)] = headerDict.compactMap { k, v in
112+
guard let value = v as? String else { return nil }
113+
return (k, value)
115114
}
116115

117116
eventRequest = .init(

0 commit comments

Comments
 (0)