ferdium / ferdium-app

All your services in one place, built by the community
https://ferdium.org
Apache License 2.0
2.84k stars 170 forks source link

Update Messenger link handling (to remove FB messenger 'cruft') #1440

Open mjeveritt opened 11 months ago

mjeveritt commented 11 months ago

Preflight Checklist

Ferdium Version

6.4.1

Recipe Name

Messenger

Recipe Version

1.8.0

Last Known working version of the recipe

No response

Steps to reproduce

This isn't a bug per-se, more a feature-request. Steps to reproduce:

Expected Behavior

Links can be copied without any tracking info or warning 'cruft' inserted by Facebook.

See https://github.com/pfych/FacebookMessengerCruftCleaner/blob/master/Remove%20Messenger%20Tracker.user.js for potential implementation.

Actual Behavior

FB encapsulates links with a prefix "https://l.messenger.com/l.php?" on all it's links, and then escapes the link preventing you from extracting it when you copy-paste elsewhere (ie. when you open said link in a browser, etc, not in Ferdium itself).

The result is you get a Facebook page with a warning which you have to click to confirm, before you can visit the link sent.

Screenshots

image

Additional Information

I've seen a few scripts/plugins that do this for browsers/etc .. but is it something that is worth incorporating in Ferdium too?

It probably needs to be an option that could be toggled on/off ?! (possibly default to Off?!)

Related to https://github.com/ferdium/ferdium-recipes/pull/446 and specifically https://github.com/ferdium/ferdium-recipes/pull/446#issuecomment-1792550878 .

mjeveritt commented 11 months ago

Can remove 'bug' tag and replace with 'enhancement' tag (no permissions here!)