Skip to content

Multiplex adapter#79

Merged
preaction merged 11 commits intopreaction:masterfrom
GrantStreetGroup:multiplex-adapter
Jan 13, 2020
Merged

Multiplex adapter#79
preaction merged 11 commits intopreaction:masterfrom
GrantStreetGroup:multiplex-adapter

Conversation

@jrubinator
Copy link
Contributor

This adds a new Multiplex adapter to Log::Any, allowing sending messages through multiple Log::Any::Adapters.

The goal of this is be able to simultaneously use structured logging with multiple adapters (which Log::Any::Adapter::Dispatch can't quite do).

@preaction
Copy link
Owner

This looks great, thanks! I'm going to make one teensy-little change: s/adapters_and_args/adapters/g and then I'll merge and release this tonight.

@jrubinator
Copy link
Contributor Author

I just noticed this should close #16 :)

@preaction preaction merged commit 5c096fe into preaction:master Jan 13, 2020
preaction added a commit that referenced this pull request Jan 13, 2020
    [Added]

    - Added 'Capture' adapter for capturing log messages in an array or
      running an arbitrary callback. This makes it easier to test log
      messages, collect messages from a certain scope for later display.
      Thanks @nrdvana! [Github #77]

    - Added Log::Any::Adapter 'get' class method to get an adapter for
      a category. This allows bypassing the Proxy and using the Adapter
      directly. Doing so breaks all the features enabled by a Proxy, and
      so should only be used in special circumstances (definitely not in
      CPAN modules). Thanks @nrdvana! [Github #78]

    - Added 'Multiplex' adapter for directing log output to one or more
      Log::Any adapters. This allows, for example, basic logging to
      Stderr and logging to a remote Syslogd or Elastic. Thanks
      @jrubinator and @GrantStreetGroup for contributing this!
      [Github #79, Github #16]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants