-
Notifications
You must be signed in to change notification settings - Fork 3
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
[FR] Replace ciso8601 with dateutil #10
Comments
If we sent data to elastic with a single wrong date it'll be discarded so we need to parse and check all the date we use as timeline info. |
Thanks for your quick response! Ok, I see. I'm curios why the timestamp would be wrong in the first place, and also wondering if you really want to parse it then? :) Anyhow, if we can move along with #11 instead, then this issue can be closed. Let me know. |
I don't know exactly why. In this case we remove this item from records, but in any case we need to parse all data value to change date format and generate timestamps. |
@berggren |
Is your feature request related to a problem? Please describe.
One of the dependencies for this project is ciso8601, which is a C library to parse datetime strings to datetime objects. This has the side effect that we need to install build tools when building docker containers.
Describe the solution you'd like
I opt to investigate if this library is essential, or if it could be replaced with something like the dateutil library. WDYT?
Additional context
This affects the Timesketch project and have added an issue when building our docker containers. I have disabled the Mans importer at the moment, but would like to enable it as soon as possible again.
The text was updated successfully, but these errors were encountered: