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

[master] Add find_all to eregmatch() nasl function. Backport #875 #876

Merged
merged 3 commits into from
Sep 15, 2021

Conversation

jjnicola
Copy link
Member

@jjnicola jjnicola commented Sep 14, 2021

Backport #875

What:
Add find_all to eregmatch() nasl function

Why:
The current implementation only returns the first match in the string.
Now it will return all matches if the option find_all: TRUE is passed to the function.
For backward compatibility, if FALSE is passed or no option, it will return the first match as before.

How:
See PR #875

Checklist:

@jjnicola jjnicola requested a review from a team as a code owner September 14, 2021 12:46
The current implementation only returns the first match in the string.
Now it will return all matches if the option `find_all: TRUE` is passed to the function.
For backward compatibility, if FALSE is passed or no option, it will return the first match as before.
@jjnicola jjnicola changed the title Add find_all to eregmatch() nasl function [master] Add find_all to eregmatch() nasl function. Backport #875 Sep 14, 2021
@y0urself y0urself merged commit 7808c13 into greenbone:master Sep 15, 2021
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