Prometheus metrics not being updated?

The metrics bridge_connected & bridge_logged_in are always 0 for Signal whereas they're - as expected 1 for the Whatsapp bridge.

Other metrics appear to work fine, e.g. bridge_count_collection & bridge_messages_total appear to be updated properly. I may be missing something since I don't know the codebase at all, but TrackConnectionState doesn't seem to be used anywhere in mautrix/signal & mautrix/go:

signal/metrics.go

Line 212 in 25ba56b

    
           func (mh *MetricsHandler) TrackConnectionState(signalID string, connected bool) {

I'm currently on 1036669.

My configuration looks like this:

signal.yml

homeserver:
    address: https://nicht-so.sexy
    domain: nicht-so.sexy
appservice:
    address: http://bridges.soost.wireguard.ist.nicht-so.sexy:8082
    hostname: bridges.soost.wireguard.ist.nicht-so.sexy
    port: 8082
    database:
        type: postgres
        uri: postgres:///signalbridge?sslmode=disable&host=/run/postgresql
    id: signal
    bot:
        username: signalbot
        displayname: Signal bridge bot
        avatar: mxc://maunium.net/wPJgTQbZOtpBFmDNkiNEMDUp
    ephemeral_events: true
    as_token: redacted
    hs_token: redacted
metrics:
    enabled: true
    listen: 0.0.0.0:7981
signal:
    device_name: mautrix-signal
bridge:
    username_template: "signal_{{.}}"
    displayname_template: '{{or .ProfileName .PhoneNumber "Unknown user"}}'
    note_to_self_avatar: mxc://maunium.net/REBIVrqjZwmaWpssCZpBlmlL
    portal_message_buffer: 128
    delivery_receipts: false
    public_portals: false
    provisioning:
        prefix: /_matrix/provision
        shared_secret: generate
        debug_endpoints: false
    command_prefix: "!signal"
    permissions:
        '*': relay
        '@ma27:nicht-so.sexy': "admin"
        nicht-so.sexy: "user"
logging:
    min_level: debug
    writers:
    - type: stdout
      format: pretty-colored

But perhaps I'm just holding it wrong?

Looks like we just didn't implement this. We don't use these metrics at Beeper, so it probably got lost during the rewrite.