fix: Use base image that includes CA certs #35
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Ran into issues where the pizza image could not make requests to GitHub due to TLS errors:
Notice the errors on not being able to verity the server's cerrs due to "unknown authority"
TLDR: we need a base image that has well known CAs. We can't just use a scratch image and I don't want to be in the business of managing and loading our own CAs for the pizza oven service.
What type of PR is this? (check all applicable)
Related Tickets & Documents
N/a - related to production usage.
Mobile & Desktop Screenshots/Recordings
N/a
Added tests?
Added to documentation?
[optional] Are there any post-deployment tasks we need to perform?
Will need to do another release and get this in prod / dev environments to validate fully but after building locally, this now works so I have high confidence this is the fix.
[optional] What gif best describes this PR or how it makes you feel?