-
-
Notifications
You must be signed in to change notification settings - Fork 42
Replies: 1 comment · 7 replies
-
|
There has not been any explicit support for those env vars added. What issues are you seeing caused by the lack of support? |
Beta Was this translation helpful? Give feedback.
All reactions
-
|
to be more precise, the issue is when I hit the /api/product?url=xxxx, I get a 500 err after a while, with this error in the logs |
Beta Was this translation helpful? Give feedback.
All reactions
-
|
Gotcha, thanks for the additional information. That API uses the package "got-scraping" to fetch the page. I'm sure I can configure it to be proxy-aware |
Beta Was this translation helpful? Give feedback.
All reactions
-
|
Something like this seems to make it working (but I couldn't check it doesn't break when no proxy is set as I can't make requests without a proxy in my env) |
Beta Was this translation helpful? Give feedback.
All reactions
-
👍 1
-
|
I’d be happy to accept a PR with the change! I can test it out without a proxy |
Beta Was this translation helpful? Give feedback.
All reactions
-
|
Here it is #482 |
Beta Was this translation helpful? Give feedback.
Uh oh!
There was an error while loading. Please reload this page.
-
I'm trying to run whishlist in an env where using an outbound proxy is mandatory for external web access. It seems setting the (almost) standard http_proxy / https_proxy env vars is not working. Anyone knows how this can be achieved ?
Beta Was this translation helpful? Give feedback.
All reactions