Re: Core 136: Calamaris proxy reports stay empty

[Michael Tremer <michael.tremer@ipfire.org>]

[-- Attachment #1: Type: text/plain, Size: 1967 bytes --]

Matthias, you are not copying the list any more in most of your emails...

> On 12 Nov 2019, at 19:05, Matthias Fischer <matthias.fischer(a)ipfire.org> wrote:
> 
> Hi,
> 
> On 12.11.2019 13:53, Michael Tremer wrote:
>> Hi,
>> 
>>> On 11 Nov 2019, at 17:17, Matthias Fischer <matthias.fischer(a)ipfire.org> wrote:
>>> 
>>> On 11.11.2019 14:14, Michael Tremer wrote:
>>>> ...
>>>> Did you try changing this line as suggested?
>>> 
>>> No, because I'm not that familiar with Perl. How would I do this?
>>> 
>>> Line 2609ff reads:
>>> 
>>> ...
>>> if ( defined(%tcp_content) ) {
>> 
>> I think this line should just be:
>> 
>> if (%tcp_content) {
> 
> Done.
> 
> Tested.
> 
> Worked on both machines.
> 
> Thanks. ;-)

Cool. Please submit a patch which we can release as a quick bug fix.

>> You might have more of those in the file, but I am sure the interpreter will tell you when you try to run it.
>> 
>> As far as I know this whole calamaris thing is not maintained upstream any more. If this isn’t easily fixable we will just have to remove the feature. Not sure how many users this has.
> 
> The last version is 2.99.4.5 (V3-beta) from 2015. I took a quick look
> and there are a *lot* of changes. Hm!
> 
> Demo-Screenshots:
> https://cord.de/files/calamaris/calamaris-3/
> https://cord.de/files/calamaris/calamaris-3-frame/

This is the HTML report. I am not sure if our version could generate those, but text is obviously easier.

> But I don't know if I can handle this - looks complicated - and I'd like
> to make a possible update dependent on the number of seriously
> interested users.

Yeah, I suppose if a drop-in replacement works we can ship it. Otherwise it is not worth the effort. We have so many other things that are more important and more people would benefit from.

> Or I just send a patch with the above change.

Yes, please do.

> 
> Anyone? Opinions?
> 
> Best,
> Matthias