commit e8a32fb70c82daf94d9b3c3f7e6ff783b24262c8
author Luca Milanesio <luca.milanesio@gmail.com> Tue Jun 02 09:14:53 2015 +0100
committer Luca Milanesio <luca.milanesio@gmail.com> Thu Jun 25 16:11:18 2015 +0000
tree 256162b62aa2950780bcf1553d8033a7865ff0d9
parent f24cc7b0928b9bfbfa37e9573512e6c8acc7c349

Clean-up scopes names when reading from config

When customising GitHub OAuth Scopes for login, we need to make sure
that scope names are cleaned up from heading/trailing spaces

Change-Id: I36d3ad968ee8f0b896eaeb9862b6c3fc4e1be944

github-oauth/src/main/java/com/googlesource/gerrit/plugins/github/oauth/GitHubOAuthConfig.java

diff --git a/github-oauth/src/main/java/com/googlesource/gerrit/plugins/github/oauth/GitHubOAuthConfig.java b/github-oauth/src/main/java/com/googlesource/gerrit/plugins/github/oauth/GitHubOAuthConfig.java
index 7d30d19..43d778a 100644
--- a/github-oauth/src/main/java/com/googlesource/gerrit/plugins/github/oauth/GitHubOAuthConfig.java
+++ b/github-oauth/src/main/java/com/googlesource/gerrit/plugins/github/oauth/GitHubOAuthConfig.java
@@ -126,7 +126,7 @@
     if (Strings.emptyToNull(scopesString) != null) {
       String[] scopesStrings = scopesString.split(",");
       for (String scope : scopesStrings) {
-        scopes.add(Enum.valueOf(Scope.class, scope));
+        scopes.add(Enum.valueOf(Scope.class, scope.trim()));
       }
     }