logstash-output-s3 icon indicating copy to clipboard operation
logstash-output-s3 copied to clipboard

Published 20 hours ago • verified publisher icon logstash-plugins

No need for counters, because temporary file names are already random

Open drscre opened this issue 7 years ago • 0 comments

What is the point of adding a .part#{counter}. to the temporary file name? It is already unique, since it includes #{SecureRandom.uuid}

https://github.com/logstash-plugins/logstash-output-s3/blob/408a814a4b205275ffad63de5b5dfcaf65163817/lib/logstash/outputs/s3/temporary_file_factory.rb#L72

drscre avatar Apr 18 '17 15:04 drscre