Skip to content

Improve 'downloads' and 'virustotals' tables#153

Open
mcantoni wants to merge 2 commits into
DinoTools:masterfrom
mcantoni:improve_virustotals_table
Open

Improve 'downloads' and 'virustotals' tables#153
mcantoni wants to merge 2 commits into
DinoTools:masterfrom
mcantoni:improve_virustotals_table

Conversation

@mcantoni

Copy link
Copy Markdown
Contributor
ISSUE TYPE
  • Feature
DIONAEA VERSION
master
OS / ENVIRONMENT
Ubuntu 16.04.4 LTS
SUMMARY

Improve logging support. Add some info to 'downloads' and 'virustotals' tables:

download_timestamp

virustotal_sha1_hash
virustotal_sha256_hash
virustotal_positives
virustotal_total
sqlite> PRAGMA table_info(downloads);
0|download|INTEGER|0||1
1|connection|INTEGER|0||0
2|download_url|TEXT|0||0
3|download_md5_hash|TEXT|0||0
4|download_timestamp|INTEGER|0||0
sqlite> PRAGMA table_info(virustotals);
0|virustotal|INTEGER|0||1
1|virustotal_md5_hash|TEXT|1||0
2|virustotal_sha1_hash|TEXT|1||0
3|virustotal_sha256_hash|TEXT|1||0
4|virustotal_positives|INTEGER|1||0
5|virustotal_total|INTEGER|1||0
6|virustotal_timestamp|INTEGER|1||0
7|virustotal_permalink|TEXT|1||0

@phibos

phibos commented Mar 14, 2018

Copy link
Copy Markdown
Member

Thanks for your work but I think there should be an ALTER command to migrate existing tables to the new format. What do you think?

@phibos phibos self-assigned this Mar 14, 2018
@phibos phibos added this to the 0.8.0 milestone Mar 14, 2018
@mcantoni

mcantoni commented Apr 6, 2018

Copy link
Copy Markdown
Contributor Author

You are right, a new solution: ab43f15

@phibos phibos modified the milestones: 0.8.0, 0.9.0 Jun 15, 2018
@phibos phibos removed this from the 0.9.0 milestone Oct 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants