Events are uploaded twice using timesketch_import_client with csv or jsonl files.

[S-Nicholas]

Describe the bug
Events are uploaded twice using timesketch_import_client with csv or jsonl files.

To Reproduce
Steps to reproduce the behavior:

1. Generate csv or json file
2. Use timesketch_importer.py to import the file
3. Events are imported twice in timesketch

Expected behavior
Events should be imported only once.

Additional context
The issue is located in the ImportStreamer class.
Private property _data_lines is not empty after calling the flush method.
When the ImportStreamer object is destroyed, flush method is called again.
Fix: ImportStreamer._upload_data_buffer method should call _reset method in case of successfull data upload.

[jaegeral]

This def needs to be reproduced.

Hope we can fix it in Q3.

[S-Nicholas]

Same issue as #2796

[berggren]

I cannot seem to reproduce this. I'm importing a CSV with 48300 events with the importer client and I get the expected result.

@S-Nicholas Do you have a stable reproduce?

[jaegeral]

I am closing this in favour to #2796