GET/PATCH for HTML data island in NSS #1715
Conversation
bourgeoa
changed the title
| @@ -128,6 +128,9 @@ async function handler (req, res, next) { | |||
| res.send(data) | |||
| return next() | |||
| } catch (err) { | |||
| if (err.message === '404') { | |||
| return next(error(404, 'HTML do not contain data island')) | |||
There was a problem hiding this comment.
| return next(error(404, 'HTML do not contain data island')) | |
| return next(error(404, 'HTML does not contain any data islands')) |
There was a problem hiding this comment.
The plural in islands seems to allow multiples. Actually my PR only allows one.
Not sure that more than one could be expected. The id="data" cannot be used more than once in an html document.
There was a problem hiding this comment.
There may be multiple islands in a single HTML doc, tho perhaps not (yet) in your environment. My error text is equivalent to saying, "I found no data island in this HTML," among other phrasings.
Another phrasing suggestion might be, HTML does not contain a data island.
|
Thanks very much for looking at this. I am very interested in this work. Was it ever tried? If not, I could perhaps try running the branch, to see what it looks like. |
|
It has been implemented in SolidOS source-pane where you can test it. |
This implements the discovery and creation/modification of HTML data island : a script HTML block containing RDF data.
See #1714