mmisw / orr-portal

ORR Frontend component
Apache License 2.0
8 stars 5 forks source link

[Snyk] Fix for 10 vulnerable dependencies #139

Closed snyk-bot closed 5 years ago

snyk-bot commented 5 years ago

Description

This PR fixes one or more vulnerable packages in the npm dependencies of this project. See the Snyk test report for more details.

Snyk Project: mmisw/orr-portal:package.json

Snyk Organization: mmisw

Changes included in this PR

Vulnerabilities that will be fixed

With an upgrade:

You can read more about Snyk's upgrade and patch logic in Snyk's documentation.

Check the changes in this PR to ensure they won't cause issues with your project.

Stay secure, The Snyk team

Note: You are seeing this because you or someone else with access to this repository has authorised Snyk to open Fix PRs. To review the settings for this Snyk project please go to the project settings page.

lewismc commented 5 years ago

@carueda how did you determine whether there was any deprecation last time around?

carueda commented 5 years ago

@lewismc I'm not remembering with all precision, but I think I first saw a notification from Snyk, which I enabled a couple of months ago, and then from Github itself.

BTW, I'll give this PR a try and report here.

carueda commented 5 years ago

I'm testing this branch locally, and against mmisw's ORR to avoid complication with the recent issues noted with the COR instance.

Some testing looking good so far, except that the angular-ui-grid used for displaying ontology contents is not showing at all! (however, the main ontology table, which also uses such component, is working just fine, in fact, #125 would be fixed).

Also, I'm noting these "transition superseded" errors while navigating routes:

image

in this case, upgrading the old version of angular-ui-router from the old 0.2.18 to 1.0.22 (the latest at this point) seems to get rid of those errors. I'll push this upgrade as well while we are able to test more and in particular to determine how to fix the missing triple table for ontology contents.

carueda commented 5 years ago

@lewismc would be good if you could also test this branch locally, to agree on what is and what's not working .

carueda commented 5 years ago

Investigating the missing table issue, looks like this is what's basically happening: https://stackoverflow.com/questions/42426006/angularjs-upgrade-1-5-to-1-6-1-7-makes-directive-scope-bindings-undefined

I noted the strange undefined bindings while tracing the dispatch of ontology contents for CF:

image

Those log lines I added locally are in rj-viewer.js:

image

Note: basically two "misbehaviors" caused by breaking changes in angular are exposed here:

So, the "obvious" solution is simply to properly migrate from angular 1.4 to 1.6, which as far as I can tell now, will require some significant effort.