jmrivas86 / django-json-widget

An alternative widget that makes it easy to edit the new Django's field JSONField (PostgreSQL specific model fields)
MIT License
440 stars 88 forks source link

Security vulnerability in the `future` dependency #75

Closed romanek-adam-b2c2 closed 1 year ago

romanek-adam-b2c2 commented 1 year ago

Description

This library depends on the future package which is not maintained, but contains a high-severity vulnerability: https://github.com/advisories/GHSA-v3c5-jqr6-7qm8

Please drop the dependency on this package if possible.

whardeman commented 1 year ago

Agreed. Makes these others all the more relevant...

https://github.com/jmrivas86/django-json-widget/pull/65 https://github.com/jmrivas86/django-json-widget/issues/72

Otherwise, I'll be forced to fork this repo until there are more folks who can keep it up to date.

ashokdelphia commented 1 year ago

I agree that stopping using future is worthwhile (see #65), but I don't think that vulnerability is reachable in practice from how it is used here. The troublesome regular expression is in the cookie-handling part of the library, which I believe isn't invoked at all here.

I think it's worth dropping as part of no longer supporting very old versions of Django and Python, as that will typically involve using libraries that are practically unmaintained.

whardeman commented 1 year ago

@ashokdelphia Thank you for your insight on this! I hadn't looked deeply until now and I agree, it's not a showstopper here.

Most folks are probably (like me) just seeing security vulnerability warnings and scrambling to keep their dependencies clean. In our case, future was only used by two dependencies (the other having already dropped it in a recent release, so just needed to upgrade).

Still hopeful that @jmrivas86 can add a few more folks to help maintain this. It's a good one!

kennell commented 1 year ago

@jmrivas86 Can we get this merged please?

jmrivas86 commented 1 year ago

Fixed in https://github.com/pifantastic/django-json-widget/pull/1