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

fix #5743 use of perl regexp vs sql like #5747

Merged
merged 1 commit into from May 12, 2024
Merged

Conversation

gvde
Copy link
Contributor

@gvde gvde commented May 10, 2024

Fixes order of REGEXP parameters and avoids devices with empty sysDescr, sysName or sysOid to match device rules with set, non-empty matches.

@netniV netniV merged commit b577c29 into Cacti:1.2.x May 12, 2024
@gvde
Copy link
Contributor Author

gvde commented May 14, 2024

@TheWitness @netniV How do you handle this with the develop branch? Do you want me to create a pull request for the develop branch, too?

@netniV
Copy link
Member

netniV commented May 14, 2024

Yes a separate pull request would be best.

@TheWitness
Copy link
Member

I will do a bunch of cherry picking once the code freeze is lifted @gvde

@gvde
Copy link
Contributor Author

gvde commented May 14, 2024

@TheWitness I have already created #5749...

@TheWitness
Copy link
Member

Yea, I saw it.

TheWitness pushed a commit that referenced this pull request May 18, 2024
@gvde gvde deleted the 1.2.x-patch-5743-2 branch May 21, 2024 05:58
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