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

[Bug]: RSS Feeds not respecting "Prevent feed from being indexed by itunes and google" #2047

Closed
DieselTech opened this issue Aug 31, 2023 · 7 comments
Labels
bug Something isn't working

Comments

@DieselTech
Copy link

Describe the issue

Turning on Prevent feed from being indexed by itunes and google doesn't apply the protections it says it does. This is confirmed both by the RSS status window showing "Prevent Indexing: No" and by checking the RSS feed itself.

<itunes:block>No</itunes:block>
<googleplay:block>no</googleplay:block>

Steps to reproduce the issue

  1. Turn on a feed
  2. Check the box to prevent
  3. View Feed

Audiobookshelf version

2.3.3

How are you running audiobookshelf?

Docker

@DieselTech DieselTech added the bug Something isn't working label Aug 31, 2023
@advplyr
Copy link
Owner

advplyr commented Sep 8, 2023

I wasn't able to reproduce this in my testing. With the prevent indexing checked I get:

<itunes:block>Yes</itunes:block>
<googleplay:block>yes</googleplay:block>

I did some reading on this though and made minor updates. I updated them to use lowercase "yes" and when not preventing indexing those tags won't be present at all. Using "no" in that tag is useless.

Are you still able to reproduce this on 2.3.3?

@advplyr advplyr added the unable to reproduce Issue is not yet reproducible label Sep 8, 2023
@DieselTech
Copy link
Author

Are you still able to reproduce this on 2.3.3?

Yes. I even turned off existing feeds that were made on prior versions and recreated them on 2.3.3

The behavior is 100% reproducible on my instance using the latest docker container.

@advplyr
Copy link
Owner

advplyr commented Sep 8, 2023

Are you able to get it to say "Yes" here at all?

image

@DieselTech
Copy link
Author

Are you able to get it to say "Yes" here at all?

Nope. No combo of actions I could take would ever get it to show up as Yes

@nichwall
Copy link
Contributor

I was just able to recreate this using a collection on 5c1c511 (edge on October 27 2023, most recent release was 2.4.4). When opening an RSS feed from a single book the Prevent Indexing was set correctly, but was not set for a collection with two books.
image

@advplyr advplyr added awaiting release Issue is resolved and will be in the next release and removed unable to reproduce Issue is not yet reproducible labels Oct 28, 2023
@advplyr
Copy link
Owner

advplyr commented Oct 28, 2023

Thanks, I found it. It was specific to collections & series

@advplyr
Copy link
Owner

advplyr commented Oct 29, 2023

Fixed in v2.5.0

@advplyr advplyr closed this as completed Oct 29, 2023
@advplyr advplyr removed the awaiting release Issue is resolved and will be in the next release label Oct 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants