Infinite recursion between deprecated API's about swift-log

[ktoso]

You didn't finish implementing your log handler, in the screenshot you paste the solution is included: @available(*, deprecated, message: "You should implement this method instead of using the default implementation")

    @available(*, deprecated, message: "You should implement this method instead of using the default implementation")
    public func log(level: Logger.Level,
                    message: Logger.Message,
                    metadata: Logger.Metadata?,
                    source: String,
                    file: String,
                    function: String,
                    line: UInt) {
// ... 
    }

you have to implement this method. It delegates to old methods for compatibility with existing LogHandler implementations which used the "old" API. This way we did not break source compat when adding the new handler APIs.

Please implement the above method and things will work as expected.

[ktoso]

Double checked, yeah this works correctly:

import Logging

@main
public struct untitled4 {

    public static func main() {
        LoggingSystem.bootstrap(Hello.init)

        var log = Logger(label: "unique.value")
        log[metadataKey: "mykey"] = "jelly"
        log.error("hi")
    }
}

struct Hello: LogHandler {

    var metadata: Logging.Logger.Metadata = [:]

    var logLevel: Logging.Logger.Level = .error

    init(_ label: String) {
    }

    subscript(metadataKey metadataKey: String) -> Logger.Metadata.Value? {
        get {
            return self.metadata[metadataKey]
        }
        set(newValue) {
            self.metadata[metadataKey] = newValue
        }
    }

    // REQUIRED
     func log(level: Logger.Level,
                    message: Logger.Message,
                    metadata: Logger.Metadata?,
                    source: String,
                    file: String,
                    function: String,
                    line: UInt) {
        print("message: \(message)")
    }
}

Please complete your LogHandler implementation and it'll work fine.
If there's anything else here please feel free to comment or reopen the issue

[nashysolutions]

Please re-open this ticket :) I will submit a PR to fix.

[ktoso]

Commented more in the PR: #249
We can't do source breaking changes like this for quite a while.