chandevel/Clover

Delete the NFC permission that the app requires

Opened this issue · 8 comments

RiQuY commented

I downloaded the app from F-Droid and the app can control NFC on smarphones.
https://f-droid.org/app/org.floens.chan
Why an imageboard viewer requires this permission?
Delete this requirement as soon as posible.
Screenshot attached:
https://cdn.discordapp.com/attachments/428542615906942986/559919257119621121/Screenshot_2019-03-26-02-58-56-563_com.google.android.packageinstaller.png
Thanks.

Adamantcheese commented

NFC is only used to push the currently viewed thread to another device. You can see the code in StartActivity and ThreadController, methods named createNDefMessage(NfcEvent event).

RiQuY commented

NFC is only used to push the currently viewed thread to another device. You can see the code in StartActivity and ThreadController, methods named createNDefMessage(NfcEvent event).

I saw it in the code and I know you are not doing something shady, but I NFC is known to be used in contactless payment systems, so I find it a bit controversial when an app requires that permission.

Did you considered using bluetooth or something similar to push the currently viewed thread to another device?

Adamantcheese commented
  1. I'm not the owner of the repo.
  2. It was added 4 years ago. It's probably not going away.
    You can always build it yourself without the NFC code and sideload it.
RiQuY commented 
1. I'm not the owner of the repo.

2. It was added 4 years ago. It's probably not going away.
   You can always build it yourself without the NFC code and sideload it.

Do you mean suggesting a pull request?

Adamantcheese commented

You're free to do that as well.

Deleted user commented

Clover only uses NFC to share threads iirc, if you don't like that deny the permission or build clover without the permission.

Demonstrandum commented

@kakolisgay stop being a fag, and stop browsing 4chin.

floens commented

Beam is deprecated, we should remove it.