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

Update webpack to v5 (or later) #22351

Closed
petemill opened this issue Apr 15, 2022 · 9 comments · Fixed by brave/brave-core#21003
Closed

Update webpack to v5 (or later) #22351

petemill opened this issue Apr 15, 2022 · 9 comments · Fixed by brave/brave-core#21003

Comments

@petemill
Copy link
Member

Webpack 4 is now becoming quite legacy and 5 is pretty mature, offering more configuration and some potential performance benefits.

Actions:

@petemill petemill added web-ui OS/Android Fixes related to Android browser functionality OS/Desktop labels Apr 15, 2022
@petemill petemill self-assigned this Apr 15, 2022
@mihaiplesa
Copy link
Contributor

This is biting us hard now. Anything we can do to upgrade webpack?

nodejs/node#40455
storybookjs/storybook#19692

[2022-11-09T09:29:03.750Z] <s> [webpack.Progress] 10% building 1/4 modules 3 active /Users/jenkins/jenkins/workspace/pr-brave-browser-fix-compiledb-patch-macos/src/brave/node_modules/babel-loader/lib/index.js??ref--4-0!/Users/jenkins/jenkins/workspace/pr-brave-browser-fix-compiledb-patch-macos/src/brave/.storybook/preview.js-generated-config-entry.js

[2022-11-09T09:29:04.067Z] node:internal/crypto/hash:71

[2022-11-09T09:29:04.067Z]   this[kHandle] = new _Hash(algorithm, xofLen);

[2022-11-09T09:29:04.067Z]                   ^

[2022-11-09T09:29:04.067Z] 

[2022-11-09T09:29:04.067Z] Error: error:0308010C:digital envelope routines::unsupported

[2022-11-09T09:29:04.067Z]     at new Hash (node:internal/crypto/hash:71:19)

[2022-11-09T09:29:04.067Z]     at Object.createHash (node:crypto:133:10)

[2022-11-09T09:29:04.067Z]     at module.exports (/Users/jenkins/jenkins/workspace/pr-brave-browser-fix-compiledb-patch-macos/src/brave/node_modules/webpack/lib/util/createHash.js:135:53)

[2022-11-09T09:29:04.067Z]     at NormalModule._initBuildHash (/Users/jenkins/jenkins/workspace/pr-brave-browser-fix-compiledb-patch-macos/src/brave/node_modules/webpack/lib/NormalModule.js:417:16)

[2022-11-09T09:29:04.068Z]     at handleParseError (/Users/jenkins/jenkins/workspace/pr-brave-browser-fix-compiledb-patch-macos/src/brave/node_modules/webpack/lib/NormalModule.js:471:10)

[2022-11-09T09:29:04.068Z]     at /Users/jenkins/jenkins/workspace/pr-brave-browser-fix-compiledb-patch-macos/src/brave/node_modules/webpack/lib/NormalModule.js:503:5

[2022-11-09T09:29:04.068Z]     at /Users/jenkins/jenkins/workspace/pr-brave-browser-fix-compiledb-patch-macos/src/brave/node_modules/webpack/lib/NormalModule.js:358:12

[2022-11-09T09:29:04.068Z]     at /Users/jenkins/jenkins/workspace/pr-brave-browser-fix-compiledb-patch-macos/src/brave/node_modules/loader-runner/lib/LoaderRunner.js:373:3

[2022-11-09T09:29:04.068Z]     at iterateNormalLoaders (/Users/jenkins/jenkins/workspace/pr-brave-browser-fix-compiledb-patch-macos/src/brave/node_modules/loader-runner/lib/LoaderRunner.js:214:10)

[2022-11-09T09:29:04.069Z]     at iterateNormalLoaders (/Users/jenkins/jenkins/workspace/pr-brave-browser-fix-compiledb-patch-macos/src/brave/node_modules/loader-runner/lib/LoaderRunner.js:221:10)

[2022-11-09T09:29:04.069Z]     at /Users/jenkins/jenkins/workspace/pr-brave-browser-fix-compiledb-patch-macos/src/brave/node_modules/loader-runner/lib/LoaderRunner.js:236:3

[2022-11-09T09:29:04.069Z]     at context.callback (/Users/jenkins/jenkins/workspace/pr-brave-browser-fix-compiledb-patch-macos/src/brave/node_modules/loader-runner/lib/LoaderRunner.js:111:13)

[2022-11-09T09:29:04.069Z]     at /Users/jenkins/jenkins/workspace/pr-brave-browser-fix-compiledb-patch-macos/src/brave/node_modules/babel-loader/lib/index.js:59:71 {

[2022-11-09T09:29:04.069Z]   opensslErrorStack: [ 'error:03000086:digital envelope routines::initialization error' ],

[2022-11-09T09:29:04.069Z]   library: 'digital envelope routines',

[2022-11-09T09:29:04.069Z]   reason: 'unsupported',

[2022-11-09T09:29:04.069Z]   code: 'ERR_OSSL_EVP_UNSUPPORTED'

[2022-11-09T09:29:04.069Z] }

[2022-11-09T09:29:04.069Z] 

[2022-11-09T09:29:04.069Z] Node.js v18.12.1

@stephendonner
Copy link

stephendonner commented Dec 7, 2023

Verification PASSED using

Brave | 1.63.24 Chromium: 120.0.6099.71 (Official Build) nightly (64-bit)
-- | --
Revision | 8858c508f9a33d9cb69c2b3f2da81a52a15040f6
OS | Windows 10 Version 22H2 (Build 19045.3758)

WebTorrent

torrent magnet
image image

New-tab page

Sponsored Image regular tab/customize dialog
image image

Brave Rewards

logged out custodian-connected
image image

Logged:

Brave Wallet

default view recovery phrase recovery-phrase matching Portfolio
image image image image

Brave Shields

Shields up, collapsed Shields up, expanded Shields down
image image image

Leo AI chat

onboarding defaults summarize questions settings overlay
image image image image image

Shortcuts

example example example example
image image image image

@kjozwiak
Copy link
Member

Removing the QA Pass label as the above needs to be re-verified/checked on 1.62.x now that it was uplifted via brave/brave-core#21380. Used #22351 (comment) as the verification needed for uplift 👍

@kjozwiak
Copy link
Member

The above requires 1.62.113 or higher for 1.62.x verification 👍

@btlechowski
Copy link

Verified with

Brave 1.62.122 Chromium: 120.0.6099.144 (Official Build) beta (64-bit)
Revision cfddebe77d394064c472fda64afcd9fbed34ceb4
OS Linux

Verified using #22351 (comment)

WebTorrent

torrent magnet
image image

New-tab page

Sponsored Image regular tab/customize dialog
image image

Brave Rewards

logged out custodian-loggedout
image image

Encountered:

Brave Wallet

image image image image image

Brave Shields

Shields up, collapsed Shields up, expanded Shields down
image image image

Leo AI chat

onboarding defaults summarize questions settings overlay
image image image image image

Shortcuts

example example example
image image image

@GeetaSarvadnya GeetaSarvadnya added the QA/In-Progress Indicates that QA is currently in progress for that particular issue label Dec 28, 2023
@LaurenWags
Copy link
Member

LaurenWags commented Dec 28, 2023

Verified with

Brave | 1.62.123 Chromium: 120.0.6099.144 (Official Build) beta (x86_64)
-- | --
Revision | 680382264ff6ae655b3bba25e80567452f38fbee
OS | macOS Version 13.6.3 (Build 22G436)

Verified using test cases from #22351 (comment)

WebTorrent

Note, theme issues are existing and were previously reported with #33524.

torrent magnet
1 2

New-tab page

Sponsored Image regular tab/customize dialog
1 2

Brave Rewards

Encountered #34775.

Custodian not connected Custodian connected Custodian logged out
1 2 3

Brave Wallet

1 2 3 4 5

Brave Shields

Shields up, collapsed Shields up, expanded Shields down
1 2 3

Leo AI chat

initial screen onboarding summarize questions settings overlay
1 2 3 4 5

Shortcuts

example example example example
1 2 3 4

@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Dec 29, 2023

Verification PASSED on Vivo X70 Pro version 12 running Bravemonoarm64.apk_1.62.121

Verified using test cases from #22351 (comment)

WebTorrent

Note: Click on torrent or magnet link via https://webtorrent.io/free-torrents download the .torrent file there is no UI to see the Files, Start torrent and Save torrent files

New-tab page

Sponsored Image regular tab/customize dialog
Screenshot_20231229_113037 Screenshot_20231229_113043

Brave Rewards

Custodian not connected Custodian connected Custodian logged out
Screenshot_20231229_114100 Screenshot_20231229_121236 Screenshot_20231229_122044

Brave Wallet

Example Example Example
Screenshot_20231229_122547 Screenshot_20231229_122559 Screenshot_20231229_122604

Brave Shields

Shields up, collapsed Shields up, expanded Shields down
Screenshot_20231229_122210 Screenshot_20231229_122214 Screenshot_20231229_122218

Leo AI chat

initial screen onboarding summarize questions settings overlay
Screenshot_20231229_122656 Screenshot_20231229_122701 Screenshot_20231229_122713 Screenshot_20231229_122725 Screenshot_20231229_122733

@MadhaviSeelam
Copy link

MadhaviSeelam commented Jan 3, 2024

Verification PASSED using

Brave | 1.62.128 Chromium: 120.0.6099.144 (Official Build) beta (64-bit)
-- | --
Revision | 41f5844c06c7db1017f39b5b6dc9ba81a6b1b9d6
OS | Windows 11 Version 22H2 (Build 22621.2861)

WebTorrent

torrent magnet
image image

New-tab page

Sponsored Image regular tab/customize dialog
image image

Brave Rewards

Custodian not connected Custodian connected Custodian logged out
image image image

Brave Wallet

example example example example
image image image image

Brave Shields

Shields up, collapsed Shields up, expanded Shields down
image image image

Leo AI chat

onboarding defaults summarize questions settings overlay
image image image image image

Shortcuts

example example example example
image image image image

@GeetaSarvadnya GeetaSarvadnya added QA Pass - Android ARM and removed QA/In-Progress Indicates that QA is currently in progress for that particular issue labels Jan 8, 2024
@Uni-verse Uni-verse added the QA/In-Progress Indicates that QA is currently in progress for that particular issue label Jan 19, 2024
@Uni-verse
Copy link
Contributor

Uni-verse commented Jan 19, 2024

Verified on Samsung Galaxy Tab S7 using version:

Brave	1.62.147 Chromium: 120.0.6099.234 (Official Build) (32-bit) 
Revision	3b25c3743150a54485dea24f0ceb1e69d6db51bc
OS	Android 13; Build/TP1A.220624.014; 33; REL

Test Plan: #22351 (comment)

Web Torrent Screenshot 2024-01-19 at 4 21 35 PM
New Tab Page
Example Example
Screenshot 2024-01-19 at 3 01 09 PM Screenshot 2024-01-19 at 3 01 24 PM
Brave Rewards Screenshot 2024-01-19 at 3 30 47 PM
Brave Wallet
Example Example Example
Screenshot 2024-01-19 at 3 40 55 PM Screenshot 2024-01-19 at 3 41 11 PM Screenshot 2024-01-19 at 3 41 21 PM
Brave Shields
Example Example Example
Screenshot 2024-01-19 at 3 42 25 PM Screenshot 2024-01-19 at 3 42 28 PM Screenshot 2024-01-19 at 3 42 31 PM
Leo
Example Example Example Example
Screenshot 2024-01-19 at 3 49 57 PM Screenshot 2024-01-19 at 3 50 08 PM Screenshot 2024-01-19 at 3 50 53 PM Screenshot 2024-01-19 at 3 51 04 PM

@Uni-verse Uni-verse added QA Pass - Android Tab and removed QA/In-Progress Indicates that QA is currently in progress for that particular issue labels Jan 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment