kenhuuu commented on code in PR #2506:
URL: https://github.com/apache/tinkerpop/pull/2506#discussion_r1512135739
##########
gremlin-javascript/src/main/javascript/gremlin-javascript/lib/utils.js:
##########
@@ -112,8 +95,36 @@ exports.getUserAgentHeader = function getUserAgentHeader() {
return 'User-Agent';
};
-const userAgent = generateUserAgent();
+exports.getUserAgent = async () => {
Review Comment:
I see. Although it would have been best for this to work the same between
the web version and the Node version, I still think its worth getting this PR
in so I think this can be overlooked. By the way, in the master branch, we will
be removing WebSocket support and transitioning to HTTP so this won't be an
issue there anyway.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
