Network monitoring refactor #1079
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Added a new service to the plugin abstraction to monitor network availability (`IConnectivityService`). You can use it to check the current network availability, add an event handler to get updates when the availability changed, and to check if we're currently HTTP/UDP banned (all through the plugin abstraction!). - Added a new enum (`NetworkAvailability`) for the network connectivity state. Possible states are; `-` No Interfaces — No interfaces were found. `-` No Gateways — No gateways were found. `-` Local Only — LAN is available, the Internet is not. `-` Partial Internet — Some Internet services are reachable. `-` Internet — All Internet services are reachable. - Added a new SignalR event stream (`"network"`) to subscribe to for the current network availability and any changes to the network availability. - Refactored the connectivity monitor job track both LAN and WAN availability. - Fixed up the queue items disabled status in the v3 API. - Add network monitoring to the queues, so any queue commands that requires internet access will now wait until the internet is available before running. - Added a command to forcefully get anime online, and made it so the normal get anime command will try to run under normal conditions, but schedule a force command if it's unable to get the anime. - Small update to the TvDB helper so it will only try to update the series when linking if network access is available, otherwise it will schedule an update for later. - Switched the WAN connectivity monitors to log at the trace level leading to less spam in the log files (though they will still appear on the console for docker users).
- Removed some outdated usage of an older sentry DSN, and updated the ASP.NET Sentry middleware to a) conditionally be activated based on the user's preference, and b) updated it to the correct settings.
- Notify any of the queues that have commands left that they can start again if they're waiting by making them re-announce their queue count.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Added a new service to the plugin abstraction to monitor network
availability (
IConnectivityService). You can use it to checkthe current network availability, add an event handler to get
updates when the availability changed, and to check if we're
currently HTTP/UDP banned (all through the plugin abstraction!).
Added a new enum (
NetworkAvailability) for the networkconnectivity state. Possible states are;
No Interfaces — No interfaces were found.
No Gateways — No gateways were found.
Local Only — LAN is available, the Internet is not.
Partial Internet — Some Internet services are reachable.
Internet — All Internet services are reachable.
Added a new SignalR event stream (
"network") to subscribe to forthe current network availability and any changes to the network
availability.
Refactored the connectivity monitor job track both LAN and WAN
availability.
Fixed up the queue items disabled status in the v3 API.
Add network monitoring to the queues, so any queue commands that
requires internet access will now wait until the internet is available
before running.
Added a command to forcefully get anime online, and made it so the
normal get anime command will try to run under normal conditions, but
schedule a force command if it's unable to get the anime.
Small update to the TvDB helper so it will only try to update the
series when linking if network access is available, otherwise it will
schedule an update for later.
Switched the WAN connectivity monitors to log at the trace level
leading to less spam in the log files (though they will still appear
on the console for docker users).
Removed some outdated usage of an older sentry DSN, and updated
the ASP.NET Sentry middleware to a) conditionally be activated
based on the user's preference, and b) updated it to the correct
settings.
Notify any of the queues that have commands left that they can start
again if they're waiting by making them re-announce their queue count.