30 pointsby todsacerdoti6 days ago10 comments
  • tengbretson14 hours ago
    Libraries shouldn't log. Just have your top-level abstractions extend an EventEmitter base, emit appropriate events, and let the user do the rest.
    • rcxdude2 hours ago
      Isn't that basically what the average logging framework is?
    • roblh12 hours ago
      This is so simple and makes so much sense. I’ve seen a couple libraries that do something similar, but I feel like this is obvious and useful enough that it should just be a stock pattern, and it clearly isn’t.
    • ivan_gammel14 hours ago
      Why shouldn’t libraries log?
      • bitwizeshift13 hours ago
        Aside from what some other users have said, logging is fundamentally an observable side-effect of your library. It’s now a behavior that can become load-bearing — and putting it in library code forces this exposed behavior on the consumer.

        As a developer, this gets frustrating. I want to present a clean and coherent output to my callers, and poorly-authored libraries ruin that — especially if they offer no mechanism to disable it.

        It’s also just _sloppy_ in many cases. Well-designed library code often shouldn’t even need to log in the first place because it should clearly articulate each units side-effects; the composition of which should become clear to understand. Sadly, “design” has become a lost art in modern software development.

      • throwway12038514 hours ago
        It depends a lot on the language, but in my field libraries that have their own logging implementation and that don't provide hooks to override it cause big problems for me because I send all the logs to the same central logging client that forwards it to the same central logging server for aggregation. Your logging probably dumps it to a file, or it writes it to STDOUT, or something similar, in which case now I have to pipe all of that data in two places by doing something hacky.

        There are some language ecosystems that have good logging systems like Java which I would be totally fine with. But systems languages like C/C++ that don't have a core concept of a "log message" are a pain to deal with when the library author decides to stream some text message somewhere. Which is probably a good argument for not using those languages in some circles, but sometimes you have to use what you have.

        So it's not really a blanket "don't do it" but you should carefully consider whether there's some well-known mechanism for application authors to intake and manage your logging output, and if that doesn't exist you should provide some hooks or consider not logging at all except with some control.

      • jandrewrogers11 hours ago
        It tends to break composability.

        The behavior of the library logging can be incompatible with the architectural requirements of the application in dimensions that are independent of the library functionality. It requires the user to understand the details of the library's logging implementation. Even if documented, design choices for the logging will automatically disqualify the library for some applications and use cases.

        Is writing the log a blocking call? Does it allocate? Synchronous or async? What is the impact on tail latencies? How does it interact with concurrency? Etc.

      • mrkeen14 hours ago
        Because they have to be compatible with your logging implementation, and they were written first.
      • tengbretson14 hours ago
        Any log produced directly by a library will just be a "what" detached from any semblance of a "why".
      • charcircuit14 hours ago
        Because libraries don't know where or even how the user wants it to log.
        • Hackbraten12 hours ago
          That's why a library (if it absolutely must do logging) should always allow the user to optionally inject their own logger implementation.
  • malfist6 days ago
    Almost every logger in java operates this way. You set your library logging to debug and the end user and configure if they want debug logs from your library or not. They can even set context variables.
    • throwaway1505 days ago
      Python too. Honestly, any mature logger should allow embedding logs in library code that can be turned on or off by the end user. This was a solved problem 20 years ago. I honestly don't see what's so novel about this today. Or is this speaking to the sorry state of software engineering that plagues the JavaScript world?
      • danudey13 hours ago
        Rust also.

        Similar to this: OpenTelemetry has a golang library that allows you to instrument your library so that you can send traces, logs, etc. to an OTEL endpoint, but that instrumentation doesn't actually do anything unless the developer of the actual application uses a separate golang library to actually trigger, collect, and transmit those traces. Otherwise it's basically a no-op.

  • cowlby6 days ago
    This feels a bit like a pub/sub pattern; I wonder what it would look like with a full pub/sub implementation.
  • reactordev5 days ago
    At least you learned something
  • KellyCriterion5 days ago
    industry-proven and mature libs like LOG4J or LOG4Net are not sufficient?
    • Merad13 hours ago
      In the .Net space log4net is horrifically outdated and there's zero reason to use it today. Logging for modern .Net apps and libraries should be built on the Microsoft.Extensions.Logging abstractions which provide the type of features covered in TFA. They also provide a clear separation between generating log events in code and determining where & how logs are stored. For basic needs you can use simple log writers that tie in directly with MEL, or for advanced needs link MEL with Serilog so that you can use its sinks and log processing pipeline.
    • mrkeen14 hours ago
      It's Strings. They go somewhere. The interface writes itself: Consumer<String>.

      At my absolute fanciest, I use a Queue, some terminal colouring, separate stderr from stdout, and write some short-hand functions (warn, err, info, etc.).

      These are the bugs I don't have: https://github.com/apache/logging-log4j2/issues

    • hansvm5 days ago
      You mean this log4j [0] with major vulnerabilities the industry missed for nearly a decade?

      [0] https://en.wikipedia.org/wiki/Log4Shell

      • mashepp5 days ago
        So you don’t use any software that has had a security vulnerability?

        What operating system and browser did you use to write your post?

        • Veserv14 hours ago
          Unary thinking has no place when considering software quality or security. Just because things have vulnerabilities does not mean that the category, severity, and frequency of them is a irrelevant consideration.

          The Log4j vulnerability was effectively calling eval() on user input strings. That is utter incompetence to the extreme with immediately obvious, catastrophic consequences to anybody with any knowledge whatsoever of software security. That should be immediately disqualifying like a construction company delivering a house without a roof. "Oh yeah, anybody could forget to put a roof on a house. We can not hold them responsible for every little mistake." is nonsense. Basic, egregious errors are disqualifying.

          Now, it could be the case that everything is horribly defective and inadequate and everybody is grossly incompetent. That does not somehow magically make inadequacy adequate and appropriate for use. It is just that in software people get away with using systems unfit for purpose because they had "no choice but to use substandard components and harm their users so they could make money".

      • ivan_gammel14 hours ago
        If this library became untouchable for you, slf4j/logback is better and very popular alternative. I‘d say the design of slf4j is actually perfect.
      • KellyCriterion5 days ago
        Have you used ever OpenSSL? :-D

        The thing is: A bug does not invalidate enterprise adoption - Microsoft ist a good example.

        • hansvm5 days ago
          That was less my point, and more that "battle-tested" doesn't have to be a cudgel to argue against in-house projects, especially when considering defect rates (the more-general solution is very often slower and buggier to support the features you don't need).
          • KellyCriterion5 days ago
            Maybe we should differ the terms:

            "industry proven" -> MS/Windows -> yes

            "battle tested" -> MS Windows -> you may discuss? :-D

            If there is an inhouse solution available and which is really working, then Id not introduce an externa component here. If you start from zero, then using a pre-existing component should be the path, in my perception. Sure, one can waste time write a logger, but should have e.g. Bezos spent time coding on a logging lib or care about the webshop and use an existing lib for that - but in most cases it does not payoff to do whatever self-implementation-voodoo someone imagines: its just a waste of time. (Esp. since most companies do not take off enough to make such an investment plausible)

    • move-on-by13 hours ago
      The article is discussing JavaScript, are you discussing JavaScript?
  • bgoosman6 days ago
    Wasn't OpenTelemetry invented for this purpose?
    • ramon15613 hours ago
      The JS impl of OpenTel is awfully glued together. Sorry for the maintainers, but its inefficient and prometheus+logging will outmatch OT any day of the week (at least for JS)
      • koakuma-chan10 hours ago
        Which OpenTel impl isn't awfully glued together? When I looked at Rust implementation... I felt really bad. I feel like rolling my own logging is the only option.
  • seniorsassycat5 days ago
    I really want something like this to be built into the language or runtime, I don't want to juggle configuration for 4 different libraries. Log4j and tracing seem to be well established without being built in, but it feels too late for js.

    I'm curious if this is enough https://nodejs.org/api/diagnostics_channel.html

    I don't like the js hotel libraries, their docs feel deliberately obtuse

  • JSR_FDED11 hours ago
    So many knee jerk comments from people who have not read the article. The author describes how existing logging systems are geared towards applications, and sometimes libraries need a way of logging that provides more granularity and control - especially when the library spans multiple levels of abstraction.
  • drdec5 days ago
    Did you consider log4js?
  • callumgare5 days ago
    [dead]
    • monkpit15 hours ago
      The OP is about LogTape…