-
Notifications
You must be signed in to change notification settings - Fork 2
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
PDFkit error on libappstest #310
Comments
Resolve with
|
@crowesn Nice job. I do see that wkhtmltopdf-binary is already in our Gemfile. Are you sure it was necessary for you to install that gem again? But maybe our bundle install into the vendor directory isn't where it likes to live? Also, that symlink looks brittle like it would break if we change our ruby version or the wkhtmltopdf-binary version. So we need to come up with a way to stabilize it or something. |
It’s possible that I was outside of the application directory for some of my testing, which could be why I needed to install wkhtmltopdf-binary gem. The gem pulls env values and interpolates them into a string to point to the binary, based on os and processor. In our case, it’s pulling “ol” for open linux from our system even though we are able to use the centos binary. I can probably express this better verbally.
|
@hortongn Can we close this issue since we've crafted a solution? |
Reviewer reported broken pdf download on libappstest:
Log showed the app was looking for a 8.7 version of wkhtmltopdf (our prior symlink fix has 8.6 version), probably bumped during updates. ln -s /opt/webapps/treatment_database/shared/vendor/bundle/ruby/2.7.0/gems/wkhtmltopdf-binary-0.12.6.5/bin/wkhtmltopdf_centos_8_amd64.gz /opt/webapps/treatment_database/shared/vendor/bundle/ruby/2.7.0/gems/wkhtmltopdf-binary-0.12.6.5/bin/wkhtmltopdf_ol_8.7_amd64.gz |
The text was updated successfully, but these errors were encountered: