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

Implement starts_with, ends_with operators for Bytes #72

Open
wants to merge 1 commit into
base: cloudflare
Choose a base branch
from

Conversation

amiremohamadi
Copy link

No description provided.

@amiremohamadi
Copy link
Author

Ping @marmeladema. Sorry for bothering, just wanted to know is this repo still under development?
And is there a chance that this PR will be merged?

@veeshi
Copy link

veeshi commented Aug 7, 2021

Not part of the project but for our project we implemented startswith and endswith as functions instead of operators as it is more flexible approach.

@RReverser
Copy link
Contributor

Yeah this kind of use-cases is exactly what functions are for, as it doesn't require any special syntax.

@amiremohamadi
Copy link
Author

Right, but why we have contains as an operator?

@RReverser
Copy link
Contributor

RReverser commented Aug 8, 2021

Because Wireshark has it and this library was designed to support similar syntax: https://www.wireshark.org/docs/wsug_html_chunked/ChWorkBuildDisplayFilterSection.html

If it wasn't for that, it could've been just a function too, yeah.

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.

None yet

3 participants