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

net block stays unresponsive if device path does not exist at startup #365

Closed
p-hamann opened this issue Apr 15, 2019 · 2 comments · Fixed by #369
Closed

net block stays unresponsive if device path does not exist at startup #365

p-hamann opened this issue Apr 15, 2019 · 2 comments · Fixed by #369

Comments

@p-hamann
Copy link
Contributor

if the tun0 interface is missing before starting i3status-rust, then the net block for tun0 device will not respond after enabling tun0 interface. I can only fix this by restarting i3status-rust after enabling the tun0 interface.

@p-hamann p-hamann changed the title net block partially unresponsive for tun0 interface net block stays unresponsive if device path does not exist at startup time Apr 15, 2019
@p-hamann p-hamann changed the title net block stays unresponsive if device path does not exist at startup time net block stays unresponsive if device path does not exist at startup Apr 15, 2019
@atheriel
Copy link
Collaborator

I thought something like this might crop up, although I'm not sure how to fix it. We avoid crashing on missing interfaces at startup to avoid annoying users.

@p-hamann
Copy link
Contributor Author

It is noticeable that if exists(&self) returns true on startup, the directory (/sys/class/tun0) can be removed again by stopping the VPN connection, but the block still remains responsive. Only if the directory does not exist when starting i3status-rust, the block remains unrespronsive.
Otherwise, do you know a workaround, so that the tun0 directory exists, without VPN gets started before?

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

Successfully merging a pull request may close this issue.

2 participants