Rahul Gupta

Results 188 comments of Rahul Gupta

Sorry, to be a bother, but I ran the new code you wrote a few minutes ago on my local system, I got this error (scroll to the bottom): ```...

I do not understand why nyc is failing with EEXIST. Will need some help here as I am blind testing on Windows.

If I disable nyc (now replaced with c8), I get just one other error https://github.com/CxRes/node-solid-server/actions/runs/10639483327/job/29497511727#step:12:702 I am not sure if this error is an artifact of running tests in my...

Tests pass. Phew! I would need guidance for writing tests specific to notifications. Also need to check with reviewers if they are happy migrating to Node 20 (which is the...

I added tests as well (coverage, though not 100%, covers all relevant functionality), which means that this PR should have been **ready to merge**. However, one [surface test](https://github.com/nodeSolidServer/node-solid-server/actions/runs/11246338869/job/31268421048?pr=1792#step:16:744), which has...

@melvincarvalho Thanks for the offer! The high level overview are the specifications: https://cxres.github.io/prep/draft-gupta-httpbis-per-resource-events.html and https://solid.github.io/solid-prep/protocol/ Re Switch: There is no switch right now. I don't think one is even needed,...

> This also needs a version bump, consistent with [semver](https://semver.org/) Bumping is the job of the maintainers ;-) !

Yay! Tests are passing again. Thanks for the flag tip @melvincarvalho!

> Another point—when using a PREP client on an HTTP/1.1 server, every GET is keeping the connection alive, so we should probably think about that. This is unfortunately a node...

> I’d suggest we aim for at least one working demonstration on a live URI to make sure OIDC is all functioning as expected. There is nothing in the code...