Skip to content

Ping public DNS from the router UI to check connectivity#37

Open
pascalzauberzeug wants to merge 2 commits intomainfrom
check_for_internet
Open

Ping public DNS from the router UI to check connectivity#37
pascalzauberzeug wants to merge 2 commits intomainfrom
check_for_internet

Conversation

@pascalzauberzeug
Copy link
Copy Markdown
Contributor

@pascalzauberzeug pascalzauberzeug commented Apr 26, 2026

Motivation

The Teltonika router UI offered no way to verify whether the device actually has working internet, only whether interfaces are up.
This made diagnosing connectivity issues in the field cumbersome — interfaces can report healthy while DNS or upstream routing is broken.

Implementation

Adds a "Check Internet"-button to the Teltonika ui and the underlying functionality to the Teltonika hardware class.

Also converted the class-level tuples to frozenset since they are only used for membership checks.

Progress

  • I chose a meaningful title that completes the sentence: "If applied, this PR will..."
  • I chose meaningful labels (if GitHub allows me to so).
  • I documented breaking changes and set the 'breaking change' label if needed
  • The implementation is complete.
  • Tests with a real hardware have been successful (or are not necessary).
  • Pytests have been added (or are not necessary).
  • Documentation has been added (or is not necessary).

@pascalzauberzeug pascalzauberzeug added this to the 0.2.0 milestone Apr 26, 2026
@pascalzauberzeug pascalzauberzeug added the enhancement New feature or request label Apr 26, 2026
@pascalzauberzeug pascalzauberzeug changed the title Add internet connection check functionality Ping public DNS from the router UI to check connectivity Apr 26, 2026
@pascalzauberzeug pascalzauberzeug marked this pull request as ready for review April 26, 2026 15:27
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.

1 participant