FrostCo / AdvancedProfanityFilter

A browser extension to filter profanity from webpages
GNU General Public License v3.0
159 stars 26 forks source link

Crunchyroll #502

Closed Commonman7 closed 1 year ago

Commonman7 commented 1 year ago

Just went back onto chrunchyroll today and it is not filtering. I've tried it on chrome, firefox and edge browers with the same results. Thanks

richardfrost commented 1 year ago

Could you share a link to the video you tested with @Commonman7?

I don't personally have a subscription to Crunchy roll right now, so I've just been testing with this video: https://www.crunchyroll.com/watch/G7PU4VVV4/its-not-okay-the-spider-lures-sanji and it seems to be working okay on it for me. Does it work for you?

Commonman7 commented 1 year ago

Haven’t tried your link yet but I will Here’s what I was watching

https://crunchyroll.com/watch/GPWUKDJ19?utm_medium=ios&utm_source=share

On Thu, May 11, 2023 at 10:41 PM Richard Frost @.***> wrote:

Could you share a link to the video you tested with @Commonman7 https://github.com/Commonman7?

I don't personally have a subscription to Crunchy roll right now, so I've just been testing with this video: https://www.crunchyroll.com/watch/G7PU4VVV4/its-not-okay-the-spider-lures-sanji and it seems to be working okay on it for me. Does it work for you?

— Reply to this email directly, view it on GitHub https://github.com/FrostCo/AdvancedProfanityFilter/issues/502#issuecomment-1545025730, or unsubscribe https://github.com/notifications/unsubscribe-auth/AUQZIBHLUPROZHIBXME6QBLXFWPMVANCNFSM6AAAAAAX632G34 . You are receiving this because you were mentioned.Message ID: @.***>

richardfrost commented 1 year ago

Thanks for sending the link @Commonman7. When you get a chance, can you see if episode 1 works? https://www.crunchyroll.com/watch/GG1U2QV9E/ryusui

If that does work for you, but episode 6 doesn't I'd guess they are doing something different between their free titles and the premium ones. If that's the case I'll re-sub for a month and take a look at it.

Commonman7 commented 1 year ago

Just tried episode one and it didn’t work. I have also tried a different device just in case and still no luck.

Thanks On Thu, May 11, 2023 at 11:45 PM Richard Frost @.***> wrote:

Thanks for sending the link @Commonman7 https://github.com/Commonman7. When you get a chance, can you see if episode 1 works? https://www.crunchyroll.com/watch/GG1U2QV9E/ryusui

If that does work for you, but episode 6 doesn't I'd guess they are doing something different between their free titles and the premium ones. If that's the case I'll re-sub for a month and take a look at it.

— Reply to this email directly, view it on GitHub https://github.com/FrostCo/AdvancedProfanityFilter/issues/502#issuecomment-1545065584, or unsubscribe https://github.com/notifications/unsubscribe-auth/AUQZIBEF7YRKSODHEWXYRHTXFWW43ANCNFSM6AAAAAAX632G34 . You are receiving this because you were mentioned.Message ID: @.***>

jaxmirox commented 1 year ago

Hi Richard, I've been emailing back and forth. I really appreciate you taking your time on this. Thanks!

richardfrost commented 1 year ago

Thanks for all the help in tracking down the issue @Commonman7 and @jaxmirox. I have identified the issue and will be working through getting it fixed in an update. I'm aiming to finish it up this evening and will update when I have submitted it for review by the various stores.

Commonman7 commented 1 year ago

Thanks for the update and for the great work you are doing.

On Sat, May 13, 2023 at 5:54 PM Richard Frost @.***> wrote:

Thanks for all the help in tracking down the issue @Commonman7 https://github.com/Commonman7 and @jaxmirox https://github.com/jaxmirox. I have identified the issue and will be working through getting it fixed in an update. I'm aiming to finish it up this evening and will update when I have submitted it for review by the various stores.

Reply to this email directly, view it on GitHub https://github.com/FrostCo/AdvancedProfanityFilter/issues/502#issuecomment-1546757063, or unsubscribe https://github.com/notifications/unsubscribe-auth/AUQZIBFPQF5G64FRMZO3WTTXF77JRANCNFSM6AAAAAAX632G34 . You are receiving this because you were mentioned.Message ID: @.***>

jaxmirox commented 1 year ago

Thank you sir. I really appreciate you doing all of this.

richardfrost commented 1 year ago

Thanks for all the encouragement and support @Commonman7 and @jaxmirox.

Sorry for the delay. This should be fixed in version 3.2.1. Feel free to comment here if you have any feedback for this update.

It is available now for Firefox, with the other stores following as soon as it passes their review process.

Commonman7 commented 1 year ago

Everything seems to be working fine. Thanks again

On Sun, May 14, 2023 at 5:58 PM Richard Frost @.***> wrote:

Thanks for all the encouragement and support @Commonman7 https://github.com/Commonman7 and @jaxmirox https://github.com/jaxmirox.

Sorry for the delay. This should be fixed in version 3.2.1 https://github.com/richardfrost/AdvancedProfanityFilter/releases/tag/3.2.1. Feel free to comment here if you have any feedback for this update.

It is available now for Firefox, with the other stores following as soon as it passes their review process.

— Reply to this email directly, view it on GitHub https://github.com/FrostCo/AdvancedProfanityFilter/issues/502#issuecomment-1547010075, or unsubscribe https://github.com/notifications/unsubscribe-auth/AUQZIBHGJS7N4O47EVGG443XGFIQ3ANCNFSM6AAAAAAX632G34 . You are receiving this because you were mentioned.Message ID: @.***>

jinzjuunanagou commented 1 year ago

I having problems getting this to work on crunchyroll, I see common got it to work. Just not sure how.

richardfrost commented 1 year ago

What are you seeing @jinzjuunanagou? First do you have the latest update (3.2.1)? Do you have any custom audio site config set for Crunchyroll (Options page > Audio tab)?

jinzjuunanagou commented 1 year ago

Ya I just downloaded it today so it should be latest version. But you might be right I be doing something wrong. So i have to custom audio for crunchy on the options page? I will give it a look.

jinzjuunanagou commented 1 year ago

Audio tab > Supported Sites Config > Copy/Pasted Crunchyroll still no luck.

richardfrost commented 1 year ago

@jinzjuunanagou, no, you shouldn't need custom audio config for it. You can remove it if you copied it. If you go to the "Help" tab you should be able to see the version in the top-right corner just to confirm you are on the latest (3.2.1). Also, which browser are you using?

Could you share a link to a video you are testing it on?

jinzjuunanagou commented 1 year ago

I am using Chrome. Sure I am trying to get it to work on https://www.crunchyroll.com/watch/G14U4E83J/stone-world 1st episode on the 11:02 min mark the CC censors actually work but no video mute happens.

Commonman7 commented 1 year ago

I logged onto Chrome tonight and it wouldn’t work. I’ve logged into Firefox and it’s working fine.

On Wed, May 17, 2023 at 6:03 PM jinzjuunanagou @.***> wrote:

I am using Chrome. Sure I am trying to get it to work on https://www.crunchyroll.com/watch/G14U4E83J/stone-world 1st episode on the 11:02 min mark the CC censors actually work but no video mute happens.

— Reply to this email directly, view it on GitHub https://github.com/FrostCo/AdvancedProfanityFilter/issues/502#issuecomment-1552146740, or unsubscribe https://github.com/notifications/unsubscribe-auth/AUQZIBESONKUR7QX6NZR36DXGVDJRANCNFSM6AAAAAAX632G34 . You are receiving this because you were mentioned.Message ID: @.***>

jaxmirox commented 1 year ago

Working fine on chrome and firefox for me.

update: I used the link provided and it didn't mute for me, but did censor. It must be a dub thing because it muted the japanese audio but not the english.

jaxmirox commented 1 year ago
dr stone english audio_english_cc

Not sure if this helps

richardfrost commented 1 year ago

Thanks for all the reports and details. I'll be taking a look in a few minutes and I'll update when I have something to report.

richardfrost commented 1 year ago

It looks like you are right @jaxmirox, they appear to be using yet another method for captions/subtitles for dubbed videos.

This custom config should work for the dubbed videos, but will prevent it working on the non-dubbed ones:

{
  "www.crunchyroll.com": [
    {
      "iframe": true,
      "mode": "elementChild",
      "note": "Dubbed videos",
      "parentSelector": "div#vilosVttJs",
      "subtitleSelector": ":scope div > div",
      "tagName": "DIV"
    }
  ]
}

I'm going to see if I can get them to both work at the same time now.

richardfrost commented 1 year ago

Okay, I think this config should support all the variations (at least that I've come across):

{
  "www.crunchyroll.com": [
    {
      "apfCaptions": true,
      "apfCaptionsSelector": "vilosVttJs",
      "displaySelector": "canvas#velocity-canvas",
      "externalSub": true,
      "externalSubTrackMode": "hidden",
      "externalSubVar": "window.v1config.media.playService.subtitles;;window.v1config.media.subtitles",
      "iframe": true,
      "mode": "cue",
      "videoCueLanguage": "en-US",
      "videoCueRequireShowing": false,
      "videoSelector": "video#player0"
    },
    {
      "iframe": true,
      "mode": "elementChild",
      "note": "Dubbed videos",
      "parentSelector": "div#vilosVttJs",
      "subtitleSelector": ":scope div > div",
      "tagName": "DIV"
    }
  ]
}

If anyone can give it a try and let me know how it works that would be great! Here are the videos I've been testing:

I included approximate timestamps for words that will be filtered with the default wordlist.

If this works, feel free to just use the above config until I can get another update published. I'll hold off with publishing this until I've heard back on how it works though.

jaxmirox commented 1 year ago

I’ll give this a try when I get home this afternoon and let you know.  Thanks for all the hard work!

jinzjuunanagou commented 1 year ago

I thank you much! This worked for me.

richardfrost commented 1 year ago

Thanks for reporting back @jinzjuunanagou, and once you get a chance to test it out @jaxmirox let me know and I'll work on getting this added (assuming it works for everyone).

jaxmirox commented 1 year ago

Will do.  I didn’t have a chance to last night but will try it today.  Thanks again

jaxmirox commented 1 year ago

This is working just fine for me as well. Thanks again!

On Fri, May 19, 2023 at 8:31 AM Jason Burton @.***> wrote:

Will do. I didn’t have a chance to last night but will try it today. Thanks again

Sent from my iPhone

On May 18, 2023, at 10:20 PM, Richard Frost @.***> wrote:



.Thanks for reporting back @jinzjuunanagou https://github.com/jinzjuunanagou, and once you get a chance to test it out @jaxmirox https://github.com/jaxmirox let me know and I'll work on getting this added (assuming it works for everyone).

— Reply to this email directly, view it on GitHub https://github.com/FrostCo/AdvancedProfanityFilter/issues/502#issuecomment-1553943812, or unsubscribe https://github.com/notifications/unsubscribe-auth/A72GXVC3KJV6P4ZR45OUV4TXG3RHPANCNFSM6AAAAAAX632G34 . You are receiving this because you were mentioned.Message ID: @.***>

richardfrost commented 1 year ago

Thanks for reporting back @jaxmirox & @jinzjuunanagou. I'll get this added into the next release and let you know here in this issue when its been released.

richardfrost commented 1 year ago

This latest config has been included in version 3.2.2. Feel free to comment here if you have any feedback for this update.