Bug fix for issue #9 (Investigate failing ip addresses).

callumrollo / erddaplogs

Quick utilities for parsing nginx and apache logs for ERDDAP requests

MIT License

0 stars 4 forks source link

Bug fix for issue #9 (Investigate failing ip addresses). #26

Closed aalloilla closed 1 month ago

aalloilla commented 1 month ago

Will concatenate even the failing responses from ip-api.com to preserve the ip addresses for further statistics. Missing columns will be filled with null, which will break some plotting functions. Further checks will be needed to plotting functions not to operate on null values where applicable.

callumrollo commented 1 month ago

Nice elegant solution! I'm going to merge PR #27 first so we have some tests in place before merging in this one so we can see what breaks downstream