Project

General

Profile

Actions

Optimization #6387

closed

mqtt: move parser registration code to the rust side

Added by Juliana Fajardini Reichow about 1 year ago. Updated 12 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Target version:
Effort:
Difficulty:
Label:
Beginner, C, Outreachy, Rust

Description

Although our MQTT app-layer parser is mostly in rust, some of the parser registration code still resides in app-layer-mqtt.c

It is possible to move that code to Rust, therefore getting rid of the related c files.

This task encompasses:
- moving the existing code from app-layer-mqtt.c to the rust mqtt parser file
- removing now unnecessary app-layer-mqtt.* files
- checking if there's any left over lines of code that must be adjusted.

Actions #1

Updated by sakshi patle about 1 year ago

Hi, I am interested on working on this issue.

Actions #2

Updated by Shivani Bhardwaj about 1 year ago

  • Status changed from New to Assigned
  • Assignee changed from Community Ticket to sakshi patle
Actions #3

Updated by Juliana Fajardini Reichow about 1 year ago

sakshi patle wrote in #note-1:

Hi, I am interested on working on this issue.

I am terribly sorry that I never properly answer you here, Sakshi, somehow, I didn't get the notification of the activity on this ticket. I saw that you claimed it, so I hope this didn't prevent you from starting to work on it!

Actions #4

Updated by Jeff Lucovsky about 1 year ago

  • Description updated (diff)
Actions #5

Updated by Juliana Fajardini Reichow about 1 year ago

  • Assignee changed from sakshi patle to Community Ticket

Hi there, according to our guidelines for stale tickets, I'm unassigning this ticket.

Thanks for your interest in our project, and feel free to reach out in case you have time and want to contribute to Suricata again :) :)

Refer to:
https://forum.suricata.io/t/important-outreachy-contribution-phase-wrap-up-prs-claimed-tickets-and-more
https://docs.suricata.io/en/latest/devguide/codebase/contributing/contribution-process.html#stale-tickets-policy

Actions #6

Updated by Lukas Sismis about 1 year ago

  • Assignee changed from Community Ticket to Lukas Sismis
Actions #7

Updated by Lukas Sismis about 1 year ago

  • Assignee changed from Lukas Sismis to Jeff Lucovsky
Actions #8

Updated by Jeff Lucovsky about 1 year ago

  • Status changed from Assigned to In Review
Actions #9

Updated by Philippe Antoine 12 months ago

  • Status changed from In Review to Closed
Actions

Also available in: Atom PDF