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

Commit

Permalink
bug: Add missing metric for notification encoding
Browse files Browse the repository at this point in the history
Added "updates.notification.encoding.{content-encoding}",
we already track vapid as "updates.vapid.{version}"

Closes #992
  • Loading branch information
jrconlin committed Aug 17, 2017
1 parent b098fd1 commit 74fbf58
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions autopush/web/webpush.py
Original file line number Diff line number Diff line change
Expand Up @@ -477,6 +477,9 @@ def post(self,
encoding = ''
if notification.data and notification.headers:
encoding = notification.headers.get('encoding', '')
self.metrics.increment(
"updates.notification.encoding.{}".format(encoding)
)
self._client_info.update(
message_id=notification.message_id,
uaid_hash=hasher(user_data.get("uaid")),
Expand Down

0 comments on commit 74fbf58

Please sign in to comment.