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

http2: app-layer event for userinfo in uri backport 6 #9822

Closed
wants to merge 1 commit into from
Closed

http2: app-layer event for userinfo in uri backport 6 #9822

wants to merge 1 commit into from

Conversation

catenacyber
Copy link
Contributor

Link to redmine ticket:
https://redmine.openinfosecfoundation.org/issues/6430

Describe changes:

Needed to expand the event after the clean cherry-pick

@catenacyber
http2: app-layer event for userinfo in uri
eae1726 
Ticket: OISF#6426

as per RFC 9113
":authority" MUST NOT include the deprecated userinfo subcomponent
for "http" or "https" schemed URIs.

(cherry picked from commit e3cd0d0)
@suricata-qa
Copy link

Information: QA ran without warnings.

Pipeline 16592

@victorjulien victorjulien mentioned this pull request Nov 17, 2023
Merged
@victorjulien
Copy link
Member

Merged in #9828, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@victorjulien victorjulien victorjulien approved these changes

@jasonish jasonish Awaiting requested review from jasonish jasonish is a code owner

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
3 participants
@catenacyber @suricata-qa @victorjulien