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.

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

[Docs] Add network flows that should be authorized #3886

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

Conversation

fl42
Copy link

@fl42 fl42 commented May 9, 2024

Hello,

Some users would like to restrict network flows, but currently it's hard to determine which flow should be authorized.
Therefore, keeping a list of remote services (URLs) required for Teslamate is useful.

Furthermore, blocking network traffic leads to error messages that are not obviously related to network.
For instance, if streaming API is not reachable, Teslamate logs are:

2024-05-09 15:58:15.849 car_id=1 [info] Start / :online
2024-05-09 15:58:15.907 car_id=1 [info] Connecting ...
2024-05-09 15:58:16.197 car_id=1 [info] Start / :online
2024-05-09 15:58:16.233 car_id=1 [info] Connecting ...
2024-05-09 15:58:16.583 car_id=1 [info] Start / :online
2024-05-09 15:58:16.636 car_id=1 [info] Connecting ...
2024-05-09 15:58:16.940 car_id=1 [info] Start / :online
2024-05-09 15:58:16.979 car_id=1 [info] Connecting ...
2024-05-09 15:58:17.427 car_id=1 [info] Start / :online
2024-05-09 15:58:17.440 car_id=1 [info] Connecting ...
2024-05-09 15:58:17.780 car_id=1 [info] Start / :online
2024-05-09 15:58:17.818 car_id=1 [info] Connecting ...

And PostgreSQL logs are:

ERROR:  canceling statement due to user request

Log messages don't allow straightforward identification of the root cause (but this is to be improved in another PR ;) )

Copy link

netlify bot commented May 9, 2024

Deploy Preview for teslamate ready!

Name Link
🔨 Latest commit 0861ab3
🔍 Latest deploy log https://app.netlify.com/sites/teslamate/deploys/663cf19ebd816f0008533392
😎 Deploy Preview https://deploy-preview-3886--teslamate.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@JakobLichterfeld JakobLichterfeld added the enhancement New feature or request label May 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants