Can't resolve 'path'

[mkrajcir]

I got an error with Angular 11 project

Error: ./node_modules/sanitize-html/node_modules/postcss/lib/input.js
Module not found: Error: Can't resolve 'path' in '/Users/mkrajcir/Devel/BMB/eshop/node_modules/sanitize-html/node_modules/postcss/lib'

Error: ./node_modules/sanitize-html/node_modules/postcss/lib/map-generator.js
Module not found: Error: Can't resolve 'path' in '/Users/mkrajcir/Devel/BMB/eshop/node_modules/sanitize-html/node_modules/postcss/lib'

Error: ./node_modules/sanitize-html/node_modules/postcss/lib/previous-map.js
Module not found: Error: Can't resolve 'path' in '/Users/mkrajcir/Devel/BMB/eshop/node_modules/sanitize-html/node_modules/postcss/lib'

[Semigradsky]

Also apostrophecms/sanitize-html#431

Looks like sanitize-html used in the browser so node's dependencies not available.

[ai]

Talk to sanitize-html or Angular builder tool. I can’t help in invastigation.

Open an issue again if you will need some explicit changed in postcss.

[solblog]

I have exactly the same problem:
ERROR in ./node_modules/sanitize-html/node_modules/postcss/lib/input.js
Module not found: Error: Can't resolve 'path' in 'node_modules\sanitize-html\node_modules\postcss\lib'

[pento]

@ai: I think this is an issue in the same vein as #971, though it's likely that path would need a polyfill, rather than just being set to false. This is likely to show up more often, since webpack 5 removed support for automatic polyfills.

If you'd prefer not to add polyfills to postcss, it might be worth updating the Browser docs to note that webpack doesn't automatically polyfill node APIs any more. Perhaps noting the specific errors that folks might see would help it show up in Google searches. 🙂

For downstream folks wanting to fix this in their own repo, browserify should work (though I haven't tested it). Alternatively, if you're using webpack installing path-browserify and adding this to your webpack config should fix it:

resolve: {
	fallback: {
		path: require.resolve( 'path-browserify' ),
	},
},

[ai]

Polyfill for path will be too huge. We need to add package.browser.path: false (so even webpack 4 will not insert the polyfill) and prepare code to work without path.

I just do not have time since I work on another open-source project.

Please send PR (you can see how we deal with fs dependency as a reference).

[ai]

The fix by @mattcompiles was released in 8.2.7