Skip to content

Commit 0e696da

Browse files
sichanyooSichan Yoo
and
Sichan Yoo
authored
chore: Make LogAgent conform to Sendable (#917)
* Make LogAgent conform to Sendable. This makes SwiftLogger Sendable. * Mark test loggers with @unchecked Sendable. --------- Co-authored-by: Sichan Yoo <[email protected]>
1 parent 6a2772e commit 0e696da

File tree

4 files changed

+4
-4
lines changed

4 files changed

+4
-4
lines changed

Sources/Smithy/Logging/LogAgent.swift

+1-1
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
// SPDX-License-Identifier: Apache-2.0
66
//
77

8-
public protocol LogAgent {
8+
public protocol LogAgent: Sendable {
99
/// name of the struct or class where the logger was instantiated from
1010
var name: String { get }
1111

Sources/Smithy/Logging/LogAgentLevel.swift

+1-1
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@
88
import Logging
99

1010
/// Wrapper for Logger.Level; used by LogAgent and SwiftLogger.
11-
public enum LogAgentLevel: String, Codable, CaseIterable {
11+
public enum LogAgentLevel: String, Codable, CaseIterable, Sendable {
1212
case trace
1313
case debug
1414
case info

Tests/ClientRuntimeTests/NetworkingTests/URLSession/FoundationStreamBridgeTests.swift

+1-1
Original file line numberDiff line numberDiff line change
@@ -104,7 +104,7 @@ class FoundationStreamBridgeTests: XCTestCase {
104104
}
105105
}
106106

107-
private class TestLogger: LogAgent {
107+
private class TestLogger: LogAgent, @unchecked Sendable {
108108
var name: String
109109

110110
var messages: [(level: LogAgentLevel, message: String)] = []

Tests/ClientRuntimeTests/OrchestratorTests/OrchestratorTests.swift

+1-1
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,7 @@ class OrchestratorTests: XCTestCase {
5858
}
5959
}
6060

61-
class TraceLogger: LogAgent {
61+
class TraceLogger: LogAgent, @unchecked Sendable {
6262
var trace: Trace = Trace()
6363
var name: String = "TestTraceLogger"
6464
var level: LogAgentLevel = .debug

0 commit comments

Comments
 (0)