Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

alpine: Don't install ca-certificates on Alpine 3.7+#307

Merged
tianon merged 2 commits into
docker-library:masterdocker-library/python:masterfrom
JayH5:alpine-3.7+-ca-certsJayH5/python:alpine-3.7+-ca-certsCopy head branch name to clipboard
Jul 25, 2018
Merged

alpine: Don't install ca-certificates on Alpine 3.7+#307
tianon merged 2 commits into
docker-library:masterdocker-library/python:masterfrom
JayH5:alpine-3.7+-ca-certsJayH5/python:alpine-3.7+-ca-certsCopy head branch name to clipboard

Conversation

@JayH5

@JayH5 JayH5 commented Jul 10, 2018

Copy link
Copy Markdown
Contributor
  • CA certs come bundled on Alpine 3.7+
  • On Alpine 3.6, rather install libressl which also include CA certs, allows us to use wget, and doesn't increase the size over ca-certificates

No significant change to image size (decreases ~0.7MB). Mostly a cleanup.

@JayH5 JayH5 force-pushed the alpine-3.7+-ca-certs branch from 7c752fa to 92f99db Compare July 16, 2018 21:03
@JayH5

JayH5 commented Jul 16, 2018

Copy link
Copy Markdown
Contributor Author

@tianon I've rebased this on top of #311

@JayH5 JayH5 force-pushed the alpine-3.7+-ca-certs branch from 92f99db to 2eaaa4a Compare July 23, 2018 15:18
* CA certs come bundled on Alpine 3.7+
* On Alpine 3.6, rather install libressl which also include CA certs,
  allows us to use wget, and doesn't increase the size over
  ca-certificates
@JayH5 JayH5 force-pushed the alpine-3.7+-ca-certs branch from 2eaaa4a to 93c1436 Compare July 23, 2018 20:01
@JayH5

JayH5 commented Jul 23, 2018

Copy link
Copy Markdown
Contributor Author

Rebased on top of #315

…tificates" installed and removing "libressl" after downloading)
@tianon tianon force-pushed the alpine-3.7+-ca-certs branch from d9536f1 to cc8d232 Compare July 25, 2018 19:21
@tianon tianon merged commit e078edc into docker-library:master Jul 25, 2018
@tianon

tianon commented Jul 25, 2018

Copy link
Copy Markdown
Member

Thanks @JayH5 ❤️

(especially for your patience 😅)

tianon added a commit to infosiftr/stackbrew that referenced this pull request Jul 26, 2018
- `gcc`: 8.2.0
- `ghost`: 1.25.2
- `mongo`: minor comment update
- `mysql`: `gpgconf --kill all` (docker-library/mysql#459)
- `postgres`: `gpgconf --kill all` (docker-library/postgres#471)
- `python`: remove `ca-certificates` from Alpine 3.7+ (docker-library/python#307)
- `redis`: suite aliases (redis/docker-library-redis#154)
- `rocket.chat`: 0.67.0
- `tomcat`: jessie arch removals
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.