bbb-install icon indicating copy to clipboard operation
bbb-install copied to clipboard

illegal option -l

Open YouAreALegend opened this issue 1 year ago • 10 comments

I saw in the document that only deploying certificates can use the - l parameter, but when I use this parameter, the command line will prompt an error: bash: illegal option -- l bbb-install: Invalid option: -

YouAreALegend avatar Jul 21 '23 06:07 YouAreALegend

Hey @YouAreALegend

Which version of BBB were you installing? Could you add here the full command (feel free to mask the email / domain)

antobinary avatar Jul 21 '23 13:07 antobinary

BBB VERSION:2.6 wget -qO- https://ubuntu.bigbluebutton.org/bbb-install-2.6.sh | bash -s -- -v focal-260 -s XXX -e XXX@XX -l

YouAreALegend avatar Jul 28 '23 08:07 YouAreALegend

There isn't a -I option in the installer.

https://github.com/bigbluebutton/bbb-install/blob/master/bbb-install-2.6.sh#L47

I saw in the document that only deploying certificates can use the - l parameter,

Can you post the link where you saw this option specified.

ffdixon avatar Jul 28 '23 22:07 ffdixon

https://github.com/bigbluebutton/bbb-install/blob/2a58d62ba756b855b2476c3004516e2ce0912235/bbb-install-2.6.sh#L80

YouAreALegend avatar Jul 29 '23 07:07 YouAreALegend

That is a lowercase L whereas you trying to pass a lowercase I.

ffdixon avatar Jul 29 '23 13:07 ffdixon

wget -qO- https://ubuntu.bigbluebutton.org/bbb-install-2.6.sh | bash -s -- -v focal-260 -s XXX -e XXX@XX -l

According to https://convertcase.net/ the above was a lowercase L, not an uppercase i. So from what I see @YouAreALegend is passing a valid -l command (lower case L)

antobinary avatar Jul 29 '23 17:07 antobinary

Hi,

imho, the bbb-install script also in version 2.7.x is not considering any -l (lovercase L) switch. despite its documentation:

OPTIONS (install Let's Encrypt certificate only):

  -s <hostname>          Configure server with <hostname> (required)
  -e <email>             Configure email for Let's Encrypt certbot (required)
  -l                     Only install Let's Encrypt certificate (not BigBlueButton)
  -x                     Use Let's Encrypt certbot with manual dns challenges (optional)

As line 134 shows, OPT can never become lowercase L and will be handled as invalid input.

while builtin getopts "hs:r:c:v:e:p:m:t:xgadwjik" opt "${@}"; do

Since I was having troubles with certificates and BBB on AWS, I was wondering about that switch since it seems I must use LE certificates. The only two options are either to skip LE creation totally via -d switch or modify it via -x switch.

Can you clarify the differences between code and its documentation?

ronator avatar Oct 05 '23 17:10 ronator

It seems, someone realized, that the -l switch never had any effect since the variable LETS_ENCRYPT_ONLY is not used in the entire script e.g. to have conditional behaviour. However, I was too lazy to follow the commits, since I cannot look before 2.5 branch in turnme and this one had the -l switch but except setting a variable, it does not affect anything.

https://github.com/bigbluebutton/bbb-install/blob/turnme/bbb-install-2.5.sh#L168C31-L168C31

ronator avatar Oct 05 '23 19:10 ronator

Thanks for reporting this -- will take a closer look.

ffdixon avatar Oct 05 '23 23:10 ffdixon

What is the expected way to renew TLS certificates? Which CLI option does renew certificates

running ./bbb-install.sh -v focal-270 -s REDACTED -e REDACTED@REDACTED -l gives ./bbb-install.sh: illegal option -- l and the help screen. Am using v2.7.x-release branch.

txt-file avatar May 21 '24 11:05 txt-file