Bug: "bower-asset/punycode": "^2.2" dependency is not valid

[olegbaturin]

Since 2.0 "mathiasbynens/punycode.js" is a commonJS lib and can't be used in the browser.

module.exports = punycode;

#20009

To reproduce an error configure the email validator for the form field like this

['email', 'email', 'enableIDN' => true],

and try to open the page with the form

[mtangoo]

Does it go away if 'enableIDN' => false?

[samdark]

@mtangoo yes, likely as in this case the lib isn't used.

[samdark]

I wonder how did it work for a year+...

[samdark]

We can force using older version.

[mtangoo]

We can force using older version.

I will try to check and fingure out how it did and may be get some solution

[mtangoo]

I have found this, which no longer appears on README.
mathiasbynens/punycode.js@28de93e

This project provides a CommonJS module that uses ES6 features and an ES6 module, which work in modern Node versions and browsers. For the old version that offers the same functionality in a UMD build with support for older pre-ES6 runtimes, including Rhino, Ringo, and Narwhal, see v1.4.1

Am not sure why it is no longer there.

[mtangoo]

After checking releases page I found this:

v2.0.0
This release drops support for old environments. Punycode.js v2.0.0 targets Node.js v6+ and nothing else. For the old version that offers the same functionality with broader support, including Rhino, Ringo, Narwhal, and web browsers, see v1.4.1.

[mtangoo]

Looks like we have to pin the version to exactly 1.4.x

[mtangoo]

@olegbaturin can you test #20284 if the issue is fixed?

[olegbaturin]

@olegbaturin can you test #20284 if the issue is fixed?

with 1.4.1 it works fine