rdf-ext-archive / rdf-body-parser Goto Github PK

View Code? Open in Web Editor NEW

0.0 0.0 4.0 44 KB

RDF body parsing middleware: REPLACED BY https://github.com/rdfjs-base/express-handler

JavaScript 100.00%

rdf-body-parser's Issues

body-parser depdendency is missing

It looks like body-parser is a dependency but it's not included in package.json

req.get is not a function

When using rdf-body-parser with Restify I get an error:

{"code":"InternalError","message":"req.get is not a function"}

I guess this is a express function. Would be great if the code does not require this as it is useful outside express use cases as well.

Code: https://github.com/rdf-ext/rdf-body-parser/blob/master/index.js#L30

Exit more gently on parser errors

Hi,
when I get a parser error as

/home/...my-project.../node_modules/n3/lib/N3Util.js:49
      throw new Error(literal + ' is not a literal');
      ^

Error:  is not a literal
    at Function.N3Util.getLiteralLanguage (/home/...my-project.../node_modules/n3/lib/N3Util.js:49:13)
    at toRdfNode (/home/...my-project.../node_modules/rdf-parser-n3/index.js:60:30)
    at pushTriple (/home/...my-project.../node_modules/rdf-parser-n3/index.js:80:11)
    at Object._callback (/home/...my-project.../node_modules/rdf-parser-n3/index.js:89:7)
    at Object.N3Parser._readPunctuation (/home/...my-project.../node_modules/n3/lib/N3Parser.js:454:12)
    at Object.N3Parser._readDataTypeOrLang (/home/...my-project.../node_modules/n3/lib/N3Parser.js:296:41)
    at /home/...my-project.../node_modules/n3/lib/N3Parser.js:683:35
    at Object.N3Lexer._tokenizeToEnd (/home/...my-project.../node_modules/n3/lib/N3Lexer.js:262:7)
    at Immediate._onImmediate (/home/...my-project.../node_modules/n3/lib/N3Lexer.js:318:38)
    at processImmediate [as _immediateCallback] (timers.js:383:17)

my application exits immediately. It would be great if the body parser would treat this error more gently, like answering the errorneous HTTP request with a 400 response and not killing the application. Cheers!

This is awesome

@bergos this is genius

SendGraph should have status code

look at send.

This can do .send(404), .send(404, "Oh noes") or just .send("oh yeah")

Unknown content types should be ignored

There should be a check if there is a parser available for the content-type of the request. Here should be the check.

rdf-ext-archive / rdf-body-parser Goto Github PK

rdf-body-parser's People

Contributors

Watchers

Forkers

rdf-body-parser's Issues

body-parser depdendency is missing

req.get is not a function

Exit more gently on parser errors

This is awesome

SendGraph should have status code

Unknown content types should be ignored

Recommend Projects

React

Vue.js

Typescript

TensorFlow

Django

Laravel

D3

Recommend Topics

javascript

web

server

Machine learning

Visualization

Game

Recommend Org

Facebook

Microsoft

Google

Alibaba

D3

Tencent