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

PostgreSQL: initial support #5721

Closed
wants to merge 1 commit into from

Conversation

jufajardini
Copy link
Contributor

  • pgsql/parser: add startup message structure, initial nom parser and corresponding unit tests
  • pgsql/parser: add a parser for the the 'user' startup parameter and corresponding unit tests
  • lib: add pgsql to the module's list

Make sure these boxes are signed before submitting your Pull Request -- thank you.

Link to redmine ticket:

Link to previous PR: #5706

Describe changes:

  • pgsql/parser: add a parser for the the 'user' startup parameter and corresponding unit tests
  • pgsql/parser: add constants for dummy protocol versions used for some message types; and for data type codes
  • StartupMessage parser: add unit test for invalid length
  • nit: replace 2020 with 2021 in the Copyright statement

#suricata-verify-pr:
#suricata-verify-repo:
#suricata-verify-branch:
#suricata-update-pr:
#suricata-update-repo:
#suricata-update-branch:
#libhtp-pr:
#libhtp-repo:
#libhtp-branch:

- pgsql/parser: add startup message structure, initial nom parser and
corresponding unit tests
- pgsql/parser: add a parser for the the 'user' startup parameter and
corresponding unit tests
- lib: add pgsql to the module's list
@jufajardini
Copy link
Contributor Author

Closed with: #5747

@jufajardini jufajardini deleted the nom-parsers-tests-v0 branch September 30, 2021 09:09
jasonish added a commit to jasonish/suricata that referenced this pull request Nov 29, 2022
Convert the logged name of settings to those found in the RFC, for
example, SETTINGS_ENABLE_PUSH instead of SETTINGSENABLEPUSH.

Also converts the settings enum to idiomatic Rust naming.

Issue: OISF#5721
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant