Skip to content

Conversation

@Sibert
Copy link

@Sibert Sibert commented Sep 26, 2022

SSID's are allowed to contain single quotes ('). This change allows the configurator to work with SSID's (and passphrases) to contain single quotes. Previous version causes following behavior:

SSID: It's a great network name
Pass: RatOSIsGreat

$ wpa_passphrase 'It's a great network name' 'RatOSIsGreat'

This won't work, so it should be

$ wpa_passphrase "It's a great network name" "RatOSIsGreat"

hence this change

SSID's are allowed to contain single quotes ('). This change allows the configurator to work with SSID's (and passphrases) to contain single quotes.
Previous version causes following behavior:
SSID: It's a great network name
Pass: RatOSIsGreat

$ wpa_passphrase 'It's a great network name' 'RatOSIsGreat'

This won't work, so it should be 

$ wpa_passphrase "It's a great network name" "RatOSIsGreat"

hence this change
@miklschmidt
Copy link
Member

Thanks!

I think a better solution is to use spawn instead of exec, we don't need to quote or escape the arguments that way (but the process can be orphaned, although that seems like an OK tradeoff here).

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants