These are chat archives for csarven/ldn

8th
Mar 2017
Xavier Sumba
@cuent
Mar 08 2017 01:19
Which version of node and npm do you use to execute ldn-tests? I'm having troubles to submit the report. In (mayktso) postContainer(req, res, ...) this line var data = req.rawBody; is undefined
Sarven Capadisli
@csarven
Mar 08 2017 09:25
@cuent v7.0.0 of node
v7.5.0 is fine too
npm 4.1.2
npm 3.10.8 is okay too
What do you see in the console?
Xavier Sumba
@cuent
Mar 08 2017 13:42
I get this error :
TypeError: "string" must be a string, Buffer, or ArrayBuffer
    at Function.byteLength (buffer.js:372:11)
    at postContainer (/Users/cuent/Desktop/tests/node_modules/mayktso/index.js:1115:32)
    at handleResource (/Users/cuent/Desktop/tests/node_modules/mayktso/index.js:809:14)
It's because var data = req.rawBody; is undefined. I checked req and it doesn't have rawBody.
My advisor allowed me to test in a public endpoint. I tried here too: http://redi.cedia.org.ec/marmottatest/ldp/inbox
Xavier Sumba
@cuent
Mar 08 2017 13:49
Fails to process notifications if implementation-specific constraints are not met. Inapplicable
Restricts list of notification URIs (eg. according to access control). Cannot tell
Inbox has type ldp:Container. Inapplicable
Rest of tests Passed
Sarven Capadisli
@csarven
Mar 08 2017 18:54
Can I use that endpoint to test?
So far from your paste, there is no particular issue.
Xavier Sumba
@cuent
Mar 08 2017 18:55
Go ahead!
Sarven Capadisli
@csarven
Mar 08 2017 18:56
Looks good! This is conforming LDN :) Congrats!
Feel free to wipe off the inbox.. and we can have 1-2 notifications in there for the report
Xavier Sumba
@cuent
Mar 08 2017 19:29
Awesome! Some idea why my submit was failing? I'm just curious..
Sarven Capadisli
@csarven
Mar 08 2017 19:44
@cuent Would yuou mind removing the existing notifications from that inbox? We can submit a report based on a fresh inbox. Not a big deal either way
Hmm, your issue is odd. mayktso assigns req.rawBody when it is initialised.
Can you doublecheck to see if the body-parser package is installed properly?
It is mentioned in package.json so I guess it should be fine.
you did npm install obviously..
Sarven Capadisli
@csarven
Mar 08 2017 19:57
I've just removed node_modules and did npm install and ran the test again, no issues.
Which node/npm versions do you have?
Xavier Sumba
@cuent
Mar 08 2017 20:00
npm 4.1.2
node v7.7.1
I am going to check again later, thanks. BTW, I removed all notifications in the inbox.
Sarven Capadisli
@csarven
Mar 08 2017 20:00
Should I try to submit a report or would you like ot?
Which URI should be used for the implementation? and also maintainer?
Xavier Sumba
@cuent
Mar 08 2017 20:04
Yes, you can submit. I just want to execute, and try for myself.
implementation: https://github.com/apache/marmotta
Name: Apache Marmotta
Mantainer: Apache Software Foundation/http://marmotta.apache.org/team-list.html
Sarven Capadisli
@csarven
Mar 08 2017 20:18
Thanks
I need to fix something though.
Sarven Capadisli
@csarven
Mar 08 2017 20:32
Fixed.
@cuent Thank you so much for getting this going. Really happy to see that this worked out of the box.
Sarven Capadisli
@csarven
Mar 08 2017 20:37
@cuent , @wikier suggests that there should be an announcement at dev@marmotta.apache.org :)
Xavier Sumba
@cuent
Mar 08 2017 20:45
Awesome, it looks nice. I didn't did so much.