-
Notifications
You must be signed in to change notification settings - Fork 53
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
feat: linux-musl target,docker image,add rc4-md5 cipher #555
Conversation
1. add rc4-md5 cipher
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm only small note bar @Itsusinn 's comments
.github/workflows/ci.yml
Outdated
push: true | ||
tags: ${{ env.TAG_VERSION }},${{ env.TAG_LATEST }} | ||
|
||
- name: Delete all images from package without tags |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove this if we only build images on release
For docker hub or GitHub docker registry? |
Currently only for GitHub image registry (ghcr.io), do I need to support docker.io? |
nah ghcr is good enough for now |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tysm!
🤔 This is a ...
🔗 Related issue link
None
💡 Background and solution
📝 Changelog
Tweaked CI build to add x86_64-unknown-linux-musl target
Added docker image based on x86_64-unknown-linux-musl package.
Added rc4-md5 cipher parsing.
☑️ Self-Check before Merge
You need to add two action secrets: DOCKER_USERNAME and DOCKER_PASSWORD.