Reverse Squid no longer working on latest development branch

[kiokoman]

@nafeasonto said in Squid no longer working on latest development branch:

/usr/local/etc/squid/squid.conf

i'm unable to reproduce this, everything seems to work fine on my side
the generation of the squid.conf is ok idk maybe some glich on your pfsense

[stephenw10]

It looks like it should be producing a line like: http_port ipaddress:80 accel defaultsite=mysite.com vhost

It should be choosing the IP address from an array of the ip addresses on the interfaces by the interface it's listening on.

I would guess that last change that went in to add IPv6 missed something for the reverse proxy case.

I also note 'vhost' is a deprecated option. Reverse Squid could use some love.

Steve

[stephenw10]

@kiokoman said in Squid no longer working on latest development branch:

i'm unable to reproduce this

In reverse mode?

[kiokoman]

i put some random stuff inside and it was working, i will try again

[stephenw10]

May be something else required to 'tickle' it. Like something in the array contents... or a lack of contents etc.

[kiokoman]

ah ok found it.. yeah it put Array in the config
bug https://redmine.pfsense.org/issues/10367

    foreach (explode(",", $ifaces) as $i => $iface) {
            $real_ifaces[] = squid_get_real_interface_address($iface);
            if ($real_ifaces[$i][0]) {

$real_ifaces[] contain only "Array"

[nafeasonto]

Good job finding it. I love when people listen to me. Thanks for taking the time to read the post.

[viktor_g]

Fix:
https://github.com/pfsense/FreeBSD-ports/pull/805

[stephenw10]

Rapido!

[kiokoman]

nice work as always viktor

[nafeasonto]

@kiokoman you updating the package on Pfsense?

[kiokoman]

nope, i'm just a regular user like you, viktor have done the work, now Jim and Renato will do the update

[stephenw10]

Pretty simple patch, you could just modify /usr/local/pkg/squid_reverse.inc yourself to test it.

https://github.com/pfsense/FreeBSD-ports/commit/7716b1cb15b3d5ad7985118bc4716565a1c7dbfc?diff=unified

Steve

[nafeasonto]

@stephenw10 The modifications in the link you told me worked. Thanks.

Will you be modifying the package for the future?

[stephenw10]

Not me personally but it will almost certainly be pulled in, yes.

Steve

[nafeasonto]

@stephenw10 okay great, thank you.

[nafeasonto]

@stephenw10 For some reason on reboot, it went back to the old settings.

[stephenw10]

It reverted the file? Curious.

You could apply it using the system patches package. You can set that to auto-apply if required.

Steve

[stephenw10]

That change is merged now. It's in the Squid package 0.4.44_15 which you should now see available.

Steve

[nafeasonto]

Awesome dude thank you.