add TODO since I forget what needs doing, rm extra

Denver Gingerich created 5 years ago

Change summary

r2s-bwmsgsv2.rb | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)

Detailed changes

r2s-bwmsgsv2.rb

@@ -69,6 +69,7 @@ while true
 		"archived_message-#{ARGV[0]}-#{tai_yyyymmdd}-total"
 	)
 	# TODO: confirm day_msg_count is integer > 0 (otherwise likely an error)
+	#  unsure how to best error here: bail would cause infinite bounce; eml?
 
 	t = Time.now
 	puts "LOG %d.%09d: total msgs for %s-%s now at %s\n" %
@@ -107,12 +108,11 @@ while true
 		puts "TODO - res: '#{res.code}' '#{res.message}': '#{res.body}'"
 	end
 
+	# TODO: add following functionality:
 	# if SGX gives back anything but response.code 200, retry 5 times-ish
 	# if SGX failed to respond after retries, email or other notify
 	#  and back off with even longer retries (can't do anything else really)
 
-	# if got HTTP 200, then:
-
 	exists = redis.exists(
 		"archived_message-#{ARGV[0]}-#{tai_yyyymmdd}-#{day_msg_count}"
 	)