Change logger from info to error when geo verification failed
Proposal
When debugging most issues including geo, we look for errors coded as ERROR
in the logs mostly.
When there's a failure occurring that should be addressed but it's tagged as INFO`, it becomes harder to notice it intuitively and at first instance.
Documenting here an example to discuss changing to error before implementing.
- Change this line from
log_info
tolog_error
# Records a checksum mismatch
#
# @param [String] checksum value which does not match the primary checksum
def verification_failed_due_to_mismatch!(checksum, primary_checksum)
message = 'Checksum does not match the primary checksum'
details = { checksum: checksum, primary_checksum: primary_checksum }
log_info(message, details)
This issue is to determine if the initial classification of INFO
as appears in geo.log
is accurate or should be changed.
Edited by Brenda Nyaringita