Skip to content

Add warning about default routes#804

Merged
wgreenberg merged 1 commit intomainfrom
ip-route
Jan 26, 2026
Merged

Add warning about default routes#804
wgreenberg merged 1 commit intomainfrom
ip-route

Conversation

@untitaker
Copy link
Collaborator

@untitaker untitaker commented Jan 25, 2026

Screenshot_20260125_224042

I hope this puts a lot of questions about SIM cards to rest. I found
that the warning also sometimes applies to "dead" SIM cards which have
expired a long time ago.

Run busybox ip route to determine whether the device has an active SIM
card. That command has been manually tested on Moxee, Orbic and TP-Link.
It's prefixed with busybox because that makes it more likely it would
work on UZ801, though it wasn't tested there. If the command invocation
fails, the alert is suppressed and a warning is logged.

The command is only run once on pageload. It could've been part of the
status endpoint, but then the UI would poll it way too often.

Fix #345

I hope this puts a lot of questions about SIM cards to rest. I found
that the warning also sometimes applies to "dead" SIM cards which have
expired a long time ago.

Run `busybox ip route` to determine whether the device has an active SIM
card. That command has been manually tested on Moxee, Orbic and TP-Link.
It's prefixed with `busybox` because that makes it more likely it would
work on UZ801, though it wasn't tested there. If the command invocation
fails, the alert is suppressed and a warning is logged.

The command is only run once on pageload. It could've been part of the
status endpoint, but then the UI would poll it way too often.
@untitaker untitaker marked this pull request as ready for review January 25, 2026 22:02
@wgreenberg
Copy link
Collaborator

i like this feature idea a lot, but i wonder if there's a more portable solution available. i ran strace ip route locally, and looks like it gets all of that info via the netlink socket API. i found a project that looks pretty promising for using that API within rust called neli, and here's an example that uses it to get link IPs: https://github.com/jbaublitz/neli/blob/main/examples/getips.rs

that said, i wager this is good enough for now, so i'll go ahead and merge and open an issue to use the more portable version. thanks untitaker!

@wgreenberg wgreenberg merged commit 9ae1563 into main Jan 26, 2026
25 checks passed
@wgreenberg wgreenberg deleted the ip-route branch January 26, 2026 19:08
@kmille
Copy link
Contributor

kmille commented Feb 4, 2026

A bit feedback: It's a bit annoying dismissing the pop up after each reload. It would be nice if this pop up is only shown once (after a reboot of the device).

@untitaker
Copy link
Collaborator Author

I'll rework the warnings next release if more people complain. to be honest I was thinking to even make the warning non-dismissable since the device is plainly unfit for purpose if it appears.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature Request]: Warn if no IP has been assigned from network

3 participants