hadoop git commit: HADOOP-11815. HttpServer2 should destroy SignerSecretProvider when it stops. Contributed by Rohith.
Repository: hadoop Updated Branches: refs/heads/branch-2 73d9e4c77 - f753e2043 HADOOP-11815. HttpServer2 should destroy SignerSecretProvider when it stops. Contributed by Rohith. Project: http://git-wip-us.apache.org/repos/asf/hadoop/repo Commit: http://git-wip-us.apache.org/repos/asf/hadoop/commit/f753e204 Tree: http://git-wip-us.apache.org/repos/asf/hadoop/tree/f753e204 Diff: http://git-wip-us.apache.org/repos/asf/hadoop/diff/f753e204 Branch: refs/heads/branch-2 Commit: f753e2043bd1fc7bfa17b40196eed14ffe075cff Parents: 73d9e4c Author: Haohui Mai whe...@apache.org Authored: Thu Apr 9 10:58:12 2015 -0700 Committer: Haohui Mai whe...@apache.org Committed: Thu Apr 9 10:58:20 2015 -0700 -- hadoop-common-project/hadoop-common/CHANGES.txt | 5 - .../src/main/java/org/apache/hadoop/http/HttpServer2.java | 5 - 2 files changed, 8 insertions(+), 2 deletions(-) -- http://git-wip-us.apache.org/repos/asf/hadoop/blob/f753e204/hadoop-common-project/hadoop-common/CHANGES.txt -- diff --git a/hadoop-common-project/hadoop-common/CHANGES.txt b/hadoop-common-project/hadoop-common/CHANGES.txt index 70a1840..b0bbef9 100644 --- a/hadoop-common-project/hadoop-common/CHANGES.txt +++ b/hadoop-common-project/hadoop-common/CHANGES.txt @@ -799,7 +799,10 @@ Release 2.7.0 - UNRELEASED HADOOP-11796. Skip TestShellBasedIdMapping.testStaticMapUpdate on Windows. (Xiaoyu Yao via cnauroth) - + +HADOOP-11815. HttpServer2 should destroy SignerSecretProvider when it +stops. (Rohith via wheat9) + Release 2.6.1 - UNRELEASED INCOMPATIBLE CHANGES http://git-wip-us.apache.org/repos/asf/hadoop/blob/f753e204/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/http/HttpServer2.java -- diff --git a/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/http/HttpServer2.java b/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/http/HttpServer2.java index 39d14d8..11ab23b 100644 --- a/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/http/HttpServer2.java +++ b/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/http/HttpServer2.java @@ -140,6 +140,7 @@ public final class HttpServer2 implements FilterContainer { protected final ListString filterNames = new ArrayList(); static final String STATE_DESCRIPTION_ALIVE = - alive; static final String STATE_DESCRIPTION_NOT_LIVE = - not live; + private final SignerSecretProvider secretProvider; /** * Class to construct instances of HTTP server with specific options. @@ -335,7 +336,7 @@ public final class HttpServer2 implements FilterContainer { this.adminsAcl = b.adminsAcl; this.webAppContext = createWebAppContext(b.name, b.conf, adminsAcl, appDir); try { - SignerSecretProvider secretProvider = + this.secretProvider = constructSecretProvider(b, webAppContext.getServletContext()); this.webAppContext.getServletContext().setAttribute (AuthenticationFilter.SIGNER_SECRET_PROVIDER_ATTRIBUTE, @@ -945,6 +946,8 @@ public final class HttpServer2 implements FilterContainer { } try { + // explicitly destroy the secrete provider + secretProvider.destroy(); // clear stop webAppContext attributes to avoid memory leaks. webAppContext.clearAttributes(); webAppContext.stop();
hadoop git commit: HADOOP-11815. HttpServer2 should destroy SignerSecretProvider when it stops. Contributed by Rohith.
Repository: hadoop Updated Branches: refs/heads/branch-2.7 9a111fcd1 - d665d11f9 HADOOP-11815. HttpServer2 should destroy SignerSecretProvider when it stops. Contributed by Rohith. Project: http://git-wip-us.apache.org/repos/asf/hadoop/repo Commit: http://git-wip-us.apache.org/repos/asf/hadoop/commit/d665d11f Tree: http://git-wip-us.apache.org/repos/asf/hadoop/tree/d665d11f Diff: http://git-wip-us.apache.org/repos/asf/hadoop/diff/d665d11f Branch: refs/heads/branch-2.7 Commit: d665d11f9937383d1d703cad38671c912252f338 Parents: 9a111fc Author: Haohui Mai whe...@apache.org Authored: Thu Apr 9 10:58:12 2015 -0700 Committer: Haohui Mai whe...@apache.org Committed: Thu Apr 9 10:58:31 2015 -0700 -- hadoop-common-project/hadoop-common/CHANGES.txt | 5 - .../src/main/java/org/apache/hadoop/http/HttpServer2.java | 5 - 2 files changed, 8 insertions(+), 2 deletions(-) -- http://git-wip-us.apache.org/repos/asf/hadoop/blob/d665d11f/hadoop-common-project/hadoop-common/CHANGES.txt -- diff --git a/hadoop-common-project/hadoop-common/CHANGES.txt b/hadoop-common-project/hadoop-common/CHANGES.txt index a4605ac..616febc 100644 --- a/hadoop-common-project/hadoop-common/CHANGES.txt +++ b/hadoop-common-project/hadoop-common/CHANGES.txt @@ -729,7 +729,10 @@ Release 2.7.0 - UNRELEASED HADOOP-11796. Skip TestShellBasedIdMapping.testStaticMapUpdate on Windows. (Xiaoyu Yao via cnauroth) - + +HADOOP-11815. HttpServer2 should destroy SignerSecretProvider when it +stops. (Rohith via wheat9) + Release 2.6.1 - UNRELEASED INCOMPATIBLE CHANGES http://git-wip-us.apache.org/repos/asf/hadoop/blob/d665d11f/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/http/HttpServer2.java -- diff --git a/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/http/HttpServer2.java b/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/http/HttpServer2.java index 39d14d8..11ab23b 100644 --- a/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/http/HttpServer2.java +++ b/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/http/HttpServer2.java @@ -140,6 +140,7 @@ public final class HttpServer2 implements FilterContainer { protected final ListString filterNames = new ArrayList(); static final String STATE_DESCRIPTION_ALIVE = - alive; static final String STATE_DESCRIPTION_NOT_LIVE = - not live; + private final SignerSecretProvider secretProvider; /** * Class to construct instances of HTTP server with specific options. @@ -335,7 +336,7 @@ public final class HttpServer2 implements FilterContainer { this.adminsAcl = b.adminsAcl; this.webAppContext = createWebAppContext(b.name, b.conf, adminsAcl, appDir); try { - SignerSecretProvider secretProvider = + this.secretProvider = constructSecretProvider(b, webAppContext.getServletContext()); this.webAppContext.getServletContext().setAttribute (AuthenticationFilter.SIGNER_SECRET_PROVIDER_ATTRIBUTE, @@ -945,6 +946,8 @@ public final class HttpServer2 implements FilterContainer { } try { + // explicitly destroy the secrete provider + secretProvider.destroy(); // clear stop webAppContext attributes to avoid memory leaks. webAppContext.clearAttributes(); webAppContext.stop();
hadoop git commit: HADOOP-11815. HttpServer2 should destroy SignerSecretProvider when it stops. Contributed by Rohith.
Repository: hadoop Updated Branches: refs/heads/trunk 1885141e9 - 63c659ddd HADOOP-11815. HttpServer2 should destroy SignerSecretProvider when it stops. Contributed by Rohith. Project: http://git-wip-us.apache.org/repos/asf/hadoop/repo Commit: http://git-wip-us.apache.org/repos/asf/hadoop/commit/63c659dd Tree: http://git-wip-us.apache.org/repos/asf/hadoop/tree/63c659dd Diff: http://git-wip-us.apache.org/repos/asf/hadoop/diff/63c659dd Branch: refs/heads/trunk Commit: 63c659ddd0641227501851dbcd1352064fa9348a Parents: 1885141 Author: Haohui Mai whe...@apache.org Authored: Thu Apr 9 10:58:12 2015 -0700 Committer: Haohui Mai whe...@apache.org Committed: Thu Apr 9 10:58:12 2015 -0700 -- hadoop-common-project/hadoop-common/CHANGES.txt | 5 - .../src/main/java/org/apache/hadoop/http/HttpServer2.java | 5 - 2 files changed, 8 insertions(+), 2 deletions(-) -- http://git-wip-us.apache.org/repos/asf/hadoop/blob/63c659dd/hadoop-common-project/hadoop-common/CHANGES.txt -- diff --git a/hadoop-common-project/hadoop-common/CHANGES.txt b/hadoop-common-project/hadoop-common/CHANGES.txt index 50fb4d7..397161d 100644 --- a/hadoop-common-project/hadoop-common/CHANGES.txt +++ b/hadoop-common-project/hadoop-common/CHANGES.txt @@ -1222,7 +1222,10 @@ Release 2.7.0 - UNRELEASED HADOOP-11796. Skip TestShellBasedIdMapping.testStaticMapUpdate on Windows. (Xiaoyu Yao via cnauroth) - + +HADOOP-11815. HttpServer2 should destroy SignerSecretProvider when it +stops. (Rohith via wheat9) + Release 2.6.1 - UNRELEASED INCOMPATIBLE CHANGES http://git-wip-us.apache.org/repos/asf/hadoop/blob/63c659dd/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/http/HttpServer2.java -- diff --git a/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/http/HttpServer2.java b/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/http/HttpServer2.java index 0f1c222..6fd34d5 100644 --- a/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/http/HttpServer2.java +++ b/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/http/HttpServer2.java @@ -140,6 +140,7 @@ public final class HttpServer2 implements FilterContainer { protected final ListString filterNames = new ArrayList(); static final String STATE_DESCRIPTION_ALIVE = - alive; static final String STATE_DESCRIPTION_NOT_LIVE = - not live; + private final SignerSecretProvider secretProvider; /** * Class to construct instances of HTTP server with specific options. @@ -335,7 +336,7 @@ public final class HttpServer2 implements FilterContainer { this.adminsAcl = b.adminsAcl; this.webAppContext = createWebAppContext(b.name, b.conf, adminsAcl, appDir); try { - SignerSecretProvider secretProvider = + this.secretProvider = constructSecretProvider(b, webAppContext.getServletContext()); this.webAppContext.getServletContext().setAttribute (AuthenticationFilter.SIGNER_SECRET_PROVIDER_ATTRIBUTE, @@ -913,6 +914,8 @@ public final class HttpServer2 implements FilterContainer { } try { + // explicitly destroy the secrete provider + secretProvider.destroy(); // clear stop webAppContext attributes to avoid memory leaks. webAppContext.clearAttributes(); webAppContext.stop();