Re: [PR] Add support for HTTP content compression. [tinkerpop]
Cole-Greer commented on PR #2669: URL: https://github.com/apache/tinkerpop/pull/2669#issuecomment-2209673011 LGTM VOTE +1 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@tinkerpop.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[PR] remove traverser serialization [tinkerpop]
vkagamlyk opened a new pull request, #2676: URL: https://github.com/apache/tinkerpop/pull/2676 traverser serialization previously used only with Bytecode, so now it can be removed -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@tinkerpop.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
(tinkerpop) branch valentyn/remove-traverser-serialization updated: ...
This is an automated email from the ASF dual-hosted git repository.
valentyn pushed a commit to branch valentyn/remove-traverser-serialization
in repository https://gitbox.apache.org/repos/asf/tinkerpop.git
The following commit(s) were added to
refs/heads/valentyn/remove-traverser-serialization by this push:
new bb6d8e67e3 ...
bb6d8e67e3 is described below
commit bb6d8e67e3261929de6b79b376223b7a775d1b47
Author: Valentyn Kahamlyk
AuthorDate: Thu Jul 4 16:24:27 2024 -0700
...
---
.../gremlin/server/handler/HttpGremlinEndpointHandler.java| 11 +--
1 file changed, 5 insertions(+), 6 deletions(-)
diff --git
a/gremlin-server/src/main/java/org/apache/tinkerpop/gremlin/server/handler/HttpGremlinEndpointHandler.java
b/gremlin-server/src/main/java/org/apache/tinkerpop/gremlin/server/handler/HttpGremlinEndpointHandler.java
index 6d74940652..91506dd1cb 100644
---
a/gremlin-server/src/main/java/org/apache/tinkerpop/gremlin/server/handler/HttpGremlinEndpointHandler.java
+++
b/gremlin-server/src/main/java/org/apache/tinkerpop/gremlin/server/handler/HttpGremlinEndpointHandler.java
@@ -203,7 +203,7 @@ public class HttpGremlinEndpointHandler extends
SimpleChannelInboundHandler timeoutFuture =
requestCtx.getTimeoutExecutor();
@@ -363,7 +363,7 @@ public class HttpGremlinEndpointHandler extends
SimpleChannelInboundHandler(),
false);
+chunk = makeChunk(context, serializer, new ArrayList<>(),
false);
nettyContext.writeAndFlush(new DefaultHttpContent(chunk));
} catch (Exception ex) {
// Bytebuf is a countable release - if it does not get written
downstream
@@ -420,7 +420,7 @@ public class HttpGremlinEndpointHandler extends
SimpleChannelInboundHandler i instanceof TemporaryException || i instanceof
Failure).findFirst();
}
-private static ByteBuf makeChunk(final Context ctx, final RequestMessageV4
msg,
- final MessageSerializerV4 serializer,
final List aggregate,
- final boolean hasMore) throws Exception {
+private static ByteBuf makeChunk(final Context ctx, final
MessageSerializerV4 serializer,
+ final List aggregate, final
boolean hasMore) throws Exception {
try {
final ChannelHandlerContext nettyContext =
ctx.getChannelHandlerContext();
(tinkerpop) branch valentyn/remove-traverser-serialization updated: ...
This is an automated email from the ASF dual-hosted git repository.
valentyn pushed a commit to branch valentyn/remove-traverser-serialization
in repository https://gitbox.apache.org/repos/asf/tinkerpop.git
The following commit(s) were added to
refs/heads/valentyn/remove-traverser-serialization by this push:
new 8e756ffb0f ...
8e756ffb0f is described below
commit 8e756ffb0f502aeb9db52e488d04b9ae3b216448
Author: Valentyn Kahamlyk
AuthorDate: Thu Jul 4 16:21:28 2024 -0700
...
---
.../org/apache/tinkerpop/gremlin/structure/io/gryo/GryoVersion.java | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
a/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/io/gryo/GryoVersion.java
b/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/io/gryo/GryoVersion.java
index 7022088678..2f17dd7c95 100644
---
a/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/io/gryo/GryoVersion.java
+++
b/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/io/gryo/GryoVersion.java
@@ -318,7 +318,7 @@ public enum GryoVersion {
add(GryoTypeReg.of(ProjectedTraverser.class, 168));
// 123 was Traverser until it was removed for 4.x
- // 122 was Bytecode until it was removed for 4.x
+// 122 was Bytecode until it was removed for 4.x
// 126 was Bytecode.Binding until it was removed 4.x
add(GryoTypeReg.of(P.class, 124, new
GryoSerializersV3.PSerializer()));
(tinkerpop) branch valentyn/remove-traverser-serialization updated: cleanup
This is an automated email from the ASF dual-hosted git repository.
valentyn pushed a commit to branch valentyn/remove-traverser-serialization
in repository https://gitbox.apache.org/repos/asf/tinkerpop.git
The following commit(s) were added to
refs/heads/valentyn/remove-traverser-serialization by this push:
new 3bfaa8ebbb cleanup
3bfaa8ebbb is described below
commit 3bfaa8ebbb0054e7b64781df62d8165818be80fd
Author: Valentyn Kahamlyk
AuthorDate: Thu Jul 4 15:58:45 2024 -0700
cleanup
---
.../structure/io/AbstractTypedCompatibilityTest.java | 18 --
1 file changed, 18 deletions(-)
diff --git
a/gremlin-util/src/test/java/org/apache/tinkerpop/gremlin/structure/io/AbstractTypedCompatibilityTest.java
b/gremlin-util/src/test/java/org/apache/tinkerpop/gremlin/structure/io/AbstractTypedCompatibilityTest.java
index ba6ebec154..c0824a7da6 100644
---
a/gremlin-util/src/test/java/org/apache/tinkerpop/gremlin/structure/io/AbstractTypedCompatibilityTest.java
+++
b/gremlin-util/src/test/java/org/apache/tinkerpop/gremlin/structure/io/AbstractTypedCompatibilityTest.java
@@ -20,7 +20,6 @@ package org.apache.tinkerpop.gremlin.structure.io;
import io.netty.buffer.ByteBuf;
import io.netty.buffer.UnpooledByteBufAllocator;
-import org.apache.tinkerpop.gremlin.process.traversal.GremlinLang;
import org.apache.tinkerpop.gremlin.process.traversal.Operator;
import org.apache.tinkerpop.gremlin.process.traversal.Order;
import org.apache.tinkerpop.gremlin.process.traversal.P;
@@ -29,7 +28,6 @@ import org.apache.tinkerpop.gremlin.process.traversal.Pop;
import org.apache.tinkerpop.gremlin.process.traversal.SackFunctions;
import org.apache.tinkerpop.gremlin.process.traversal.Scope;
import org.apache.tinkerpop.gremlin.process.traversal.TextP;
-import org.apache.tinkerpop.gremlin.process.traversal.Traverser;
import org.apache.tinkerpop.gremlin.process.traversal.util.Metrics;
import org.apache.tinkerpop.gremlin.process.traversal.util.TraversalMetrics;
import org.apache.tinkerpop.gremlin.structure.Column;
@@ -743,22 +741,6 @@ public abstract class AbstractTypedCompatibilityTest
extends AbstractCompatibili
});
}
-@Test
-public void shouldReadWriteTraverser() throws Exception {
-final String resourceName = "traverser";
-
-final Traverser resource = findModelEntryObject(resourceName);
-final Traverser fromStatic = read(readFromResource(resourceName),
Traverser.class);
-final Traverser recycled = read(write(fromStatic, Traverser.class,
resourceName), Traverser.class);
-assertNotSame(fromStatic, recycled);
-assertEquals(resource.bulk(), recycled.bulk());
-assertEquals(resource.bulk(), fromStatic.bulk());
-assertEquals(resource.get(), recycled.get());
-assertEquals(resource.get(), fromStatic.get());
-assertVertex((Vertex) resource.get(), (Vertex) recycled.get());
-assertVertex((Vertex) resource.get(), (Vertex) fromStatic.get());
-}
-
@Test
public void shouldReadWriteUUID() throws Exception {
final String resourceName = "uuid";
(tinkerpop) 01/01: remove traverser serialization
This is an automated email from the ASF dual-hosted git repository.
valentyn pushed a commit to branch valentyn/remove-traverser-serialization
in repository https://gitbox.apache.org/repos/asf/tinkerpop.git
commit 6dbc5fa713890b41cc7bcf635d4c3e94ec9591a8
Author: Valentyn Kahamlyk
AuthorDate: Thu Jul 4 15:40:37 2024 -0700
remove traverser serialization
---
.../gremlin/structure/io/binary/DataType.java | 2 +-
.../io/binary/TypeSerializerRegistry.java | 3 -
.../io/binary/types/TraverserSerializer.java | 51
.../structure/io/graphson/GraphSONModule.java | 5 --
.../io/graphson/TraversalSerializersV2.java| 31 --
.../io/graphson/TraversalSerializersV3.java| 48
.../structure/io/gryo/GryoSerializersV1.java | 15 -
.../structure/io/gryo/GryoSerializersV3.java | 15 -
.../gremlin/structure/io/gryo/GryoVersion.java | 9 +--
.../graphson/GraphSONMapperEmbeddedTypeTest.java | 10
.../GraphSONMapperPartialEmbeddedTypeTest.java | 8 ---
.../gremlin/structure/io/gryo/GryoMapperTest.java | 10
.../driver/remote/DriverRemoteTraversal.java | 1 -
.../server/handler/HttpGremlinEndpointHandler.java | 12
.../gremlin/server/util/TraverserIterator.java | 67 --
.../gremlin/util/ser/AbstractRoundTripTest.java| 5 --
.../ser/binary/TypeSerializerFailureTests.java | 2 -
17 files changed, 4 insertions(+), 290 deletions(-)
diff --git
a/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/io/binary/DataType.java
b/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/io/binary/DataType.java
index f8c844f223..0f7f52788d 100644
---
a/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/io/binary/DataType.java
+++
b/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/io/binary/DataType.java
@@ -57,7 +57,7 @@ public enum DataType {
P(0X1E),
SCOPE(0X1F),
T(0X20),
-TRAVERSER(0X21),
+// was TRAVERSER(0X21),
BIGDECIMAL(0X22),
BIGINTEGER(0X23),
BYTE(0X24),
diff --git
a/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/io/binary/TypeSerializerRegistry.java
b/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/io/binary/TypeSerializerRegistry.java
index 5273f77113..71520bb4fc 100644
---
a/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/io/binary/TypeSerializerRegistry.java
+++
b/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/io/binary/TypeSerializerRegistry.java
@@ -30,7 +30,6 @@ import
org.apache.tinkerpop.gremlin.process.traversal.SackFunctions;
import org.apache.tinkerpop.gremlin.process.traversal.Scope;
import org.apache.tinkerpop.gremlin.process.traversal.TextP;
import org.apache.tinkerpop.gremlin.process.traversal.TraversalStrategy;
-import org.apache.tinkerpop.gremlin.process.traversal.Traverser;
import org.apache.tinkerpop.gremlin.process.traversal.step.util.BulkSet;
import org.apache.tinkerpop.gremlin.process.traversal.step.util.Tree;
import org.apache.tinkerpop.gremlin.process.traversal.util.AndP;
@@ -83,7 +82,6 @@ import
org.apache.tinkerpop.gremlin.structure.io.binary.types.TransformSerialize
import
org.apache.tinkerpop.gremlin.structure.io.binary.types.TraversalExplanationSerializer;
import
org.apache.tinkerpop.gremlin.structure.io.binary.types.TraversalMetricsSerializer;
import
org.apache.tinkerpop.gremlin.structure.io.binary.types.TraversalStrategySerializer;
-import
org.apache.tinkerpop.gremlin.structure.io.binary.types.TraverserSerializer;
import org.apache.tinkerpop.gremlin.structure.io.binary.types.TreeSerializer;
import org.apache.tinkerpop.gremlin.structure.io.binary.types.UUIDSerializer;
import
org.apache.tinkerpop.gremlin.structure.io.binary.types.VertexPropertySerializer;
@@ -172,7 +170,6 @@ public class TypeSerializerRegistry {
new RegistryEntry<>(TextP.class, new PSerializer<>(DataType.TEXTP,
TextP.class)),
new RegistryEntry<>(Scope.class, EnumSerializer.ScopeSerializer),
new RegistryEntry<>(T.class, EnumSerializer.TSerializer),
-new RegistryEntry<>(Traverser.class, new TraverserSerializer()),
new RegistryEntry<>(BigDecimal.class, new BigDecimalSerializer()),
new RegistryEntry<>(BigInteger.class, new BigIntegerSerializer()),
new RegistryEntry<>(Byte.class,
SingleTypeSerializer.ByteSerializer),
diff --git
a/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/io/binary/types/TraverserSerializer.java
b/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/io/binary/types/TraverserSerializer.java
deleted file mode 100644
index 6361fb01e8..00
---
a/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/io/binary/types/TraverserSerializer.java
+++ /dev/null
@@ -1,51 +0,0 @@
-/*
- * Licensed to the Apache Software
(tinkerpop) branch valentyn/remove-traverser-serialization created (now 6dbc5fa713)
This is an automated email from the ASF dual-hosted git repository. valentyn pushed a change to branch valentyn/remove-traverser-serialization in repository https://gitbox.apache.org/repos/asf/tinkerpop.git at 6dbc5fa713 remove traverser serialization This branch includes the following new commits: new 6dbc5fa713 remove traverser serialization The 1 revisions listed above as "new" are entirely new to this repository and will be described in separate emails. The revisions listed as "add" were already present in the repository and have only been added to this reference.
Re: [PR] improvement: optimization of readmap function [tinkerpop]
xiazcy commented on PR #2673: URL: https://github.com/apache/tinkerpop/pull/2673#issuecomment-2209597341 Thank you for the improvement and added test cases, LGTM. VOTE +1 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@tinkerpop.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [PR] improvement: optimization of readmap function [tinkerpop]
vkagamlyk commented on PR #2673: URL: https://github.com/apache/tinkerpop/pull/2673#issuecomment-2209580786 VOTE+1 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@tinkerpop.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
(tinkerpop) 01/01: Merge branch '3.7-dev'
This is an automated email from the ASF dual-hosted git repository. kenhuuu pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/tinkerpop.git commit 08357014093f89aecb2f574c0aec2797a85688d1 Merge: e9f6aac856 0ea5b14db1 Author: Ken Hu <106191785+kenh...@users.noreply.github.com> AuthorDate: Thu Jul 4 14:09:13 2024 -0700 Merge branch '3.7-dev' .../tinkerpop/gremlin/server/GremlinServerAuthzIntegrateTest.java | 2 -- 1 file changed, 2 deletions(-)
(tinkerpop) branch master updated (e9f6aac856 -> 0835701409)
This is an automated email from the ASF dual-hosted git repository. kenhuuu pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/tinkerpop.git from e9f6aac856 Merge branch '3.7-dev' add 0ea5b14db1 Remove unused imports from GremlinServerAuthzIntegrateTest CTR new 0835701409 Merge branch '3.7-dev' The 1 revisions listed above as "new" are entirely new to this repository and will be described in separate emails. The revisions listed as "add" were already present in the repository and have only been added to this reference. Summary of changes: .../tinkerpop/gremlin/server/GremlinServerAuthzIntegrateTest.java | 2 -- 1 file changed, 2 deletions(-)
(tinkerpop) branch 3.7-dev updated: Remove unused imports from GremlinServerAuthzIntegrateTest CTR
This is an automated email from the ASF dual-hosted git repository. kenhuuu pushed a commit to branch 3.7-dev in repository https://gitbox.apache.org/repos/asf/tinkerpop.git The following commit(s) were added to refs/heads/3.7-dev by this push: new 0ea5b14db1 Remove unused imports from GremlinServerAuthzIntegrateTest CTR 0ea5b14db1 is described below commit 0ea5b14db16b9ae27ce8a916d065b6be5a030b0f Author: Ken Hu <106191785+kenh...@users.noreply.github.com> AuthorDate: Thu Jul 4 14:01:48 2024 -0700 Remove unused imports from GremlinServerAuthzIntegrateTest CTR --- .../tinkerpop/gremlin/server/GremlinServerAuthzIntegrateTest.java | 2 -- 1 file changed, 2 deletions(-) diff --git a/gremlin-server/src/test/java/org/apache/tinkerpop/gremlin/server/GremlinServerAuthzIntegrateTest.java b/gremlin-server/src/test/java/org/apache/tinkerpop/gremlin/server/GremlinServerAuthzIntegrateTest.java index 9a1e2972b1..f46138073f 100644 --- a/gremlin-server/src/test/java/org/apache/tinkerpop/gremlin/server/GremlinServerAuthzIntegrateTest.java +++ b/gremlin-server/src/test/java/org/apache/tinkerpop/gremlin/server/GremlinServerAuthzIntegrateTest.java @@ -32,7 +32,6 @@ import org.apache.tinkerpop.gremlin.driver.remote.DriverRemoteConnection; import org.apache.tinkerpop.gremlin.process.traversal.AnonymousTraversalSource; import org.apache.tinkerpop.gremlin.process.traversal.dsl.graph.GraphTraversalSource; import org.apache.tinkerpop.gremlin.process.traversal.dsl.graph.__; -import org.apache.tinkerpop.gremlin.process.traversal.strategy.verification.AbstractWarningVerificationStrategy; import org.apache.tinkerpop.gremlin.server.auth.AllowAllAuthenticator; import org.apache.tinkerpop.gremlin.server.auth.SimpleAuthenticator; import org.apache.tinkerpop.gremlin.server.authz.AllowListAuthorizer; @@ -42,7 +41,6 @@ import org.apache.tinkerpop.gremlin.structure.io.graphson.GraphSONTokens; import org.apache.tinkerpop.gremlin.util.function.Lambda; import org.apache.tinkerpop.shaded.jackson.databind.JsonNode; import org.apache.tinkerpop.shaded.jackson.databind.ObjectMapper; -import org.codehaus.groovy.tools.shell.CommandException; import org.junit.AfterClass; import org.junit.Assert; import org.junit.Before;
(tinkerpop) branch master updated (2b3b743b0d -> e9f6aac856)
This is an automated email from the ASF dual-hosted git repository. kenhuuu pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/tinkerpop.git from 2b3b743b0d Merge branch '3.7-dev' add 2da0921312 TINKERPOP-3081 Fix traversal argument propagation under authenticated traversal settings. add 0d72e23e2e TINKERPOP-3081 Fix traversal argument propagation under authenticated traversal settings. add e641ef2b06 TINKERPOP-3081 fixing conflict. add ba594fdeed TINKERPOP-3081 Updating to use RequestMessage pattern. add db9b1104ef TINKERPOP-3081 Adding test. add d1d2b7f87e TINKERPOP-3081 Adding test. add 3b7c92708b TINKERPOP-3081 convert to seconds add a00b5dd63b TINKERPOP-3081 convert to seconds add 9bf2566be3 TINKERPOP-3081 Resorting to a timer. add e4d2e72e5a Merge pull request #2622 add 190525dc43 Merge branch '3.6-dev' into 3.7-dev new e9f6aac856 Merge branch '3.7-dev' The 1 revisions listed above as "new" are entirely new to this repository and will be described in separate emails. The revisions listed as "add" were already present in the repository and have only been added to this reference. Summary of changes: CHANGELOG.asciidoc | 1 + .../handler/WebSocketAuthorizationHandler.java | 8 +++ .../server/GremlinServerAuthzIntegrateTest.java| 26 ++ 3 files changed, 30 insertions(+), 5 deletions(-)
(tinkerpop) 01/01: Merge branch '3.7-dev'
This is an automated email from the ASF dual-hosted git repository. kenhuuu pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/tinkerpop.git commit e9f6aac85685249433cc183cb5c73772111a0cef Merge: 2b3b743b0d 190525dc43 Author: Ken Hu <106191785+kenh...@users.noreply.github.com> AuthorDate: Thu Jul 4 13:19:48 2024 -0700 Merge branch '3.7-dev' CHANGELOG.asciidoc | 1 + .../handler/WebSocketAuthorizationHandler.java | 8 +++ .../server/GremlinServerAuthzIntegrateTest.java| 26 ++ 3 files changed, 30 insertions(+), 5 deletions(-)
(tinkerpop) 01/01: Merge branch '3.6-dev' into 3.7-dev
This is an automated email from the ASF dual-hosted git repository. kenhuuu pushed a commit to branch 3.7-dev in repository https://gitbox.apache.org/repos/asf/tinkerpop.git commit 190525dc431d87aa77bdb1f4cd7427966a7de735 Merge: dc8f203ea3 e4d2e72e5a Author: Ken Hu <106191785+kenh...@users.noreply.github.com> AuthorDate: Thu Jul 4 13:19:20 2024 -0700 Merge branch '3.6-dev' into 3.7-dev CHANGELOG.asciidoc | 1 + .../handler/WebSocketAuthorizationHandler.java | 8 +++ .../server/GremlinServerAuthzIntegrateTest.java| 26 ++ 3 files changed, 30 insertions(+), 5 deletions(-)
(tinkerpop) branch 3.7-dev updated (dc8f203ea3 -> 190525dc43)
This is an automated email from the ASF dual-hosted git repository. kenhuuu pushed a change to branch 3.7-dev in repository https://gitbox.apache.org/repos/asf/tinkerpop.git from dc8f203ea3 Merge branch 'pr-2666' into 3.7-dev add 2da0921312 TINKERPOP-3081 Fix traversal argument propagation under authenticated traversal settings. add 0d72e23e2e TINKERPOP-3081 Fix traversal argument propagation under authenticated traversal settings. add e641ef2b06 TINKERPOP-3081 fixing conflict. add ba594fdeed TINKERPOP-3081 Updating to use RequestMessage pattern. add db9b1104ef TINKERPOP-3081 Adding test. add d1d2b7f87e TINKERPOP-3081 Adding test. add 3b7c92708b TINKERPOP-3081 convert to seconds add a00b5dd63b TINKERPOP-3081 convert to seconds add 9bf2566be3 TINKERPOP-3081 Resorting to a timer. add e4d2e72e5a Merge pull request #2622 new 190525dc43 Merge branch '3.6-dev' into 3.7-dev The 1 revisions listed above as "new" are entirely new to this repository and will be described in separate emails. The revisions listed as "add" were already present in the repository and have only been added to this reference. Summary of changes: CHANGELOG.asciidoc | 1 + .../handler/WebSocketAuthorizationHandler.java | 8 +++ .../server/GremlinServerAuthzIntegrateTest.java| 26 ++ 3 files changed, 30 insertions(+), 5 deletions(-)
Re: [PR] TINKERPOP-3081 Fix traversal argument propagation under authentication [tinkerpop]
kenhuuu merged PR #2622: URL: https://github.com/apache/tinkerpop/pull/2622 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@tinkerpop.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
(tinkerpop) 01/01: Merge pull request #2622
This is an automated email from the ASF dual-hosted git repository. kenhuuu pushed a commit to branch 3.6-dev in repository https://gitbox.apache.org/repos/asf/tinkerpop.git commit e4d2e72e5acc51ea92132f30d52553a043d0409b Merge: e26f7c00bf 9bf2566be3 Author: kenhuuu <106191785+kenh...@users.noreply.github.com> AuthorDate: Thu Jul 4 13:18:19 2024 -0700 Merge pull request #2622 TINKERPOP-3081 Fix traversal argument propagation under authentication CHANGELOG.asciidoc | 1 + .../handler/WebSocketAuthorizationHandler.java | 8 +++ .../server/GremlinServerAuthzIntegrateTest.java| 26 ++ 3 files changed, 30 insertions(+), 5 deletions(-) diff --cc CHANGELOG.asciidoc index 67051055cd,edc4ab3970..5c8355811e --- a/CHANGELOG.asciidoc +++ b/CHANGELOG.asciidoc @@@ -23,8 -23,8 +23,9 @@@ image::https://raw.githubusercontent.co [[release-3-6-8]] === TinkerPop 3.6.8 (NOT OFFICIALLY RELEASED YET) + * Fixed a bug in GremlinServer not properly propagating arguments when authentication is enabled. * Fixed bug in Java driver where connection pool was not removing dead connections under certain error conditions. +* Raised handshake exceptions for Java driver for `NoHostAvailableException` situations. [[release-3-6-7]] === TinkerPop 3.6.7 (April 8, 2024)
(tinkerpop) branch TINKERPOP-3081 deleted (was 9bf2566be3)
This is an automated email from the ASF dual-hosted git repository. kenhuuu pushed a change to branch TINKERPOP-3081 in repository https://gitbox.apache.org/repos/asf/tinkerpop.git was 9bf2566be3 TINKERPOP-3081 Resorting to a timer. The revisions that were on this branch are still contained in other references; therefore, this change does not discard any commits from the repository.
(tinkerpop) branch 3.6-dev updated (e26f7c00bf -> e4d2e72e5a)
This is an automated email from the ASF dual-hosted git repository. kenhuuu pushed a change to branch 3.6-dev in repository https://gitbox.apache.org/repos/asf/tinkerpop.git from e26f7c00bf Add instructions to update PR template to release documentation CTR. add 2da0921312 TINKERPOP-3081 Fix traversal argument propagation under authenticated traversal settings. add 0d72e23e2e TINKERPOP-3081 Fix traversal argument propagation under authenticated traversal settings. add e641ef2b06 TINKERPOP-3081 fixing conflict. add ba594fdeed TINKERPOP-3081 Updating to use RequestMessage pattern. add db9b1104ef TINKERPOP-3081 Adding test. add d1d2b7f87e TINKERPOP-3081 Adding test. add 3b7c92708b TINKERPOP-3081 convert to seconds add a00b5dd63b TINKERPOP-3081 convert to seconds add 9bf2566be3 TINKERPOP-3081 Resorting to a timer. new e4d2e72e5a Merge pull request #2622 The 1 revisions listed above as "new" are entirely new to this repository and will be described in separate emails. The revisions listed as "add" were already present in the repository and have only been added to this reference. Summary of changes: CHANGELOG.asciidoc | 1 + .../handler/WebSocketAuthorizationHandler.java | 8 +++ .../server/GremlinServerAuthzIntegrateTest.java| 26 ++ 3 files changed, 30 insertions(+), 5 deletions(-)
(tinkerpop) branch http-compression updated (8027edb3c8 -> c89a98f888)
This is an automated email from the ASF dual-hosted git repository. kenhuuu pushed a change to branch http-compression in repository https://gitbox.apache.org/repos/asf/tinkerpop.git discard 8027edb3c8 Add support for HTTP content compression. add c89a98f888 Add support for HTTP content compression. This update added new revisions after undoing existing revisions. That is to say, some revisions that were in the old version of the branch are not in the new version. This situation occurs when a user --force pushes a change and generates a repository containing something like this: * -- * -- B -- O -- O -- O (8027edb3c8) \ N -- N -- N refs/heads/http-compression (c89a98f888) You should already have received notification emails for all of the O revisions, and so the following emails describe only the N revisions from the common base, B. Any revisions marked "omit" are not gone; other references still refer to them. Any revisions marked "discard" are gone forever. No new revisions were added by this update. Summary of changes: .../org/apache/tinkerpop/gremlin/driver/simple/SimpleHttpClient.java | 2 ++ .../tinkerpop/gremlin/server/GremlinServerAuthIntegrateTest.java | 3 ++- 2 files changed, 4 insertions(+), 1 deletion(-)
Re: [PR] TINKERPOP-3081 Fix traversal argument propagation under authentication [tinkerpop]
lyndonbauto commented on PR #2622: URL: https://github.com/apache/tinkerpop/pull/2622#issuecomment-2209416216 @kenhuuu go ahead and merge it, sorry I am caught up in some stuff right now -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@tinkerpop.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[PR] Bump ws from 8.17.0 to 8.18.0 in /gremlin-javascript/src/main/javascript/gremlin-javascript [tinkerpop]
dependabot[bot] opened a new pull request, #2675:
URL: https://github.com/apache/tinkerpop/pull/2675
Bumps [ws](https://github.com/websockets/ws) from 8.17.0 to 8.18.0.
Release notes
Sourced from https://github.com/websockets/ws/releases;>ws's
releases.
8.18.0
Features
Added support for Blob (https://redirect.github.com/websockets/ws/issues/2229;>#2229).
8.17.1
Bug fixes
Fixed a DoS vulnerability (https://redirect.github.com/websockets/ws/issues/2231;>#2231).
A request with a number of headers exceeding
the[server.maxHeadersCount][]
threshold could be used to crash a ws server.
const http = require('http');
const WebSocket = require('ws');
const wss = new WebSocket.Server({ port: 0 }, function () {
const chars =
!#$%'*+-.0123456789abcdefghijklmnopqrstuvwxyz^_`|~.split('');
const headers = {};
let count = 0;
for (let i = 0; i chars.length; i++) {
if (count === 2000) break;
for (let j = 0; j lt; chars.length; j++) {
const key = chars[i] + chars[j];
headers[key] = 'x';
if (++count === 2000) break;
}
}
headers.Connection = 'Upgrade';
headers.Upgrade = 'websocket';
headers['Sec-WebSocket-Key'] = 'dGhlIHNhbXBsZSBub25jZQ==';
headers['Sec-WebSocket-Version'] = '13';
const request = http.request({
headers: headers,
host: '127.0.0.1',
port: wss.address().port
});
request.end();
});
The vulnerability was reported by https://github.com/rrlapointe;>Ryan LaPointe in https://redirect.github.com/websockets/ws/issues/2230;>websockets/ws#2230.
... (truncated)
Commits
https://github.com/websockets/ws/commit/976c53c4065c49ede73bfba824caf5a6e0f290cb;>976c53c
[dist] 8.18.0
https://github.com/websockets/ws/commit/59b9629b78aa66bcf9acce20468004cd3751c136;>59b9629
[feature] Add support for Blob (https://redirect.github.com/websockets/ws/issues/2229;>#2229)
https://github.com/websockets/ws/commit/0d1b5e6c4acad16a6b1a1904426eb266a5ba2f72;>0d1b5e6
[security] Use more descriptive text for 2017 vulnerability link
https://github.com/websockets/ws/commit/15f11a052a231d1f819fffef53bf4b287646d1ca;>15f11a0
[security] Add new DoS vulnerability to SECURITY.md
https://github.com/websockets/ws/commit/3c56601092872f7d7566989f0e379271afd0e4a1;>3c56601
[dist] 8.17.1
https://github.com/websockets/ws/commit/e55e5106f10fcbaac37cfa89759e4cc0d073a52c;>e55e510
[security] Fix crash when the Upgrade header cannot be read (https://redirect.github.com/websockets/ws/issues/2231;>#2231)
https://github.com/websockets/ws/commit/6a00029edd924499f892aed8003cef1fa724cfe5;>6a00029
[test] Increase code coverage
https://github.com/websockets/ws/commit/ddfe4a804d79e7788ab136290e609f91cf68423f;>ddfe4a8
[perf] Reduce the amount of crypto.randomFillSync() calls
See full diff in https://github.com/websockets/ws/compare/8.17.0...8.18.0;>compare
view
[](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores)
Dependabot will resolve any conflicts with this PR as long as you don't
alter it yourself. You can also trigger a rebase manually by commenting
`@dependabot rebase`.
[//]: # (dependabot-automerge-start)
[//]: # (dependabot-automerge-end)
---
Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
- `@dependabot rebase` will rebase this PR
- `@dependabot recreate` will recreate this PR, overwriting any edits that
have been made to it
- `@dependabot merge` will merge this PR after your CI passes on it
- `@dependabot squash and merge` will squash and merge this PR after your CI
passes on it
- `@dependabot cancel merge` will cancel a previously requested merge and
block automerging
- `@dependabot reopen` will reopen this PR if it is closed
- `@dependabot close` will close this PR and stop Dependabot recreating it.
You can achieve the same result by closing it manually
- `@dependabot show ignore conditions` will show all of
the ignore conditions of the specified dependency
- `@dependabot ignore this major version` will close this PR and stop
Dependabot creating any more for this major version (unless you reopen the PR
or upgrade to it yourself)
- `@dependabot ignore this minor version` will close this PR and stop
Dependabot creating any more for this minor version (unless you reopen the PR
or upgrade to it yourself)
- `@dependabot ignore this dependency` will close this PR and stop
Dependabot creating any more for this dependency (unless you reopen the PR or
upgrade to it yourself)
--
This is an automated message from the Apache Git Service.
To respond to the message,
(tinkerpop) branch dependabot/npm_and_yarn/gremlin-javascript/src/main/javascript/gremlin-javascript/3.6-dev/ws-8.18.0 created (now 43c7f9d10b)
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/npm_and_yarn/gremlin-javascript/src/main/javascript/gremlin-javascript/3.6-dev/ws-8.18.0 in repository https://gitbox.apache.org/repos/asf/tinkerpop.git at 43c7f9d10b Bump ws in /gremlin-javascript/src/main/javascript/gremlin-javascript No new revisions were added by this update.
