Skip to content
This repository has been archived by the owner on Nov 13, 2024. It is now read-only.

Restore missing metrics & remove unused ones #26

Merged
merged 8 commits into from
Feb 23, 2024

Conversation

AndrewFerr
Copy link
Member

This PR merges in mautrix#463 to work with the changes to logout handling added by mautrix#439, which is not in upstream.

It also removes a metric that was exclusive to the Element fork, the usage of which was lost since an upstream refactoring to Signal message handling (mautrix@9efe31d) and doesn't seem to be important enough to warrant being restored.

Also use Signal UUIDs as keys instead strings
Decrement a user's connection/login count only if a count for that user
has been created.
instead of only when logged out
Mimic the naming scheme of mautrix-whatsapp metrics
as this metric is less useful now that the Signal event handler doesn't
directly monitor/return errors anymore
@AndrewFerr AndrewFerr self-assigned this Feb 23, 2024
@AndrewFerr AndrewFerr requested a review from a team as a code owner February 23, 2024 04:58
Copy link
Member

@Half-Shot Half-Shot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems like an improvement 👍

@AndrewFerr AndrewFerr merged commit bbf267e into element-main Feb 23, 2024
12 checks passed
@AndrewFerr AndrewFerr deleted the af/tidy-element-metrics branch February 23, 2024 15:00
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants