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.

Sign up for GitHub

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

Jump to bottom

Adding DragonFlyBSD support. #2467

Merged
merged 1 commit into from
Dec 10, 2020
Merged

Adding DragonFlyBSD support. #2467

merged 1 commit into from
Dec 10, 2020

Conversation

devnexen
Copy link
Contributor

@devnexen devnexen commented Dec 4, 2020

No description provided.

@zimmerle zimmerle self-assigned this Dec 7, 2020
@zimmerle zimmerle self-requested a review December 7, 2020 17:46
@zimmerle zimmerle added the 3.x Related to ModSecurity version 3.x label Dec 7, 2020
@zimmerle
Copy link
Contributor

zimmerle commented Dec 7, 2020 •
edited
Loading

Thank you for the patch.

Reference for the reader

@devnexen out of curiosity, is that any specific reason to add DragonFly on whoAmI?

https://github.com/SpiderLabs/ModSecurity/blob/afefda53c69bb17e2ec16d24dd6fbc2c8fd7d063/src/modsecurity.cc#L112-L152

🐲

@devnexen
Copy link
Contributor Author

devnexen commented Dec 7, 2020

Well DragonFlyBSD stands on its own thus deserves to be distinct from FreeBSD.

@zimmerle zimmerle merged commit 560f812 into owasp-modsecurity:v3/master Dec 10, 2020
@zimmerle
Copy link
Contributor

merged! Thank you @devnexen ;)

@zimmerle zimmerle mentioned this pull request Dec 24, 2020
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zimmerle zimmerle Awaiting requested review from zimmerle

Assignees

@zimmerle zimmerle

Labels
3.x Related to ModSecurity version 3.x
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@devnexen @zimmerle