You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@heron.apache.org by ni...@apache.org on 2022/05/30 01:43:16 UTC

[incubator-heron] 01/01: Removed random long in filename which caused leaking in upload storage

This is an automated email from the ASF dual-hosted git repository.

nicknezis pushed a commit to branch nicknezis/uploader-leak-fix
in repository https://gitbox.apache.org/repos/asf/incubator-heron.git

commit 3a60dc13478110c8ff2b7300a18e33e74931cfd2
Author: Nicholas Nezis <ni...@gmail.com>
AuthorDate: Sun May 29 21:43:12 2022 -0400

    Removed random long in filename which caused leaking in upload storage
---
 heron/spi/src/java/org/apache/heron/spi/utils/UploaderUtils.java | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/heron/spi/src/java/org/apache/heron/spi/utils/UploaderUtils.java b/heron/spi/src/java/org/apache/heron/spi/utils/UploaderUtils.java
index 4365f5a715d..c77d417470b 100644
--- a/heron/spi/src/java/org/apache/heron/spi/utils/UploaderUtils.java
+++ b/heron/spi/src/java/org/apache/heron/spi/utils/UploaderUtils.java
@@ -65,8 +65,8 @@ public final class UploaderUtils {
       String tag,
       int version,
       String extension) {
-    return String.format("%s-%s-%s-%d-%d%s",
-        topologyName, role, tag, version, new Random().nextLong(), extension);
+    return String.format("%s-%s-%s-%d%s",
+        topologyName, role, tag, version, extension);
   }
 
   public static void copyToOutputStream(String inFile,