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.

Sign up for GitHub

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

Jump to bottom

fix: dioscuri migration uses invalid base64 decode flag #279

Merged

shreddedbacon merged 1 commit into main from fix-dioscuri-migration

Dec 12, 2023

Member

rocketeerbkw commented Dec 12, 2023

An error was introduced in #265 due to invalid base64 flag:

base64: unrecognized option: decode
BusyBox v1.36.0 (2023-05-05 06:41:49 UTC) multi-call binary.

Usage: base64 [-d] [-w COL] [FILE]

Base64 encode or decode FILE to standard output

	-d	Decode data
	-w COL	Wrap lines at COL (default 76, 0 disables)

This PR removes the call to base64 altogether in favor of jq. Tested in a live build-deploy-tool pod.


          fix: dioscuri migration uses invalid base64 decode flag

ebd8ee5

shreddedbacon approved these changes

View reviewed changes

Member

shreddedbacon left a comment

Nice catch

shreddedbacon merged commit b849107 into main

2 checks passed

shreddedbacon deleted the fix-dioscuri-migration branch

December 12, 2023 20:04

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet