Problem with res.send of a MongoDB-ObjectID

[mm765]

Hi,
i am having a problem with res.send() in decaf/modules/http/lib/Response.js

So far, everything worked, while ive been requesting files and some very simple json-data (like a login request returning if the user/password combination is valid).

Now i tried requesting/returning some data from the database (simple stuff - id (Objectid), username and language-code)
and i get an internal server error.
After about 2-3 hours of searching, i found out that the problem is the ObjectId - as soon as it is in the data that is being sent, the error happens, if i dont send it, no error.

the stacktrace:
Stack Trace
InternalError: Java class "[Ljava.lang.reflect.Constructor;" has no public instance field or method named "toJSON". (/usr/local/decaf/modules/http/lib/Response.js#185)
at /usr/local/decaf/modules/http/lib/Response.js:178 (anonymous)
at /opt/itk/modules/UserHandler.js:38 (anonymous)
at /opt/itk/modules/UserHandler.js:12 (anonymous)
at /opt/itk/libs/decaf-jolt/lib/Application.js:233 (anonymous)
at /opt/itk/libs/decaf-jolt/lib/Application.js:170 (anonymous)
at /usr/local/decaf/modules/http/lib/Child.js:69 (handleRequest)
at /usr/local/decaf/modules/http/lib/Child.js:102 (Child)
at /usr/local/decaf/modules/Threads/lib/Thread.js:197 (anonymous)

interestingly it shows the error happening in line 178 whereas, when i use the debugger, the error happens in line 182 of Response.js (which makes sense because i'm sending json-data, not text).
.
Please advise/help.

[mschwartz]

The ObjectId is a Java object. Convert it to hex with id.toHex().

On Tuesday, February 2, 2016, mm765 notifications@github.com wrote:

Hi,
i am having a problem with res.send() in decaf/modules/http/lib/Response.js

So far, everything worked, while ive been requesting files and some very
simple json-data (like a login request returning if the user/password
combination is valid).

Now i tried requesting/returning some data from the database (simple stuff

• id (Objectid), username and language-code)
  and i get an internal server error.
  After about 2-3 hours of searching, i found out that the problem is the
  ObjectId - as soon as it is in the data that is being sent, the error
  happens, if i dont send it, no error.

the stacktrace:
Stack Trace
InternalError: Java class "[Ljava.lang.reflect.Constructor;" has no public
instance field or method named "toJSON".
(/usr/local/decaf/modules/http/lib/Response.js#185)
at /usr/local/decaf/modules/http/lib/Response.js:178 (anonymous)
at /opt/itk/modules/UserHandler.js:38 (anonymous)
at /opt/itk/modules/UserHandler.js:12 (anonymous)
at /opt/itk/libs/decaf-jolt/lib/Application.js:233 (anonymous)
at /opt/itk/libs/decaf-jolt/lib/Application.js:170 (anonymous)
at /usr/local/decaf/modules/http/lib/Child.js:69 (handleRequest)
at /usr/local/decaf/modules/http/lib/Child.js:102 (Child)
at /usr/local/decaf/modules/Threads/lib/Thread.js:197 (anonymous)

interestingly it shows the error happening in line 178 whereas, when i use
the debugger, the error happens in line 182 of Response.js (which makes
sense because i'm sending json-data, not text).
.
Please advise/help.

—
Reply to this email directly or view it on GitHub
#16.

[mm765]

i tried It.toHexString() and i get the same error as above :( will try toString() and see what happens

[mm765]

Same thing with toString()

[mschwartz]

http://api.mongodb.org/java/current/org/bson/types/ObjectId.html

https://github.com/decafjs/decaf-mongodb/blob/master/lib/ObjectId.js#L20
Java ObjectId is returned.

On Tuesday, February 2, 2016, mm765 notifications@github.com wrote:

Same thing with toString()

—
Reply to this email directly or view it on GitHub
#16 (comment).

[mm765]

Hmm.. maybe it returns a java string and that has to be converted into a javascript string ?
One more thin i noticed when i tried the console.dir() is that i have an array in the data, consisting of objectids - and those get converted to strings - isnt that weird ?

[mm765]

This is the output of the dir() and the roles array consists of objectids in the database

(object)
text Sessionuser:
id
[date]
[getClass]
[wait]
[getTime]
[notifyAll]
[compareTo]
[notify]
[timeSecond]
[hashCode]
[getTimestamp]
[getDate]
[toStringMongod]
[class]
[timestamp]
[getCounter]
[toHexString]
[machineIdentifier]
[counter]
[getMachineIdentifier]
[equals]
[toByteArray]
[getTimeSecond]
[toString]
[getProcessIdentifier]
[time]
[processIdentifier]
name Admin
roles
0 5697d386b0210f3ea1417137
lang en

[mschwartz]

Heh... console.dir() is really smart about Java objects. console.log() just
loops through the arguments calling System.out.print() on each - suitable
for strings.

var s = String(javaString)

On Tuesday, February 2, 2016, mm765 notifications@github.com wrote:

Hmm.. maybe it returns a java string and that has to be converted into a
javascript string ?
One more thin i noticed when i tried the console.dir() is that i have an
array in the data, consisting of objectids - and those get converted to
strings - isnt that weird ?

—
Reply to this email directly or view it on GitHub
#16 (comment).

[mschwartz]

You will need to convert Date() to string or milliseconds to send as json.

On Tuesday, February 2, 2016, mm765 notifications@github.com wrote:

This is the output of the dir() and the roles array consists of objectids
in the database

(object)
text http://string Sessionuser:
id http://JavaObject
[date]
[getClass]
[wait]
[getTime]
[notifyAll]
[compareTo]
[notify]
[timeSecond]
[hashCode]
[getTimestamp]
[getDate]
[toStringMongod]
[class]
[timestamp]
[getCounter]
[toHexString]
[machineIdentifier]
[counter]
[getMachineIdentifier]
[equals]
[toByteArray]
[getTimeSecond]
[toString]
[getProcessIdentifier]
[time]
[processIdentifier]
name http://string Admin
roles http://array
0 http://string 5697d386b0210f3ea1417137
lang http://string en

—
Reply to this email directly or view it on GitHub
#16 (comment).

[mm765]

hmm.. that paste omitted the most important stuff..
for the id it lists all the methods... for the roles:
roles
0 5697d386b0210f3ea1417137
So it clearly sees that one as a string whereas the id is an objectId - in the database both are the same..

[mm765]

let me switch back to the older mongo-driver - maybe its a problem with the new driver i'm using.

[mm765]

well, that wasn't it either..

[mschwartz]

There are some things you cannot serialize as json. Date is one, function
is another. You will have to process the record from mongo into a
serializable object to send it.

Or use only serializable types in your documents. Instead of date, use
milliseconds. Instead of ObjectId, use the string returned by new
ObjectId().toHexString().

On Tuesday, February 2, 2016, mm765 notifications@github.com wrote:

let me switch back to the older mongo-driver - maybe its a problem with
the new driver i'm using.

—
Reply to this email directly or view it on GitHub
#16 (comment).

[mm765]

I just tried the toHexString again - and it returns a javaobject (console.dir shows it as such) - but as a string-object, not an objectid (different functions) - how do i convert that into a javascript-string ?

[mschwartz]

This is a snippet from my working project:

String(document._id.toHexString())

On Tuesday, February 2, 2016, mm765 notifications@github.com wrote:

I just tried the toHexString again - and it returns a javaobject
(console.dir shows it as such) - but as a string-object, not an objectid
(different functions) - how do i convert that into a javascript-string ?

—
Reply to this email directly or view it on GitHub
#16 (comment).

[mschwartz]

http://www.newtonsoft.com/json/help/html/DatesInJSON.htm

On Tuesday, February 2, 2016, Mike Schwartz mykesx@gmail.com wrote:

This is a snippet from my working project:

String(document._id.toHexString())

On Tuesday, February 2, 2016, mm765 <notifications@github.com
javascript:_e(%7B%7D,'cvml','notifications@github.com');> wrote:

I just tried the toHexString again - and it returns a javaobject
(console.dir shows it as such) - but as a string-object, not an objectid
(different functions) - how do i convert that into a javascript-string ?

—
Reply to this email directly or view it on GitHub
#16 (comment).