Skip to content

Commit

Permalink
Insight. missing decode() when parsing ifconfig output leading to une…
Browse files Browse the repository at this point in the history
…xpected results. sqlite seems to store the binary in a varchar element, which looks right, but you can't compare it in your where statements. This doesn't seem to lead to issues when there is already valid content in the timeserie table.
  • Loading branch information
AdSchellevis committed Jul 12, 2019
1 parent e914148 commit 24dc2a8
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/opnsense/scripts/netflow/lib/parse.py
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ def __init__(self):
output_stream.seek(0)
if_index = 1
for line in output_stream.readline().split():
self._if_index["%s" % if_index] = line
self._if_index["%s" % if_index] = line.decode()
if_index += 1

def if_device(self, if_index):
Expand Down

1 comment on commit 24dc2a8

@fichtner
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

back to normal, thanks a lot! ❤️

Please sign in to comment.