Skip to content
This repository has been archived by the owner on Feb 16, 2022. It is now read-only.

Upgrade PgHero, Rails, and Ruby. Use Alpine base image for size #12

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

gaffneyc
Copy link

@gaffneyc gaffneyc commented Mar 15, 2018

I ran into an error trying to run against PostgreSQL 10.3 due to changes
in the stat functions. Upgrading both Rails and PgHero was pretty
straight forward so I'm tackling both.

I removed rails_12factor as it's no longer necessary with Rails 5,
though there are config changes that I've integrated into the config.

This upgrades Ruby to 2.5.0 (or whatever the latest 2.x series is) and
switches to an Alpine based image.

Using the Debian base image produced a container that was ~850MB.
Switching to the recommended Alpine image reduced that to ~250MB (3/10th
the size).

Fixes #10

Ran into an error trying to run against PostgreSQL 10.3 due to changes
in the stat functions. Upgrading both Rails and PgHero was pretty
straight forward so I'm tackling both.

I removed rails_12factor as it's no longer necessary with Rails 5,
though there are config changes that I've integrated into the config.
The upgrades Ruby to 2.5.0 (or whatever the latest 2.x series is) and
switches to an Alpine based image.

Using the Debian base image produced a container that was roughly 850MB.
Switching to the recommended Alpine image reduced that to ~250MB (3/10th
the size).
Loofah had a set of CVEs for cross site scripting issues, none of which
should impact pghero.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant