Jelmerro / Vieb

Vim Inspired Electron Browser - Vim bindings for the web by design
https://vieb.dev
GNU General Public License v3.0
1.32k stars 63 forks source link

Vieb 12.0.0 adblocker prevents some sites from loading #563

Open HashData opened 4 months ago

HashData commented 4 months ago

Some websites such as amazon.com and youtube.com don't load on Vieb 12.0.0 but they do on 11.0.0. This is the error message:

[522542:0521/161254.205774:ERROR:atom_cache.cc(229)] Add _NET_WM_STATE_MODAL to kAtomsToCache MESA-INTEL: warning: Haswell Vulkan support is incomplete libva error: /usr/lib/x86_64-linux-gnu/dri/iHD_drv_video.so init failed Error occurred in handler for 'GUEST_VIEW_MANAGER_CALL': Error: Script failed to execute, this normally means an error was thrown. Check the renderer console for the error. at node:electron/js2c/renderer_init:2:16470 at IpcRendererInternal. (node:electron/js2c/renderer_init:2:10723) at IpcRendererInternal.emit (node:events:518:28) at Object.onMessage (node:electron/js2c/renderer_init:2:8837) Error occurred in handler for 'run-isolated-js-head-check': Error: Script failed to execute, this normally means an error was thrown. Check the renderer console for the error. at node:electron/js2c/renderer_init:2:16470 at IpcRendererInternal. (node:electron/js2c/renderer_init:2:10723) at IpcRendererInternal.emit (node:events:518:28) at Object.onMessage (node:electron/js2c/renderer_init:2:8837)

Specs

Jelmerro commented 4 months ago

This seems to be not our issue, as this is solved when turning off the adblocker. I can reproduce your stacktrace for different sites. Seems like the adblocker module broke something very badly by updating the resources file recently. For now you can temporarily disable the adblocker before visiting these sites. Please if you have to time try to reproduce the issue with this project and then report the issue in that repo.

se7en-x230 commented 3 months ago

Same problem now for FB. Disabling adblock works

jkarstens commented 1 month ago

Disabling adblocker does allow YouTube (et al) to load, but I think the issue might be in this repo. In the devtools console on YouTube with adblocker enabled I see

Uncaught TypeError: Cannot read properties of null (reading 'innerText')

This console error goes away when adblocker is off (and the site loads normally)

Looks like this may be because document.head is null with the adblocker enabled which causes this line to error? https://github.com/Jelmerro/Vieb/blob/419c43dbf84d7d9bb08ce68b958e66b6182c56c0/app/renderer/tabs.js#L589

YouTube with adblocker: image (23) image (24)

YouTube without adblocker:

image (25) image (26)

purgat0ri commented 1 month ago

^ That would explain why one is still able to view embedded YouTube videos (e.g., https://www.reddit.com/r/planetemacs/comments/1eqqkhf/emacs_way_of_reading_novels_and_books_via_epub/) with adblockers enabled, even though the YouTube site does not load at all.