adolfintel / OpenPods

The Free and Open Source app for monitoring your AirPods on Android
https://fdossena.com/?p=openPods/index.frag
GNU General Public License v3.0
957 stars 161 forks source link

Issues after restart #23

Closed N72826 closed 5 years ago

N72826 commented 5 years ago

I am having problems with OpenPods sending notifications after a restart. I am on a Oneplus 7 pro and I have to reinstall the OpenPods app to get it working again.

adolfintel commented 5 years ago

I cannot reproduce the problem, is there anything specific to your ROM or unusual installed apps that could cause this? (power saving, permission managers, ...)

N72826 commented 5 years ago

I have not changed the factory ROM build, it should be Oxygen OS 9.5.9.GM31CB . I have some other apps from f-droid: Blokada VPN(Isn't on all the time), Clover, Download Navi, Etar, OSMAnd+, Moonlight, NewPipe, Orbot, Slide, and Tachiyomi. Mind you the first thing I looked for when swapping from Iphone was an airpods notification app, I downloaded your app a while back before I had most of these other apps installed and it still did not work then. I just reinstalled it recently to see if you had fixed it but it seems like its my problem not the app's.

adolfintel commented 5 years ago

I don't know, it's the first time I hear about this problem.

The app is registered to start on boot, if for some reason it doesn't receive it, there's little I can do about this. The only alternative I can suggest is to not hide the app icon and run OpenPods manually after booting the phone, this should be enough to start the service.

N72826 commented 5 years ago

Thank you for the quick replies. I had thought of doing it manually but to me it's kind of counter-intuitive. I just wanted to be able to startup my phone and have it launch as described. Thank you for helping.

adolfintel commented 5 years ago

Well the app is supposed to start on boot, and it does on every device that I tested it on.

Unless you explicitly denied the permission to start on boot in the settings, the bug is probably caused by OxygenOS and you should report it on their forum, and maybe find some other more popular app that does not properly start on boot.

adolfintel commented 5 years ago

Also, if you know how to use logcat, I can send you a debug build of the app and we can see if anything interesting happens on boot.

N72826 commented 5 years ago

As I said I have just swapped from iphone so I am not too familiar with the ins and outs of android, I may still be able to follow instructions with the debug build installed but I'd have to do it when I come home today. Thanks again I will be messaging you later today.

adolfintel commented 5 years ago

Logcat is a development tool so it's part of android studio so it's not the easiest thing to do. If you don't want to do this, it's fine, but here's the instructions if you want to try:

Here's a screenshot of what it should look like (it will be very long): image

N72826 commented 5 years ago

image image image Rebooted twice and got this error both times. Sorry for the late reply, had to fiddle with android studio for a bit and was a tad pre-occupied.

adolfintel commented 5 years ago

Thanks for taking the time to do that, I will investigate this error ASAP.

adolfintel commented 5 years ago

I've implemented an alternative method to start on boot. Here's a test APK for you: app-debug.zip

It's a debug build so you can use logcat on it, you just install it and connect your phone, no need to press play.

Before you report it as working, please test it for a few hours because I suspect this breaks more things than it fixes.

adolfintel commented 5 years ago

New test version, should be more stable: app-debug.zip

N72826 commented 5 years ago

Hell yeah no problem as long as I could help!

N72826 commented 5 years ago

Thank you I will try these out later, today is Friday so I am open all weekend to fix this after I come home.

On Fri, Aug 16, 2019, 5:38 AM Federico Dossena notifications@github.com wrote:

New test version, should be more stable: app-debug.zip https://github.com/adolfintel/OpenPods/files/3508973/app-debug.zip

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/adolfintel/OpenPods/issues/23?email_source=notifications&email_token=AFAYEX5QGJ2HHMTFH62RYETQEZYSVA5CNFSM4IL5JWUKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4OFRQA#issuecomment-521951424, or mute the thread https://github.com/notifications/unsubscribe-auth/AFAYEX5BRMAVZL7OX3MOFKLQEZYSVANCNFSM4IL5JWUA .

N72826 commented 5 years ago

I just finished a quick test before leaving and Im pretty sure whatever you did just fixed the problem, I will test it again but it works after a restart.

adolfintel commented 5 years ago

So far it seems to work but it stops receiving bluetooth events after a while for some dumb reason. I have to figure that out then I'll post another test version

N72826 commented 5 years ago

Ok, I'm running the build on my phone and will tell you what happens throughout the day when I get back.

On Fri, Aug 16, 2019, 8:37 AM Federico Dossena notifications@github.com wrote:

So far it seems to work but it stops receiving bluetooth events after a while for some dumb reason. I have to figure that out then I'll post another test version

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/adolfintel/OpenPods/issues/23?email_source=notifications&email_token=AFAYEXZNKKGQZAGSWSKVXQ3QE2NQ7A5CNFSM4IL5JWUKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4OQDJI#issuecomment-521994661, or mute the thread https://github.com/notifications/unsubscribe-auth/AFAYEX24W5WZQRZX6PPIIITQE2NQ7ANCNFSM4IL5JWUA .

adolfintel commented 5 years ago

Last one, I promise: app-debug.zip

This one is based on a workaround for your problem that I found in another repo.

N72826 commented 5 years ago

It's seriously no problem I enjoy the troubleshooting. The old debug build I installed this morning just stopped working like you said it would, I will have to wait until around 4 to install the newer version when I get home. I appreciate it.

On Fri, Aug 16, 2019, 10:31 AM Federico Dossena notifications@github.com wrote:

Last one, I promise: app-debug.zip https://github.com/adolfintel/OpenPods/files/3509998/app-debug.zip

This one is based on a workaround for your problem that I found in another repo.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/adolfintel/OpenPods/issues/23?email_source=notifications&email_token=AFAYEX7QYCQAJZTEQLGVRGLQE224FA5CNFSM4IL5JWUKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4OYXTQ#issuecomment-522030030, or mute the thread https://github.com/notifications/unsubscribe-auth/AFAYEX5CUB2LYOAN5HWXLQ3QE224FANCNFSM4IL5JWUA .

N72826 commented 5 years ago

image image Looks like you struck money with that last one, doesn't look like this debug build is causing any errors but I will let the android dev be the judge.

adolfintel commented 5 years ago

So this new version works properly on your phone?

Also what is up with that error about not having ACCESS_COARSE_LOCATION? Did you deny the permission?

N72826 commented 5 years ago

This new version works just fine on my phone, I have not denied the app any permissions so I have no clue as to why it's giving me that error. Should I be using the debug build on the daily or wait for you to update it through github?

adolfintel commented 5 years ago

Well, what matters is that it works. Do some more testing in the next 1-2 days if you can, then I'll release it officially with some other minor improvements.

adolfintel commented 5 years ago

Thanks for helping me sort this out.

N72826 commented 5 years ago

Sounds good! No problem.

N72826 commented 5 years ago

Screenshot_20190817-011451 Should I have the blank openpods notification in the notification center without the airpods connected and Bluetooth off? Photo attached is what I'm talking about.

adolfintel commented 5 years ago

No, that's definitely not supposed to be there. Does it go away if you connect your airpods or turn bluetooth on/off?

adolfintel commented 5 years ago

Here's another test version: app-debug.zip

If this doesn't work, I'm completely out of ideas and you'll have to report this issue to oneplus

adolfintel commented 5 years ago

I just had another idea. This version is the same as the one on F-Droid, except it tells the system that it's from the Android 6.0 era, before that error message was introduced. See if it works:app-debug.zip

N72826 commented 5 years ago

I just had another idea. This version is the same as the one on F-Droid, except it tells the system that it's from the Android 6.0 era, before that error message was introduced. See if it works:app-debug.zip

This build has been working flawlessly but I will let you know what happens (if anything) by tomorrow.

adolfintel commented 5 years ago

Awesome. This build has the added advantage of not being allowed on Google Play so assholes can't sell it (it happened)

N72826 commented 5 years ago

That is really gross, did you just see your app being sold for money one day on the store? Here is a pastebin of the app running

adolfintel commented 5 years ago

More than once, I had to send at least 5 or 6 DMCA takedowns in the last 5 months

N72826 commented 5 years ago

Thank you for all of the wonderful help, I enjoyed learning how to debug software on android. If the problem occurs again I will reopen the thread but I am pretty sure you just fixed it for good. I wish you the best of luck in taking down all the copycats stealing the source code and re-uploading it, Thanks again for walking me through it and taking the time to fix a bug that could go easily ignored, I appreciate it.

adolfintel commented 5 years ago

No problem, thanks for helping me debug this issue

adolfintel commented 5 years ago

I was looking at the source code for GadgetBridge, an app to connect smartwatches and stuff like that, and I noticed that it uses a very similar mechanism to start on boot as OpenPods.

Can you try installing it from f-droid to see if it has the same problem when you reboot the phone? If it has the same bug, you'll know because you won't see a toast message saying "Bluetooth is disabled" after booting.

N72826 commented 5 years ago

I just installed GadgetBridge and after a reboot I got no toast notification saying bluetooth was disabled. Since I have closed the thread I do not receive notifications anymore, I would have done this sooner.

adolfintel commented 5 years ago

@Nicktheslick69 So it is a bug in OxygenOS. You should report it on their forum, tell them that GadgetBridge and OpenPods fail to start on boot with a permission error if the app has target SDK >= 26

N72826 commented 5 years ago

Unfortunately I have to wait 15 days until I can post on the oneplus forum app since I have just registered.

adolfintel commented 5 years ago

Wow that sucks

On August 18, 2019 5:09:50 PM GMT+02:00, Nicktheslick69 notifications@github.com wrote:

Unfortunately I have to wait 15 days until I can post on the oneplus forum app since I have just registered.

-- You are receiving this because you commented. Reply to this email directly or view it on GitHub: https://github.com/adolfintel/OpenPods/issues/23#issuecomment-522329912

-- Sent from my Android device with K-9 Mail. Please excuse my brevity.