-
Notifications
You must be signed in to change notification settings - Fork 74
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
PHP scrapers that use composer are all failed #1204
Comments
I've been able to reproduce this in my dev environment. |
So at minimum we're going to have to jump to heroku-16 which has quite a different set of packages. I don't know how this will affect things and I don't really have a good way to test this before we roll it out everywhere. I can easily test things on my dev instance, but I'm not going to be able to test everything.. |
@jamezpolley I'm pretty sure it's possible to change the buildpacks without upgrading to heroku-16. See for example https://github.com/openaustralia/buildstep/blob/master/Dockerfile#L11. That install a new buildpack that isn't there by default. I strongly suspect you can use the same or similar command to upgrade an existing buildpack. |
However, having said that it will obviously be necessary soon to upgrade to heroku-16 if cedar-14 is near its end-of-life. Maybe it can be done in stages to minimise potential disruption and catch small errors soon? |
Is there a fix to this issue or do no php scrapers work at this time? |
It's not well documented, but we've added support for herokuish:18 as a build platform. To use it, add a |
I've tried that fix and it does not resolve the issue. It does change the the error slightly to this (I've left in all the "tofu" characters morph seems to be adding. I'm not sure why they are appearing):
|
It was working until some upgrade performed in the backend in Dec.
Error message
And maybe relating to this with heroku
Examples of scrapers that use composer and not working:
https://morph.io/planningalerts-scrapers/townsville
https://morph.io/planningalerts-scrapers/tweed-shire
https://morph.io/planningalerts-scrapers/albury
Example of scraper doesn't use composer and working:
https://morph.io/planningalerts-scrapers/hurstville
The text was updated successfully, but these errors were encountered: