sipcapture / homer

HOMER - 100% Open-Source SIP, VoIP, RTC Packet Capture & Monitoring
https://sipcapture.org
GNU Affero General Public License v3.0
1.64k stars 242 forks source link

Search filter #488

Closed pengweichu closed 3 years ago

pengweichu commented 3 years ago

The button "OR" "AND" seems not affected, it's always AND here even set it to OR/AND, what's do I missed?

image

adubovikov commented 3 years ago

confirmed

adubovikov commented 3 years ago

fixed here https://github.com/sipcapture/homer-app/commit/e80016705dac51cab142b0ffa3b4a25a3863af13

the package 1.4.11 is compiling

pengweichu commented 3 years ago

fixed here sipcapture/homer-app@e800167

the package 1.4.11 is compiling

Thanks for your fix, below is the feedback. Yesterday I installed the HOMER by Bash installer, today I pull the homer-app, compile and replaced /usr/local/bin/homer-app, I observed the below behaviors:

  1. The new SIP messages send from my server to HOMER can't be quired, even don't set any filter just choose the time range to last 3 hours. But no problem to query the previous SIP messages which were sent yesterday.
  2. I set the AND logic for the filters, works fine, get the 5 SIP messages to display there. Then I switch to OR logic for filters, search again, but the filters not affected, all SIP messages are displayed even the messages don't meet any filter. After then, I click the button to switch the logic to AND, search again, all SIP messages are displayed(I expect only display 5 SIP messages as the first time search), even the messages don't meet any filter. I must refresh the whole page by F5 and search again to get the AND logic to work fine.
  3. The OR logic does not work anyway.

Should I install the HOMER by BASH INSTALLER again to instead compile the Homer-app and replace it manually?

adubovikov commented 3 years ago

not needed open the same issue twice. As I already replied, we couldn't reproduce your issues and it related to your system

https://drive.google.com/file/d/11h5JOoaGxpqvaokjsMNlfVEDzGcrfE8u/view?usp=drivesdk