Programmer Humor

32410 readers

153 users here now

Post funny things about programming here! (Or just rant about your favourite programming language.)

Rules:

Posts must be relevant to programming, programmers, or computer science.
No NSFW content.
Jokes must be in good taste. No hate speech, bigotry, etc.

founded 5 years ago

MODERATORS

[email protected]

241

I may not be the brightest (lemmy.world)

submitted 2 months ago by aidan to c/[email protected]

25 comments fedilink hide all child comments

you are viewing a single comment's thread
view the rest of the comments

[–] [email protected] 3 points 2 months ago (1 children)

Here's what I'm reading:

startup-script line 27 threw the error.

I'm reading this and interpreting that line 27 of that script is

sudo echo "# FYI quotes(") must be escaped with \ like \"

I am confused why there is no trailing double quote, the last 3 chars should be \"" so perhaps this is a bad assumption but the best I can do with the available information.

So the fix here is to change startup-script line 27 so that you're not echoing things that might contain characters that might be interpreted by echo or your shell.

Now if startup-script is provided by your distro, there may be a reason that it's using echo, but I will tell you now whatever dipshit reason they provide they're fucking wrong because EXHIBIT A: # " fucks the script and rule 0 of linux is "don't break userspace".

Everything else allows any printable char after the # in a comment, that script is not special, comments are not to be interpreted by the program. That is a show-stopping bug in startup-script and must be fixed.

EOF

[–] aidan 1 points 2 months ago (1 children)

I’m reading this and interpreting that line 27 of that script is

And your interpretation is wrong. Line 27 is actuallly

sudo echo "${server_service}" > /lib/systemd/system/server.service

${server_service} is read from the file I posted in the 2nd image. Since it was a test script I hadn't bothered implementing any escaping tools, I wanted to make sure terraform allowed this first.

[–] [email protected] 3 points 2 months ago* (last edited 2 months ago) (1 children)

And there's your problem. You're echoing using double quotes which will interpret characters. Don't do that. That's a bug. cat or cp the file to the destination; printf if the contents are all in that variable.

[–] aidan 1 points 2 months ago

No, you're still misunderstanding what's being done. ${server_service} is an injected string, the string is the whole contents of the file. That file is not stored locally on the server, except through being injected here(by a terraform file template). And no, printf won't be any better than echo because its not format string, and I don't want any formatting from printf applied to it.