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

Update `rdf-canonize`, node versions, and more.

Open davidlehn opened this issue 1 year ago • 5 comments

davidlehn avatar Feb 07 '24 20:02 davidlehn

Codecov Report

Merging #548 (7357959) into main (5367858) will increase coverage by 0.10%. The diff coverage is 100.00%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #548      +/-   ##
==========================================
+ Coverage   93.63%   93.74%   +0.10%     
==========================================
  Files          24       24              
  Lines        2986     2989       +3     
==========================================
+ Hits         2796     2802       +6     
+ Misses        190      187       -3     
Files Coverage Δ
lib/fromRdf.js 98.58% <100.00%> (+0.07%) :arrow_up:
lib/jsonld.js 85.26% <100.00%> (+0.98%) :arrow_up:
lib/toRdf.js 99.21% <100.00%> (-0.03%) :arrow_down:

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 5367858...7357959. Read the comment docs.

codecov-commenter avatar Feb 07 '24 20:02 codecov-commenter

This pr fixes some security issues. Would be good to update the http library too to the 4.11 version. the http library on the 3.xx branch has a dependency on udincy or something like that wich leaks memory.

Is there any way of helping with this or testing the beta branch with the fixes?

gabboman avatar Jul 22 '24 09:07 gabboman

update: I tried myself installing just the newer version of the httpclient on my project and that caused an infinite loop in some cases. This may need some investigation and just updating dependencies may not just cut it

gabboman avatar Jul 24 '24 07:07 gabboman