jsonld.js icon indicating copy to clipboard operation
jsonld.js copied to clipboard

Added URL to context resolution error message.

Open brianorwhatever opened this issue 1 year ago • 2 comments

brianorwhatever avatar Oct 02 '23 22:10 brianorwhatever

  • It's unfortunate to duplicate like this, but perhaps helpful. In theory the details and cause should still be looked at to possibly find a lower level issue. I'm not sure how to cleanly indicate this.
  • Would it be better to put the URL at the end? Would make the common text easier to read before the potentially long variable content.
  • The node doc loader outputs something similar for regular invalid URLs. It doesn't handle garbage URLs very well and puts them directly in the error message without good sanitizing. I think that problem will happen before URLs get here.

davidlehn avatar Oct 02 '23 23:10 davidlehn

@davidlehn do you mind merging this PR, and handling the question of URLs at the end in a separate issue? This is really useful diagnostics.

dmitrizagidulin avatar Aug 03 '24 21:08 dmitrizagidulin