Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bugfix Third Party Filtering #433

Open
wants to merge 8 commits into
base: dev
Choose a base branch
from
Open

Conversation

F4FXL
Copy link

@F4FXL F4FXL commented Dec 10, 2022

This addresses issue #428.

  • Filters like t/o would not work on 3rd Party packets since the payload was not inspected.
  • Also budlist filter was applied on the outgoing igate and not on the payload. Budlist has to be apply on the payload as we want to filter the source of the packet. Outgoing igate shall be considered as a digipeater. This si also corrected here.
  • Filters like t/{ still works same as before and will match any 3rd party packet.

@F4FXL F4FXL changed the title Bugfix/thirs party type Bugfix third Party Filtering Dec 10, 2022
@F4FXL F4FXL changed the title Bugfix third Party Filtering Bugfix Third Party Filtering Dec 10, 2022
@wb2osz
Copy link
Owner

wb2osz commented Jan 1, 2023

Thank you for the submission,
I agree that filtering should look inside of third party packets and operate on the actual payload,
The commits don't look right because they reference 17 files, mostly updates I made recently.
I need some time to think about the different cases, add to the automated test cases, and figure out how to revise the documentation,

@F4FXL
Copy link
Author

F4FXL commented Jan 2, 2023

Well I made the fixes in a separate branch and messed up merging your dev branch into it. The relevant commit are all the ones prior to 157d874, everything else is me failing to properly merge your dev branch in to my bugfix branch.

@F4FXL
Copy link
Author

F4FXL commented Jul 2, 2023

Is this commit an attempt to fix this issue ? e84a622

@wb2osz
Copy link
Owner

wb2osz commented Jul 2, 2023

pfilter all ready called decode_aprs at the beginning to parse the packet and extract various properties.
There was a lot of duplication of functionality in these two places, perhaps done inconsistently.
It made sense, to me, for pfilter to rely on the work already done by decode_aprs rather duplicate the functionality.
New self-test cases were added to pfilter to check behavior for packets wrapped in third party headers.
Does this have the same functionality you were looking for?

@F4FXL
Copy link
Author

F4FXL commented Jul 2, 2023

As I am reading your reply it all makes sense and seem to have simplified the code somewhat. It makes sense to only not duplicate same functionality all over the place. I will test this during a couple of days and let you know the what comes out.

@F4FXL
Copy link
Author

F4FXL commented Jul 7, 2023

Ok, I just gave this e84a622 a try. Unfortunately it is not working.
I have following filter to prevent digipeating of objects, items and telemetry
FILTER 0 0 !t/oith

Unfortunately this does not work as soon as the object or items are encapsulated into 3rd party. Even h filter has no effect....

EDIT: Nevermind, my test 3rd party frame was malformed and categorized as something else. It is working. Iw ill conduct some more tests and close the issue asap.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants