DataTablesSrc icon indicating copy to clipboard operation
DataTablesSrc copied to clipboard

Upgrade to [email protected] in distribution repos

Open jonathanloos opened this issue 3 years ago • 7 comments

yarn audit returning XSS vulnerability (https://snyk.io/vuln/SNYK-JS-DATATABLESNET-1540544) for datatables.net < 1.11.3. I'd like to propose an update to the distribution repositories to:

  1. datatables.net-bs4 to point to [email protected].
  2. All other distribution repos point to updated version of datatables.net-bs4.

If accepted this will be my first contribution to an open source project so please bear with me. If I missed anything here please let me know!

Best, Jon

jonathanloos avatar Oct 04 '21 13:10 jonathanloos

Good point, we've just assumed that the dependency of >= as is at the moment is good enough. That isn't the case for something such as this.

We have scripts which can update all of the distribution repos if you don't fancy changing them all, but if you do, it will be a lot of contributions to get you started in the OSS world :-).

AllanJard avatar Oct 04 '21 13:10 AllanJard

Hey @AllanJard ! Wow that was fast..

Although I do appreciate the exposure value for me of spraying PRs around I don't think the juice is worth the squeeze. If you guys have a tool to automate this by all means go ahead!

-Jon

jonathanloos avatar Oct 04 '21 14:10 jonathanloos

Yup - we'll get it done :)

AllanJard avatar Oct 04 '21 14:10 AllanJard

@AllanJard do you have an estimated timeline for the fix? Don't mean to push, just trying to plan around this blocker.

jonathanloos avatar Oct 04 '21 18:10 jonathanloos

We'll hopefully get the changes made today, but actually tagging and releasing everything might take a little while as this would be the only change.

AllanJard avatar Oct 05 '21 07:10 AllanJard

That's it done (for example).

Was we tag up new releases these will progress through. Until then, an npm install or update should be bringing in the latest versions (unless you have a lock file restricting it to an older version).

AllanJard avatar Oct 05 '21 12:10 AllanJard

Great, thank you!

jonathanloos avatar Oct 05 '21 20:10 jonathanloos