You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tinkerpop.apache.org by sp...@apache.org on 2016/10/11 19:04:07 UTC

[1/2] tinkerpop git commit: bcrypt: use lowest complexity for performance

Repository: tinkerpop
Updated Branches:
  refs/heads/master 6976fa05d -> 9d7cb947a


bcrypt: use lowest complexity for performance


Project: http://git-wip-us.apache.org/repos/asf/tinkerpop/repo
Commit: http://git-wip-us.apache.org/repos/asf/tinkerpop/commit/636fbc0d
Tree: http://git-wip-us.apache.org/repos/asf/tinkerpop/tree/636fbc0d
Diff: http://git-wip-us.apache.org/repos/asf/tinkerpop/diff/636fbc0d

Branch: refs/heads/master
Commit: 636fbc0d3718ff96d6d02e51cdcba0c1b4f11356
Parents: 00949d4
Author: Robert Dale <ro...@gmail.com>
Authored: Mon Oct 10 08:40:40 2016 -0400
Committer: Robert Dale <ro...@gmail.com>
Committed: Mon Oct 10 08:40:40 2016 -0400

----------------------------------------------------------------------
 CHANGELOG.asciidoc                                                | 1 +
 .../gremlin/groovy/plugin/dsl/credential/CredentialGraph.java     | 3 ++-
 2 files changed, 3 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/636fbc0d/CHANGELOG.asciidoc
----------------------------------------------------------------------
diff --git a/CHANGELOG.asciidoc b/CHANGELOG.asciidoc
index 746d38c..90a7c7a 100644
--- a/CHANGELOG.asciidoc
+++ b/CHANGELOG.asciidoc
@@ -26,6 +26,7 @@ image::https://raw.githubusercontent.com/apache/tinkerpop/master/docs/static/ima
 TinkerPop 3.2.3 (Release Date: NOT OFFICIALLY RELEASED YET)
 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 
+* Increased performance of CredentialGraph authentication
 * Added `HasContainerHolder.removeHasContainer()`-method with default `UnsupportedOperationException` implementation.
 * `TraversalSource.withComputer()` is simplified to add a `VertexProgramStrategy`. Easier for language variants.
 * Fixed a `Set`, `List`, `Map` bug in the various `Translators` where such collections were not being internally translated.

http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/636fbc0d/gremlin-groovy/src/main/java/org/apache/tinkerpop/gremlin/groovy/plugin/dsl/credential/CredentialGraph.java
----------------------------------------------------------------------
diff --git a/gremlin-groovy/src/main/java/org/apache/tinkerpop/gremlin/groovy/plugin/dsl/credential/CredentialGraph.java b/gremlin-groovy/src/main/java/org/apache/tinkerpop/gremlin/groovy/plugin/dsl/credential/CredentialGraph.java
index c2abdbf..8c0277c 100644
--- a/gremlin-groovy/src/main/java/org/apache/tinkerpop/gremlin/groovy/plugin/dsl/credential/CredentialGraph.java
+++ b/gremlin-groovy/src/main/java/org/apache/tinkerpop/gremlin/groovy/plugin/dsl/credential/CredentialGraph.java
@@ -35,6 +35,7 @@ import static org.apache.tinkerpop.gremlin.process.traversal.dsl.graph.__.drop;
  */
 public class CredentialGraph {
 
+    private final int BCRYPT_ROUNDS = 4;
     private final Graph graph;
     private final GraphTraversalSource g;
     private final boolean supportsTransactions;
@@ -70,7 +71,7 @@ public class CredentialGraph {
         try {
             final Vertex v =  graph.addVertex(T.label, CredentialGraphTokens.VERTEX_LABEL_USER,
                                               CredentialGraphTokens.PROPERTY_USERNAME, username,
-                                              CredentialGraphTokens.PROPERTY_PASSWORD, BCrypt.hashpw(password, BCrypt.gensalt()));
+                                              CredentialGraphTokens.PROPERTY_PASSWORD, BCrypt.hashpw(password, BCrypt.gensalt(BCRYPT_ROUNDS)));
             if (supportsTransactions) graph.tx().commit();
             return v;
         } catch (Exception ex) {


[2/2] tinkerpop git commit: Merge branch 'pr-452'

Posted by sp...@apache.org.
Merge branch 'pr-452'


Project: http://git-wip-us.apache.org/repos/asf/tinkerpop/repo
Commit: http://git-wip-us.apache.org/repos/asf/tinkerpop/commit/9d7cb947
Tree: http://git-wip-us.apache.org/repos/asf/tinkerpop/tree/9d7cb947
Diff: http://git-wip-us.apache.org/repos/asf/tinkerpop/diff/9d7cb947

Branch: refs/heads/master
Commit: 9d7cb947a8246677a1accef63d71a92eda542eba
Parents: 6976fa0 636fbc0
Author: Stephen Mallette <sp...@genoprime.com>
Authored: Tue Oct 11 13:20:09 2016 -0400
Committer: Stephen Mallette <sp...@genoprime.com>
Committed: Tue Oct 11 13:20:09 2016 -0400

----------------------------------------------------------------------
 CHANGELOG.asciidoc                                                | 1 +
 .../gremlin/groovy/plugin/dsl/credential/CredentialGraph.java     | 3 ++-
 2 files changed, 3 insertions(+), 1 deletion(-)
----------------------------------------------------------------------