replace basename with dirname

[melvincarvalho]

No description provided.

[dmitrizagidulin]

@melvincarvalho - thank you for the PR! Can you tell us a bit more about what it's trying to fix?

[melvincarvalho]

Consider the solid spec, when POSTing to an inbox.

Should something listening on the inbox be notified when a new resource is created?

From what I can tell. The dirname command will notify the parent directory. The basename will notify a string that is not a URI.

[dmitrizagidulin]

👍 , thanks @melvincarvalho! Merging.

[CxRes]

@melvincarvalho could you please explain the if statement in your change? AFAICT parent will never equal req.originalUrl?

var parent = path.dirname(req.originalUrl) + path.sep
if (parent !== req.originalUrl) {
  solidWs.publish(parent)
}

On the other hand you may want to check if you are at server root. Is that the intention?

[jaxoncreed]

NSS will also need a PR to update it's dependency on on this.

[CxRes]

@jaxoncreed May I suggest that you use dependabot on solid repos...