Youtube content getting filtered on Squid

[JonathanLee]

@themaharshpatel thanks I will test soon.

[themaharshpatel]

@jonathanlee Have you tested?

[michmoor]

@themaharshpatel when i did implement Squid this was the reason i uninstalled the package. 'Restricted' under every youtube video. It was weird. Considered it a bug as i had safesearch disabled.
This is a great find if it works. Should be added to the documentation.
If its confirmed that it works, i will open a redmine for you if you havent done so already.

[themaharshpatel]

@michmoor It is working perfectly for me with this fix. I am just waiting for some confirmation before opening the issue on Redmine. I have already opened a PR on Github check out. As you were also facing it, I will go ahead and file the issue over there.

[michmoor]

@themaharshpatel thanks for this. albeit i am confused. if the header 'none' is being added why is that interpreted as restrictions being turned on?

[themaharshpatel]

@michmoor Because in its underlying PHP code where it is generating the config file for the squid. It is checking for a null value to skip the content filtration config. But here as there was none mentioned. It was considered as a string and the condition was not getting fulfilled to skip the config generation.

[JonathanLee]

@themaharshpatel

testing now

[michmoor]

@themaharshpatel great find/troubleshooting. simple fix. Maybe it can make it into the new release.

[themaharshpatel]

@jonathanlee do let us know if it works out! :)

[themaharshpatel]

@michmoor said in Youtube content getting filtered on Squid:

@themaharshpatel great find/troubleshooting. simple fix. Maybe it can make it into the new release.

Thanks:) Lets see how it works out!

[JonathanLee]

@themaharshpatel

Fixed it great find with this!!

[themaharshpatel]

@jonathanlee

[JonathanLee]

@themaharshpatel

Now Testing with restrictions enabled

Header restrictions functional

Thank !!! this option now has the ability to be enabled and disabled.

[michmoor]

@themaharshpatel I can confirm this is fixed as well for me. Thanks again !!

[themaharshpatel]

Redmine issue opened: https://redmine.pfsense.org/issues/13811
Thank you @JonathanLee and @michmoor for testing this. Now, let's hope that the PR gets merged soon!

[michmoor]

@themaharshpatel I hope your ticket gets some movement. Its an obvious coding flaw that truthfully...shouldve been picked up years ago but i suppose it shows how often the Squid package gets used by people. Do you know if theres a way to notify folks? Its still sitting there with no acknowledgement.

[themaharshpatel]

@michmoor said in Youtube content getting filtered on Squid:

@themaharshpatel I hope your ticket gets some movement. Its an obvious coding flaw that truthfully...shouldve been picked up years ago but i suppose it shows how often the Squid package gets used by people. Do you know if theres a way to notify folks? Its still sitting there with no acknowledgement.

There might be no acknowledgment due to holidays. And this is the first time I am opening an issue so I don't know about notifying the mods.

[JonathanLee]

@themaharshpatel same issue with 23.05

[michmoor]

@jonathanlee The redmine hasnt been actioned. The Git pull request hasnt been approved from what i can tell.
TBH, considering staff is extremely limited i doubt this will ever get resolved. I would instead see if you could pin this thread at the top.

[JonathanLee]

I have added a new pull