Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Javascript libraries nested in Odoo source code #1834

Closed
ddufresne opened this issue Mar 3, 2021 · 2 comments
Closed

Javascript libraries nested in Odoo source code #1834

ddufresne opened this issue Mar 3, 2021 · 2 comments

Comments

@ddufresne
Copy link
Contributor

Hello, I have read the following discussion about managing external javascript libraries in OCA modules:
#842

The diverse opinions are interesting and illustrate well the problem we are facing with Odoo.

Odoo keeps outdated versions of multiple javascript libraries. Some very important libraries are outdated by more than 6 years.
https://github.com/odoo/odoo/tree/14.0/addons/web/static/lib

We have a client reporting javascript vulnerabilities by a security firm. This firm was mandated with auditing our client's website (a Odoo website/ecommerce).

My questions are:

  1. What do you think is the actual risk introduced by these libraries?
  2. How do you mitigate these vulnerabilities in your Odoo installations?
@pedrobaeza
Copy link
Member

Bundling a library in Odoo doesn't mean the vulnerability can be used, as you should know the use of such library inside Odoo. Simply having an outdated library version and checking the list of "updates" is not enough. Anyway, if you find a real security exploit, you should notify to Odoo in security at odoo dot com.

@ddufresne
Copy link
Contributor Author

@pedrobaeza thanks for your input.

For me, these possible vulnerabilities are similar to a live virtual machine with missing updates.

Also, there is a question of compliance of the system for insurances.
I am not an expert on insurances, but it is important for our client to comply with audit reports and patch the system in order
to insure his company.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants
@pedrobaeza @ddufresne and others