okta-signin-widget icon indicating copy to clipboard operation
okta-signin-widget copied to clipboard

Upgrade jQueryCourage to more recent than 1.12.4

Open shk3bq4d opened this issue 4 years ago • 16 comments

:information_source: If you have a question, please post it on the Okta Developer Forum instead. Issues in this repository are reserved for bug reports and feature requests.

I'm submitting a

  • [x] bug report
  • [ ] feature request

Background info

Okta sign-in widget uses an outdated version of jQuery 1.12.4 named as global variable jQueryCourage which was EOL'ed on 2016.05.20 and suffers from CVE-2015-9251

Reference: Vendor - http://jquery.com/Advisory - https://github.com/jquery/jquery/commit/f60729f3903d17917dc351f3ac87794de379b0cc Advisory - https://github.com/jquery/jquery/issues/2432 Advisory - https://snyk.io/vuln/npm:jquery:20150627 CVE: CVE-2015-9251

Expected behavior

Upgrade to version 3.5.0 or later of jQuery

Steps to reproduce

Either:

  • include Okta Sign-In widget
  • evaluate jQueryCourage.fn.jquery it will return 1.12.4

Or see in the source code that the legacy version is still present

  • https://github.com/okta/okta-signin-widget/blob/3.9/packages/@okta/courage-dist/jquery.js
  • https://github.com/okta/okta-signin-widget/blob/4.2.1/packages/@okta/courage-dist/jquery.js

Your environment

  • Okta Sign-In Widget Version: 4.2.1
  • Browser: Chromium
  • OS: Ubuntu 20.04
  • Language: English

shk3bq4d avatar Jul 13 '20 14:07 shk3bq4d

We did patch those security vulnerabilities in this jquery-1.12.4. We're also working toward upgrade to latest jQuery. Stay tune.

haishengwu-okta avatar Aug 28 '20 06:08 haishengwu-okta

Is there a timeline when this dependency will be upgraded? It looks like CVE-2019-11358/CVE-2019-5428 was patched but there are a number of other vulnerabilities affecting this version: https://snyk.io/test/npm/jquery/1.12.4.

kevinwuhoo avatar Oct 06 '20 14:10 kevinwuhoo

@kevinwuhoo - Thanks for the question. Our team confirms that all vulnerabilities in that list have already been patched in our internal fork of jquery 1.12.4.

swiftone avatar Oct 06 '20 16:10 swiftone

Hello, is there any updates on when jQuery would be updated beyond 1.12.4 for this library?

man0a avatar Oct 26 '20 16:10 man0a

@Dewar0019 - No updates. Security patches get high priority (we are not running 1.12.4, but rather a patched version) but otherwise changing that version is not a priority task.

Can you tell us a bit more about what issues the older (base) version is causing you?

swiftone avatar Oct 26 '20 17:10 swiftone

We were getting flags for this issue with our security scanners. I think it is only looking at the version number to raise this flag. How do you determine whether not a version of v1.12.4 jQuery being used is patched (or if several patches were applied)? Should we assume that all versions using v1.12.4 of jquery is patched?

man0a avatar Oct 26 '20 19:10 man0a

@swiftone Since you havent changed the version ASVs still detect this as vulnerable version and will continue flag it and fail anyone doing a PCI-DSS scan. How much effort can it be to change the version in your code?

Cbryndum avatar Nov 30 '20 19:11 Cbryndum

Can you say when you guys patched jQuery version 1.12.4?

jjbags avatar Mar 03 '21 17:03 jjbags

This is showing up in security scanners/pen testing reports as an issue. You list the fix as being on an internal fork last OCTOBER - when will this be released?

stefbauer avatar Mar 12 '21 15:03 stefbauer

Add internal ticket to get the issue prioritized.

Internal Ref: OKTA-377873

shuowu avatar Mar 12 '21 15:03 shuowu

@Dewar0019 - No updates. Security patches get high priority (we are not running 1.12.4, but rather a patched version) but otherwise changing that version is not a priority task.

Can you tell us a bit more about what issues the older (base) version is causing you?

When was 1.12.4 patched?

lee-bennie avatar May 26 '21 14:05 lee-bennie

Hey, what is the timeline on updating the version number/s to indicate that this has been patched?

jcrew99 avatar Jul 19 '21 01:07 jcrew99

please provide an update on this issue.

Waseemrajashaik avatar Feb 10 '22 17:02 Waseemrajashaik

@shuowu - Can we please get an update on this issue? Security scans are picking this up constantly and we have to justify why our IDP is using an outdated javascript library. Since you are running a patched version, it might be helpful to provide a periodical update on what was patched. Thanks!

kochste avatar Apr 30 '23 20:04 kochste

@kochste The most recent major version of SIW (v7) includes jQuery 3.6 (https://github.com/okta/okta-signin-widget/blob/master/packages/%40okta/courage-dist/package.json#L9)

jaredperreault-okta avatar May 01 '23 14:05 jaredperreault-okta

hello, just wanted to check if there is any progress on this. jQuery 1.12.4 has been picked up during a pen test for us as well, we understand it has been patched but will be great to see this updated...

Choo57 avatar Aug 26 '24 08:08 Choo57