Conversation
|
Hey dude, nice fixes. I noticed the tileservice bug ages ago and I'm glad it's finally got a solution. Hope you can sign off on your commits so they can merge them asap and get this fixed! |
|
Thanks. It's not up to me to sign off on my commits. I submitted a PR for the Tailscale devs. They have to approve me PR and merge my change with master branch |
|
I meant sign off with DCO as it's been auto declined due to that. |
b1685e2 to
3bdf669
Compare
|
oh yea thanks for pointing that out. I think its fixed now |
…ing another VPN until Tailscale app is force closed because currentState=Ipn.State.Stopping on subsequent attempts to start TS Signed-off-by: Segi <segihovav@gmail.com>
9126695 to
6ebc7bb
Compare
Signed-off-by: Segi <segihovav@gmail.com>
4df1eb1 to
5457749
Compare
Signed-off-by: Segi <segihovav@gmail.com>
Signed-off-by: Segi <segihovav@gmail.com>
Signed-off-by: Segi <segihovav@gmail.com>
16af475 to
05a3065
Compare
|
is there anything else that I need to do to push this along ? |
|
Github: This branch has conflicts that must be resolved |
Signed-off-by: Segi <SegiH@users.noreply.github.com>
|
@mio-19 I just resolved it |
|
I don't know if it is related, but I have a similar issue where Tailscale sometimes randomly disconnects on network changes, and when this happens the toggle also breaks and I also have to force stop the app, so I really hope this gets merged and perhaps fixes my problem as well |
Updates #17190
After enabling another VPN app in Android, you cannot enable the Tailscale radio toggle until you force stop it. The radio button will not enter the enabled state.
Steps to recreate this issue:
When tracing this issue, I found that at step 3, a break point at this point shows that the value of currentState is Ipn.State.Stopping not Ipn.State.Stopped as you might expect. When you enable Tailscale initially in step 1, currentState is Ipn.State.Stopped.
Enabling/Disabling Tailscale is done in 2 places in the Android app and this PR is to fix this issue in 2 places:
I check if the value of currentState is Ipn.State.Stopping and if it is, call stopVPN().
I also added a note to the README that its required to run
make libtailscaleto make the Tailscale AAR necessary to build the Android Tailscale app.