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

WIP : Socket metrics #205

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

killian-mannarelli
Copy link
Contributor

Compte le nombre d'users connectés au socket chat.
Ouvre un nouveau serveur express dédié à cela
Expose le bon port dans les fichiers Docker du socket

!!!!!
Changement du ENV après cette PR
Ajout de :
SOCKET_METRICS_PORT=4001
!!!!!

@killian-mannarelli killian-mannarelli self-assigned this Dec 20, 2022
@killian-mannarelli killian-mannarelli linked an issue Dec 20, 2022 that may be closed by this pull request
@killian-mannarelli killian-mannarelli changed the title Socket metrics WIP : Socket metrics Dec 20, 2022
Copy link
Contributor

@celian-rib celian-rib left a comment

Choose a reason for hiding this comment

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

Ta pu tester ?

@@ -73,6 +73,7 @@ services:
SOCKET_PORT: '${SOCKET_PORT}'
ports:
- '127.0.0.1:${SOCKET_PORT}:4000'
- '127.0.0.1:${SOCKET_METRICS_PORT}:4001'
Copy link
Contributor

Choose a reason for hiding this comment

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

Update le README

@killian-mannarelli
Copy link
Contributor Author

Ta pu tester ?

Nope, pas encore

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.

Socket Metrics
3 participants