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

Progetto Maggi: aggiunta a nDPI della feature per estrarre il filename da HTTP attachment file transfers #282

Closed
ChiaraMaggi opened this issue Jun 16, 2023 · 1 comment

Comments

@ChiaraMaggi
Copy link
Contributor

Il progetto consiste nell'estrarre il filename (se presente) dal Content-Disposition presente nel response header di una risposta HTTP regolare. L'aggiunta del nome del file identificherà meglio i trasferimenti sospetti/dannosi.

@lucaderi
Copy link
Owner

Come descritto in ntop/nDPI#1970 occorre fare:

  • utilizzare il pcap https://github.com/ntop/nDPI/blob/dev/tests/cfgs/default/pcap/exe_download.pcap per fare le prove
  • aggiungere in src/include/ndpi_typedefs.h (riga 1445) dentro il campo http un nuovo campo chiamato char *filename;
  • modificare src/lib/protocols/http.c (riga 288) e creare una funzione (o modificare il codice esistente) che processi nella risposta HTTP il campo "Content-Disposition" estraendo il filename se esistente. e lo salvi dentro il campo filename del punto precedente
  • in src/lib/ndpi_main.c (attorno alla ala 5330) liberare la memoria del filename se popolata
  • definire in src/reader_util.h (riga 290) il campo filename
  • modificare example/reader_util.c (attorno alla riga 1392) salvando dentro flow->http.filename il campo popolato in precedenza
  • modificare example/ndpiReader.c (riga 792, 1470 e 1761) aggiungendo la stampa di filename (o stringa vuota se assente)

Per vedere se non sono stati introdotti problemi eseguire tests/do.sh

ChiaraMaggi added a commit to ChiaraMaggi/sgr that referenced this issue Jul 14, 2023
lucaderi pushed a commit that referenced this issue Jul 14, 2023
@lucaderi lucaderi closed this as completed Sep 1, 2023
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

No branches or pull requests

2 participants