Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Use host:port for server logging and add extra fields to EventCollector logs #1428

Open
wants to merge 2 commits into
base: v1.20
Choose a base branch
from
Open

Use host:port for server logging and add extra fields to EventCollector logs #1428

wants to merge 2 commits into from

Commits on Sep 19, 2024

  1. Track servers by host:port instead of serverConnectionId 

    The server's connection ID is unique within the context of a single server and is only reported by MongoDB 4.2+. It should not be used to uniquely identify a server.

This reverts recent changes from 50d1db7.
    @jmikola
    jmikola committed Sep 19, 2024
    Configuration menu
    Copy the full SHA
    c2680d2 View commit details
    Browse the repository at this point in the history

  2. Rename connectionId and add extra fields to EventCollector logs 

    This renames connectionId to server, to avoid ambiguity with serverConnectionId.

Additional fields have also been added to the output. Since PHPC 1.19, database name is available on all APM events.
    @jmikola
    jmikola committed Sep 19, 2024
    Configuration menu
    Copy the full SHA
    bd6b38c View commit details
    Browse the repository at this point in the history