Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unclickable links in RSS feeds since Dec 28, 2022 push to master tree #2426

Closed
3 tasks done
Antorell opened this issue Dec 31, 2022 · 4 comments
Closed
3 tasks done

Comments

@Antorell
Copy link

Please complete the following tasks.

  • Web browser cache cleared
  • Link provided to install script if applicable
  • Web browser, ruTorrent, PHP and OS version provided

Tell us about your environment

ruTorrent v4.0 beta (master Dec 28, 2022)
php 8.1
Ubuntu 22.10

Tell us how you installed ruTorrent

git clone from the master branch (Dec 28, 2022)

Describe the bug

Posting a bug report before I forget.

There's still an issue with the new RSS changes that were pushed to the master branch the Dec 28, 2022. The links in the description of the feeds are not clickable anymore. The links are colored like other links, or the links before the RSS changes, but they cannot be clicked. Feeds can be loaded without problems.

Happy new year.

Steps to reproduce

Load a RSS feed that has links to the release page or external links

Feed with links: https://feed.animetosho.org/rss2

Expected behavior

No response

Additional context

Screenshot (52)

@stickz
Copy link
Collaborator

stickz commented Dec 31, 2022

Hi @Antorell, this is intentional for security reasons. It could cause serious phishing implications if we don't disable links.

@Antorell
Copy link
Author

Antorell commented Jan 1, 2023

Yes, the other day I didn't check the sanitize.config.js in the js/ folder and thought the RESTRICTED keyword at the end of sanitize.config in the init.js of the RSS plugin was fixed feature and not an option.

I switched RESTRICTED to BASIC in the init.js of the RSS plugin, and it works like before the sanitize changes. I also added a _blank parameter to add attribute so links open in a new tab.

   add_attributes: {
     'a': {
            'rel': 'nofollow',
	    'target': '_blank'   
          }
   },

Cheers

@Antorell Antorell closed this as completed Jan 1, 2023
@Novik
Copy link
Owner

Novik commented Jan 1, 2023

@Antorell
AFAIK, in this case 'nofollow' should be replaced with 'noreferrer noopener'.

@Antorell
Copy link
Author

Antorell commented Jan 1, 2023

I will, thanks.

@Antorell Antorell changed the title Unlickable links in RSS feeds since Dec 28, 2022 push to master tree Unclickable links in RSS feeds since Dec 28, 2022 push to master tree Jan 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants