[BUG] Possible regression in newest version: Error checking websites "'Response' object has no attribute 'status_code' #2122
Replies: 39 comments 32 replies
-
|
Beta Was this translation helpful? Give feedback.
-
let me know if you require more full log. IMO - i dont see anything else pertinent to this in the logs changedetection | 2024-01-19 21:47:24.893 | ERROR | changedetectionio.update_worker:run:437 - Exception reached processing watch UUID: 04b23b65-6540-4e9b-91d3-fa396b932e04 |
Beta Was this translation helpful? Give feedback.
-
using changedetection with playwright (browserless/chrome:1-chrome-stable) |
Beta Was this translation helpful? Give feedback.
-
Thanks! hmmm yeah, can you 'share' the watch and provide the share 'link' ? |
Beta Was this translation helpful? Give feedback.
-
No prob! thanks for this awesome app! |
Beta Was this translation helpful? Give feedback.
-
Thanks! unable to reproduce.. something different is happening, like the page is crashing or something else... |
Beta Was this translation helpful? Give feedback.
-
yeah 60 seconds is a long time... on your hardware maybe browserless is timing out? and the |
Beta Was this translation helpful? Give feedback.
-
btw, in the |
Beta Was this translation helpful? Give feedback.
-
is it running on windows or linux or mac or ? |
Beta Was this translation helpful? Give feedback.
-
It's running on linux. I went back to 0.45.13 after you confirmed the shared link works. Then after launching a bit more checks im seeing this is sporadic. Initially it looked to me an obvious and evident situation but now it seems less the case and could be localized to my environment. No difference with browserless image. If you would like me to try something else let me know. |
Beta Was this translation helpful? Give feedback.
-
Ah even better changedetection.io/docker-compose.yml Line 21 in 2e88872 uncomment this line and set it to |
Beta Was this translation helpful? Give feedback.
-
Here's the full watch with trace enabled below. Just sidenote -- i also have a x11vnc on the playwright docker and it appears to be loading the page fine in headless=false mode:
|
Beta Was this translation helpful? Give feedback.
-
For good measure i went back to the previous version and all watches work |
Beta Was this translation helpful? Give feedback.
-
is it always the same sites that have the problem? if you add a totall different site like https://changedetection.io , do you get an error? |
Beta Was this translation helpful? Give feedback.
-
|
Beta Was this translation helpful? Give feedback.
-
docker-compose.yml
|
Beta Was this translation helpful? Give feedback.
-
@parttimer777 please use the default options for playwright and let me know if it solves it
|
Beta Was this translation helpful? Give feedback.
-
Tested one watch and same error. couldnt test another due to amazon captcha. I'll also do a clean docker/data vol of changedetect and report back |
Beta Was this translation helpful? Give feedback.
-
For now i'll bump it to discussions if it was an issue others would report it, but it seems to be something very specific to your setup but we cant figure it out yet |
Beta Was this translation helpful? Give feedback.
-
Thanks for your help. Sounds good. I'll keep an eye out on the boards. One last update -- did a clean data volume, added the failing watch from the share link and it fails with the error. Then I downgraded to previous version and reran it, and it worked. Strange.. |
Beta Was this translation helpful? Give feedback.
-
Maybe your browserless container is crashing and not even returning a can you attach logs of that container? (not the changedetection one) |
Beta Was this translation helpful? Give feedback.
-
Can confirm the same error happening with my setup. docker compose file
It throws errors randomly. If I re-check it again it mostly works. |
Beta Was this translation helpful? Give feedback.
-
#2130 added some extra logging and checking |
Beta Was this translation helpful? Give feedback.
-
Please test with Then please attach a few lines of the logs, there should be some new output https://github.com/dgtlmoon/changedetection.io/pull/2130/files @addicted-ai @parttimer777 please use this container (make sure you pull the latest) and try to reproduce the error and paste more debug info here from the container logs |
Beta Was this translation helpful? Give feedback.
-
I think you're advanced in your investigation, but just want to mention that I experience the same errors on a few of my trackings. |
Beta Was this translation helpful? Give feedback.
-
As a data point, I've been getting this error sporadically as well (no proxy, Raspberries Pi, running within Kubernetes of all godforsaken things, all via playwright). Subscribing to see what folks uncover. |
Beta Was this translation helpful? Give feedback.
-
Guys. if you have a log like
if you change the env like looks like FYI, my image is
|
Beta Was this translation helpful? Give feedback.
-
In the case you're running on a raspberry pi or similar, i'm afraid theres nothing you can do about it - you simply dont have enough CPU, thats the physics of it try to set your |
Beta Was this translation helpful? Give feedback.
-
@Constantin1489 this thread is getting too long and a little off-topic, did |
Beta Was this translation helpful? Give feedback.
-
one reason behind the preference for our own browser Also means we can unify all the max wait seconds etc etc from changedetection to the browser, right now its a big mix, about 3 or 4 variables, but it could be all set in the sockpuppetbrowser then everything can work accordingly |
Beta Was this translation helpful? Give feedback.
Uh oh!
There was an error while loading. Please reload this page.
-
Describe the bug
Regression in newest version. Reverted and the previous version works fine.
Receiving this error for every website configured for change detection:
Error checking websites "'Response' object has no attribute 'status_code'
Version
0.45.13
To Reproduce
Error checking any website
Expected behavior
Should check website, apply xpath filter, and detect changes
Screenshots
n/a
Desktop (please complete the following information):
docker amd64
Smartphone (please complete the following information):
n/a
Additional context
n/a
Beta Was this translation helpful? Give feedback.
All reactions