Pieter Hoste

Results 393 comments of Pieter Hoste

Official fix seems to have landed here: [ACP2E-4328: Suspicious requests are throwing exceptions in the exception.log ](https://github.com/magento/magento2/commit/a2e91ce36426696e3eb13ddae94fb1025795b867) However, I really like @DmitryFurs's suggestion, would be good if we can investigate...

@engcom-November : can I ask you to review my earlier remark: https://github.com/magento/magento2/issues/40189#issuecomment-3569560714, I feel like there may be a better solution to this issue.

Thanks @djadobe16! I was also following https://github.com/OpenMage/magento-lts/pull/4173 a bit (the community fork of Magento 1), which seems to suggest that the `initStatements` was a hack for very old PHP versions...

Will probably be a limitation caused by the fact that Magento doesn't yet use utf8mb4 encoding in the mysql database. So probably https://github.com/magento/magento2/pull/35174 will fix this...

@ihor-sviziev: just a random remark, but I always learned that GET requests should only fetch data, never perform "actions". Some browsers sometimes pre-fetch GET requests without any user interaction, if...

This reminds me of https://github.com/magento/magento2/issues/37925 which was fixed by removing the reference to `sourceMappingURL` in https://github.com/magento/magento2/commit/11add87193d7ba5fd822631df8e329998f841026 Maybe we should tackle all incorrect references to .map files for javascript files at...

I gave #95 a try and indeed it solves the inconsistent behavior between verbose and non-verbose mode. But it doesn't solve the fact that certain url rewrites are not generated....

What needs to happen to release this fix in a new stable version? Thanks!

I just noticed [a new version was released](https://github.com/less/less.js/releases/tag/v4.4.2) but this fix wasn't included for some reason? Do we have an idea on a timeframe when this fix will get released?...

I think the last two lines should be put in https://github.com/magento/magento2/blob/2.4-develop/dev/tests/acceptance/.gitignore instead of the `.gitignore` file of the entire project. There are a whole bunch of `.gitignore` files in the...