revanced-patches icon indicating copy to clipboard operation
revanced-patches copied to clipboard

feat: A shared patch that removes all FLAG_SECURE references from an app

Open bennett-sh opened this issue 2 years ago • 8 comments

Application

shared (all apps)

Issue

Currently, many apps like banking apps, browsers etc. block screenshotting. This can be really annoying sometimes when you just want to e.g. send someone a screenshot of your IBAN in your banking app.

Patch

I'm unsure but I think you'd have to a) remove all FLAG_SECURE's from the manifest and if it's possible, you'd also have to scan the bytecode for FLAG_SECURE and remove the lines.

Motivation

I already described my motivation in the Issue-field. It's that I want to be able to screenshot anything even though it may risk my personal security.

Acknowledgements

  • [X] I have searched the existing issues and this is a new and no duplicate or related to another open issue.
  • [X] I have written a short but informative title.
  • [X] I filled out all of the requested information in this issue properly.

bennett-sh avatar Jan 06 '23 00:01 bennett-sh

While it's probably not the solution you're looking for I suggest https://github.com/VarunS2002/Xposed-Disable-FLAG_SECURE as a temporary replacement.

pacjo avatar Jan 08 '23 13:01 pacjo

I think most banking apps don’t work when they are modified because they have some checks in place.

Linus789 avatar Jan 08 '23 20:01 Linus789

I didn't have this issue, but the banking app I use isn't very demanding in terms of security (doesn't require hiding root and I even got it working without passing safetynet).

pacjo avatar Jan 09 '23 07:01 pacjo

My bank is also really dumb about security so it would be worth a try.

On Mon, Jan 9, 2023, 08:20 pacjo @.***> wrote:

I didn't have this issue, but the banking app I use isn't very demanding in terms of security (doesn't require hiding root and I even got it working without passing safetynet).

— Reply to this email directly, view it on GitHub https://github.com/revanced/revanced-patches/issues/1434#issuecomment-1375198972, or unsubscribe https://github.com/notifications/unsubscribe-auth/A2NWAWQTVDRNVJJHUFBOJ2DWRO33LANCNFSM6AAAAAATSRTIV4 . You are receiving this because you authored the thread.Message ID: @.***>

bennett-sh avatar Jan 09 '23 08:01 bennett-sh

I believe it can be closed now as pr 1455 is merged. ReVanced/revanced-patches-template#1522 can be closed too.

pacjo avatar Feb 25 '23 11:02 pacjo

This issue is for all secure flags.

oSumAtrIX avatar Feb 25 '23 13:02 oSumAtrIX

Sorry, wasn't aware. How about ReVanced/revanced-patches-template#1522 ?

pacjo avatar Feb 25 '23 15:02 pacjo

Closed

oSumAtrIX avatar Feb 25 '23 15:02 oSumAtrIX