rapid7 / recog

Describe the bug
bin/recog_standardize crashes when trying to process a file that does not exist.

Sometimes values are added to the identifier files and then they are removed from the fingerprints later without cleaning up the identifier files. bin/recog_standardize doesn't remove orphaned entries.

If, instead, it were to create missing files then that would make it easy for contributors to ensure the accuracy of files in identifiers/ by simply deleting all of the txt files and running the tool with the -w flag.

Additionally, it would be nice to have a flag that just does this for us for any file that recog_standardize is processing.

To Reproduce

$ rm identifiers/vendor.txt 

$ ruby bin/recog_standardize -w xml/*
Traceback (most recent call last):
	2: from bin/recog_standardize:52:in `<main>'
	1: from bin/recog_standardize:10:in `load_identifiers'
bin/recog_standardize:10:in `readlines': No such file or directory @ rb_sysopen - /Users/something/git/recog/identifiers/vendor.txt (Errno::ENOENT)

Expected behavior
recog_standardize creates any missing files

For the flag, we could just have it return an empty map here when set

recog/bin/recog_standardize

Lines 8 to 14 in 5432512

    
           def load_identifiers(path) 
        
             res = {} 
        
             File.readlines(path).map{|line| line.strip}.each do |ident| 
        
               res[ident] = true 
        
             end 
        
             return res 
        
           end

The check for file existence should likely be just above the File.readlines() and do the same thing.

	def load_identifiers(path)
	res = {}
	File.readlines(path).map{\|line\| line.strip}.each do \|ident\|
	res[ident] = true
	end
	return res
	end

recog_standard crashes when identifier files are missing.