Skip to content
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

Remove linebreaks when Base64 encoding #154

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

roshkins
Copy link
Contributor

base64 adds linebreaks when encoding. use -w 0 to remove them. Throws JSON errors otherwise

`base64` adds linebreaks when encoding. use -w 0 to remove them. Throws JSON errors otherwise
@TrevorJTClarke TrevorJTClarke self-requested a review April 1, 2022 18:00
@TrevorJTClarke
Copy link
Contributor

@roshkins are you on linux?
I fully feel your pain - adding this change however doesn't work for the people most using this scripting... :(
Therefore, the solution here is to keep this as a referencable fix somewhere, or setup a second set of scripts for linux users.

The bigger problem is actually how the deploy fails for code, which ultimately means linux users cannot do the scripts until near-cli fixes/supports the PR: near/near-cli#938

@TrevorJTClarke TrevorJTClarke added P-Low Priority: Low S-L2 Size: Minor 10-20 code line change C-Testing Unit testing / integration testing T-Blocked Type: Blocked by another issue labels Apr 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Testing Unit testing / integration testing P-Low Priority: Low S-L2 Size: Minor 10-20 code line change T-Blocked Type: Blocked by another issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants