-
Notifications
You must be signed in to change notification settings - Fork 28
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
fix(unattended): remove quotes that messed up database package selection #4139
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 1
Review Details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (1)
- centreon/unattended.sh (1 hunks)
Additional Context Used
ShellCheck (114)
centreon/unattended.sh (114)
[info] 11-11: Double quote to prevent globbing and word splitting.
[info] 39-39: Double quote to prevent globbing and word splitting.
[info] 41-41: Double quote to prevent globbing and word splitting.
[warning] 43-43: This $? refers to echo/printf, not a previous command. Assign to variable to avoid it being overwritten.
[style] 43-43: Check exit code directly with e.g. 'if ! mycmd;', not indirectly with $?.
[info] 49-49: Double quote to prevent globbing and word splitting.
[warning] 53-53: CENTREON_RELEASE_VERSION appears unused. Verify use (or export if used externally).
[warning] 128-128: Assigning an array to a string! Assign as array, or use * instead of @ to concatenate.
[info] 175-175: Double quote to prevent globbing and word splitting.
[info] 200-200: Double quote to prevent globbing and word splitting.
[warning] 209-209: The surrounding quotes actually unquote this. Remove or escape them.
[info] 209-209: Double quote to prevent globbing and word splitting.
[info] 211-211: Command appears to be unreachable. Check usage (or ignore if invoked indirectly).
[info] 216-216: Command appears to be unreachable. Check usage (or ignore if invoked indirectly).
[warning] 220-220: The surrounding quotes actually unquote this. Remove or escape them.
[info] 220-220: Double quote to prevent globbing and word splitting.
[info] 222-222: Command appears to be unreachable. Check usage (or ignore if invoked indirectly).
[error] 263-263: -n doesn't work with unquoted arguments. Quote or use [[ ]].
[info] 263-263: Double quote to prevent globbing and word splitting.
[info] 266-266: read without -r will mangle backslashes.
[info] 266-266: Double quote to prevent globbing and word splitting.
[style] 336-336: Use [ -n .. ] instead of ! [ -z .. ].
[info] 336-336: Double quote to prevent globbing and word splitting.
[warning] 355-355: Arrays don't work as operands in [ ]. Use a loop (or concatenate with * instead of @).
[style] 384-384: Check exit code directly with e.g. 'if ! mycmd;', not indirectly with $?.
[style] 409-409: Check exit code directly with e.g. 'if ! mycmd;', not indirectly with $?.
[error] 525-525: Argument mixes string and array. Use * or separate argument.
[error] 526-526: Double quote array expansions to avoid re-splitting elements.
[info] 578-578: Double quote to prevent globbing and word splitting.
[info] 580-580: Double quote to prevent globbing and word splitting.
[info] 581-581: Double quote to prevent globbing and word splitting.
[style] 628-628: Check exit code directly with e.g. 'if ! mycmd;', not indirectly with $?.
[warning] 673-673: This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten.
[style] 717-717: Check exit code directly with e.g. 'if ! mycmd;', not indirectly with $?.
[info] 734-734: Double quote to prevent globbing and word splitting.
[style] 735-735: Check exit code directly with e.g. 'if ! mycmd;', not indirectly with $?.
[style] 750-750: Check exit code directly with e.g. 'if ! mycmd;', not indirectly with $?.
[style] 752-752: Check exit code directly with e.g. 'if ! mycmd;', not indirectly with $?.
[style] 769-769: Check exit code directly with e.g. 'if mycmd;', not indirectly with $?.
[style] 771-771: Check exit code directly with e.g. 'if mycmd;', not indirectly with $?.
[style] 774-774: Check exit code directly with e.g. 'if ! mycmd;', not indirectly with $?.
[style] 780-780: Check exit code directly with e.g. 'if ! mycmd;', not indirectly with $?.
[info] 860-860: Double quote to prevent globbing and word splitting.
[info] 861-861: Double quote to prevent globbing and word splitting.
[info] 862-862: Double quote to prevent globbing and word splitting.
[info] 863-863: Double quote to prevent globbing and word splitting.
[info] 864-864: Double quote to prevent globbing and word splitting.
[info] 865-865: Double quote to prevent globbing and word splitting.
[info] 866-866: Double quote to prevent globbing and word splitting.
[info] 867-867: Double quote to prevent globbing and word splitting.
[info] 868-868: Double quote to prevent globbing and word splitting.
[info] 869-869: Double quote to prevent globbing and word splitting.
[info] 870-870: Double quote to prevent globbing and word splitting.
[info] 875-875: Double quote to prevent globbing and word splitting.
[info] 876-876: Double quote to prevent globbing and word splitting.
[warning] 900-900: This { is literal. Check expression (missing ;/\n?) or quote it.
[warning] 900-900: This } is literal. Check expression (missing ;/\n?) or quote it.
[style] 911-911: See if you can use ${variable//search/replace} instead.
[info] 911-911: Double quote to prevent globbing and word splitting.
[warning] 943-943: This { is literal. Check expression (missing ;/\n?) or quote it.
[warning] 943-943: This } is literal. Check expression (missing ;/\n?) or quote it.
[style] 952-952: Use $(...) notation instead of legacy backticks
...
.
[style] 960-960: See if you can use ${variable//search/replace} instead.
[info] 960-960: Double quote to prevent globbing and word splitting.
[warning] 981-981: This { is literal. Check expression (missing ;/\n?) or quote it.
[warning] 981-981: This } is literal. Check expression (missing ;/\n?) or quote it.
[style] 989-989: Use $(...) notation instead of legacy backticks
...
.
[warning] 1012-1012: This { is literal. Check expression (missing ;/\n?) or quote it.
[warning] 1012-1012: This } is literal. Check expression (missing ;/\n?) or quote it.
[style] 1022-1022: Use $(...) notation instead of legacy backticks
...
.
[info] 1030-1030: Double quote to prevent globbing and word splitting.
[warning] 1048-1048: This { is literal. Check expression (missing ;/\n?) or quote it.
[warning] 1048-1048: This } is literal. Check expression (missing ;/\n?) or quote it.
[style] 1057-1057: Use $(...) notation instead of legacy backticks
...
.
[info] 1068-1068: Double quote to prevent globbing and word splitting.
[info] 1072-1072: Double quote to prevent globbing and word splitting.
[info] 1073-1073: read without -r will mangle backslashes.
[info] 1073-1073: Double quote to prevent globbing and word splitting.
[warning] 1081-1081: This { is literal. Check expression (missing ;/\n?) or quote it.
[warning] 1081-1081: This } is literal. Check expression (missing ;/\n?) or quote it.
[style] 1091-1091: Use $(...) notation instead of legacy backticks
...
.
[info] 1099-1099: Double quote to prevent globbing and word splitting.
[info] 1100-1100: Double quote to prevent globbing and word splitting.
[info] 1111-1111: Double quote to prevent globbing and word splitting.
[info] 1115-1115: Double quote to prevent globbing and word splitting.
[info] 1116-1116: read without -r will mangle backslashes.
[info] 1116-1116: Double quote to prevent globbing and word splitting.
[warning] 1124-1124: This { is literal. Check expression (missing ;/\n?) or quote it.
[warning] 1124-1124: This } is literal. Check expression (missing ;/\n?) or quote it.
[style] 1134-1134: Use $(...) notation instead of legacy backticks
...
.
[info] 1142-1142: Double quote to prevent globbing and word splitting.
[info] 1143-1143: Double quote to prevent globbing and word splitting.
[style] 1189-1189: Check exit code directly with e.g. 'if ! mycmd;', not indirectly with $?.
[style] 1196-1196: Check exit code directly with e.g. 'if ! mycmd;', not indirectly with $?.
[info] 1205-1214: Expressions don't expand in single quotes, use double quotes for that.
[style] 1236-1236: Check exit code directly with e.g. 'if ! mycmd;', not indirectly with $?.
[info] 1240-1240: Double quote to prevent globbing and word splitting.
[style] 1241-1241: Check exit code directly with e.g. 'if ! mycmd;', not indirectly with $?.
[info] 1256-1256: Double quote to prevent globbing and word splitting.
[style] 1257-1257: Check exit code directly with e.g. 'if ! mycmd;', not indirectly with $?.
[error] 1287-1287: Double quote array expansions to avoid re-splitting elements.
[style] 1288-1288: Check exit code directly with e.g. 'if ! mycmd;', not indirectly with $?.
[info] 1295-1295: Double quote to prevent globbing and word splitting.
[info] 1297-1297: Double quote to prevent globbing and word splitting.
[info] 1304-1304: This - will be a regular '-' in this context.
[info] 1304-1304: This : will be a regular ':' in this context.
[info] 1304-1304: This / will be a regular '/' in this context.
[info] 1324-1324: Command appears to be unreachable. Check usage (or ignore if invoked indirectly).
[info] 1358-1358: Double quote to prevent globbing and word splitting.
[info] 1455-1455: Double quote to prevent globbing and word splitting.
[info] 1457-1457: Double quote to prevent globbing and word splitting.
[info] 1461-1461: Possible misspelling: DBMS may not be assigned. Did you mean dbms?
[info] 1471-1471: Double quote to prevent globbing and word splitting.
[info] 1472-1472: Double quote to prevent globbing and word splitting.
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 1
Review Details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (1)
- centreon/unattended.sh (1 hunks)
Additional Context Used
ShellCheck (114)
centreon/unattended.sh (114)
[info] 11-11: Double quote to prevent globbing and word splitting.
[info] 39-39: Double quote to prevent globbing and word splitting.
[info] 41-41: Double quote to prevent globbing and word splitting.
[warning] 43-43: This $? refers to echo/printf, not a previous command. Assign to variable to avoid it being overwritten.
[style] 43-43: Check exit code directly with e.g. 'if ! mycmd;', not indirectly with $?.
[info] 49-49: Double quote to prevent globbing and word splitting.
[warning] 53-53: CENTREON_RELEASE_VERSION appears unused. Verify use (or export if used externally).
[warning] 128-128: Assigning an array to a string! Assign as array, or use * instead of @ to concatenate.
[info] 175-175: Double quote to prevent globbing and word splitting.
[info] 200-200: Double quote to prevent globbing and word splitting.
[warning] 209-209: The surrounding quotes actually unquote this. Remove or escape them.
[info] 209-209: Double quote to prevent globbing and word splitting.
[info] 211-211: Command appears to be unreachable. Check usage (or ignore if invoked indirectly).
[info] 216-216: Command appears to be unreachable. Check usage (or ignore if invoked indirectly).
[warning] 220-220: The surrounding quotes actually unquote this. Remove or escape them.
[info] 220-220: Double quote to prevent globbing and word splitting.
[info] 222-222: Command appears to be unreachable. Check usage (or ignore if invoked indirectly).
[error] 263-263: -n doesn't work with unquoted arguments. Quote or use [[ ]].
[info] 263-263: Double quote to prevent globbing and word splitting.
[info] 266-266: read without -r will mangle backslashes.
[info] 266-266: Double quote to prevent globbing and word splitting.
[style] 336-336: Use [ -n .. ] instead of ! [ -z .. ].
[info] 336-336: Double quote to prevent globbing and word splitting.
[warning] 355-355: Arrays don't work as operands in [ ]. Use a loop (or concatenate with * instead of @).
[style] 384-384: Check exit code directly with e.g. 'if ! mycmd;', not indirectly with $?.
[style] 409-409: Check exit code directly with e.g. 'if ! mycmd;', not indirectly with $?.
[error] 525-525: Argument mixes string and array. Use * or separate argument.
[error] 526-526: Double quote array expansions to avoid re-splitting elements.
[info] 578-578: Double quote to prevent globbing and word splitting.
[info] 580-580: Double quote to prevent globbing and word splitting.
[info] 581-581: Double quote to prevent globbing and word splitting.
[style] 628-628: Check exit code directly with e.g. 'if ! mycmd;', not indirectly with $?.
[warning] 673-673: This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten.
[style] 717-717: Check exit code directly with e.g. 'if ! mycmd;', not indirectly with $?.
[info] 734-734: Double quote to prevent globbing and word splitting.
[style] 735-735: Check exit code directly with e.g. 'if ! mycmd;', not indirectly with $?.
[style] 750-750: Check exit code directly with e.g. 'if ! mycmd;', not indirectly with $?.
[style] 752-752: Check exit code directly with e.g. 'if ! mycmd;', not indirectly with $?.
[style] 769-769: Check exit code directly with e.g. 'if mycmd;', not indirectly with $?.
[style] 771-771: Check exit code directly with e.g. 'if mycmd;', not indirectly with $?.
[style] 774-774: Check exit code directly with e.g. 'if ! mycmd;', not indirectly with $?.
[style] 780-780: Check exit code directly with e.g. 'if ! mycmd;', not indirectly with $?.
[info] 860-860: Double quote to prevent globbing and word splitting.
[info] 861-861: Double quote to prevent globbing and word splitting.
[info] 862-862: Double quote to prevent globbing and word splitting.
[info] 863-863: Double quote to prevent globbing and word splitting.
[info] 864-864: Double quote to prevent globbing and word splitting.
[info] 865-865: Double quote to prevent globbing and word splitting.
[info] 866-866: Double quote to prevent globbing and word splitting.
[info] 867-867: Double quote to prevent globbing and word splitting.
[info] 868-868: Double quote to prevent globbing and word splitting.
[info] 869-869: Double quote to prevent globbing and word splitting.
[info] 870-870: Double quote to prevent globbing and word splitting.
[info] 875-875: Double quote to prevent globbing and word splitting.
[info] 876-876: Double quote to prevent globbing and word splitting.
[warning] 900-900: This { is literal. Check expression (missing ;/\n?) or quote it.
[warning] 900-900: This } is literal. Check expression (missing ;/\n?) or quote it.
[style] 911-911: See if you can use ${variable//search/replace} instead.
[info] 911-911: Double quote to prevent globbing and word splitting.
[warning] 943-943: This { is literal. Check expression (missing ;/\n?) or quote it.
[warning] 943-943: This } is literal. Check expression (missing ;/\n?) or quote it.
[style] 952-952: Use $(...) notation instead of legacy backticks
...
.
[style] 960-960: See if you can use ${variable//search/replace} instead.
[info] 960-960: Double quote to prevent globbing and word splitting.
[warning] 981-981: This { is literal. Check expression (missing ;/\n?) or quote it.
[warning] 981-981: This } is literal. Check expression (missing ;/\n?) or quote it.
[style] 989-989: Use $(...) notation instead of legacy backticks
...
.
[warning] 1012-1012: This { is literal. Check expression (missing ;/\n?) or quote it.
[warning] 1012-1012: This } is literal. Check expression (missing ;/\n?) or quote it.
[style] 1022-1022: Use $(...) notation instead of legacy backticks
...
.
[info] 1030-1030: Double quote to prevent globbing and word splitting.
[warning] 1048-1048: This { is literal. Check expression (missing ;/\n?) or quote it.
[warning] 1048-1048: This } is literal. Check expression (missing ;/\n?) or quote it.
[style] 1057-1057: Use $(...) notation instead of legacy backticks
...
.
[info] 1068-1068: Double quote to prevent globbing and word splitting.
[info] 1072-1072: Double quote to prevent globbing and word splitting.
[info] 1073-1073: read without -r will mangle backslashes.
[info] 1073-1073: Double quote to prevent globbing and word splitting.
[warning] 1081-1081: This { is literal. Check expression (missing ;/\n?) or quote it.
[warning] 1081-1081: This } is literal. Check expression (missing ;/\n?) or quote it.
[style] 1091-1091: Use $(...) notation instead of legacy backticks
...
.
[info] 1099-1099: Double quote to prevent globbing and word splitting.
[info] 1100-1100: Double quote to prevent globbing and word splitting.
[info] 1111-1111: Double quote to prevent globbing and word splitting.
[info] 1115-1115: Double quote to prevent globbing and word splitting.
[info] 1116-1116: read without -r will mangle backslashes.
[info] 1116-1116: Double quote to prevent globbing and word splitting.
[warning] 1124-1124: This { is literal. Check expression (missing ;/\n?) or quote it.
[warning] 1124-1124: This } is literal. Check expression (missing ;/\n?) or quote it.
[style] 1134-1134: Use $(...) notation instead of legacy backticks
...
.
[info] 1142-1142: Double quote to prevent globbing and word splitting.
[info] 1143-1143: Double quote to prevent globbing and word splitting.
[style] 1189-1189: Check exit code directly with e.g. 'if ! mycmd;', not indirectly with $?.
[style] 1196-1196: Check exit code directly with e.g. 'if ! mycmd;', not indirectly with $?.
[info] 1205-1214: Expressions don't expand in single quotes, use double quotes for that.
[style] 1236-1236: Check exit code directly with e.g. 'if ! mycmd;', not indirectly with $?.
[info] 1240-1240: Double quote to prevent globbing and word splitting.
[style] 1241-1241: Check exit code directly with e.g. 'if ! mycmd;', not indirectly with $?.
[info] 1256-1256: Double quote to prevent globbing and word splitting.
[style] 1257-1257: Check exit code directly with e.g. 'if ! mycmd;', not indirectly with $?.
[error] 1287-1287: Double quote array expansions to avoid re-splitting elements.
[style] 1288-1288: Check exit code directly with e.g. 'if ! mycmd;', not indirectly with $?.
[info] 1295-1295: Double quote to prevent globbing and word splitting.
[info] 1297-1297: Double quote to prevent globbing and word splitting.
[info] 1304-1304: This - will be a regular '-' in this context.
[info] 1304-1304: This : will be a regular ':' in this context.
[info] 1304-1304: This / will be a regular '/' in this context.
[info] 1324-1324: Command appears to be unreachable. Check usage (or ignore if invoked indirectly).
[info] 1358-1358: Double quote to prevent globbing and word splitting.
[info] 1455-1455: Double quote to prevent globbing and word splitting.
[info] 1457-1457: Double quote to prevent globbing and word splitting.
[info] 1461-1461: Possible misspelling: DBMS may not be assigned. Did you mean dbms?
[info] 1471-1471: Double quote to prevent globbing and word splitting.
[info] 1472-1472: Double quote to prevent globbing and word splitting.
Additional comments not posted (1)
centreon/unattended.sh (1)
1176-1176
: Refactor the version comparison logic for clarity and maintainability.- if [[ "$version" =~ ^24\.0[1-9]$ || "$version" =~ ^24\.1[0-2]$ ]]; then + if [[ "$version" =~ ^24\.0[1-9]$|^24\.1[0-2]$ ]]; thenThis change simplifies the regex by combining the conditions, making it easier to read and maintain.
…ON-fix-unattended
WalkthroughThe Changes
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 1
Outside diff range and nitpick comments (3)
centreon/unattended.sh (3)
Line range hint
128-128
: Ensure arrays are correctly handled when assigning to a string.- CENTREON_REPO="" + declare -a CENTREON_REPO=()
Line range hint
263-263
: Quote the variable to prevent word splitting and globbing issues.- if [ -n $2 ]; then + if [ -n "$2" ]; then
Line range hint
266-266
: Use the-r
option withread
to prevent backslashes from being interpreted.- read -t $timeout -s -n 1 -p "${1}" + read -r -t $timeout -s -n 1 -p "${1}"
Review Details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (1)
- centreon/unattended.sh (1 hunks)
Additional Context Used
ShellCheck (114)
centreon/unattended.sh (114)
[info] 11-11: Double quote to prevent globbing and word splitting.
[info] 39-39: Double quote to prevent globbing and word splitting.
[info] 41-41: Double quote to prevent globbing and word splitting.
[warning] 43-43: This $? refers to echo/printf, not a previous command. Assign to variable to avoid it being overwritten.
[style] 43-43: Check exit code directly with e.g. 'if ! mycmd;', not indirectly with $?.
[info] 49-49: Double quote to prevent globbing and word splitting.
[warning] 53-53: CENTREON_RELEASE_VERSION appears unused. Verify use (or export if used externally).
[warning] 128-128: Assigning an array to a string! Assign as array, or use * instead of @ to concatenate.
[info] 175-175: Double quote to prevent globbing and word splitting.
[info] 200-200: Double quote to prevent globbing and word splitting.
[warning] 209-209: The surrounding quotes actually unquote this. Remove or escape them.
[info] 209-209: Double quote to prevent globbing and word splitting.
[info] 211-211: Command appears to be unreachable. Check usage (or ignore if invoked indirectly).
[info] 216-216: Command appears to be unreachable. Check usage (or ignore if invoked indirectly).
[warning] 220-220: The surrounding quotes actually unquote this. Remove or escape them.
[info] 220-220: Double quote to prevent globbing and word splitting.
[info] 222-222: Command appears to be unreachable. Check usage (or ignore if invoked indirectly).
[error] 263-263: -n doesn't work with unquoted arguments. Quote or use [[ ]].
[info] 263-263: Double quote to prevent globbing and word splitting.
[info] 266-266: read without -r will mangle backslashes.
[info] 266-266: Double quote to prevent globbing and word splitting.
[style] 336-336: Use [ -n .. ] instead of ! [ -z .. ].
[info] 336-336: Double quote to prevent globbing and word splitting.
[warning] 355-355: Arrays don't work as operands in [ ]. Use a loop (or concatenate with * instead of @).
[style] 384-384: Check exit code directly with e.g. 'if ! mycmd;', not indirectly with $?.
[style] 409-409: Check exit code directly with e.g. 'if ! mycmd;', not indirectly with $?.
[error] 525-525: Argument mixes string and array. Use * or separate argument.
[error] 526-526: Double quote array expansions to avoid re-splitting elements.
[info] 578-578: Double quote to prevent globbing and word splitting.
[info] 580-580: Double quote to prevent globbing and word splitting.
[info] 581-581: Double quote to prevent globbing and word splitting.
[style] 628-628: Check exit code directly with e.g. 'if ! mycmd;', not indirectly with $?.
[warning] 673-673: This $? refers to a condition, not a command. Assign to a variable to avoid it being overwritten.
[style] 717-717: Check exit code directly with e.g. 'if ! mycmd;', not indirectly with $?.
[info] 734-734: Double quote to prevent globbing and word splitting.
[style] 735-735: Check exit code directly with e.g. 'if ! mycmd;', not indirectly with $?.
[style] 750-750: Check exit code directly with e.g. 'if ! mycmd;', not indirectly with $?.
[style] 752-752: Check exit code directly with e.g. 'if ! mycmd;', not indirectly with $?.
[style] 769-769: Check exit code directly with e.g. 'if mycmd;', not indirectly with $?.
[style] 771-771: Check exit code directly with e.g. 'if mycmd;', not indirectly with $?.
[style] 774-774: Check exit code directly with e.g. 'if ! mycmd;', not indirectly with $?.
[style] 780-780: Check exit code directly with e.g. 'if ! mycmd;', not indirectly with $?.
[info] 860-860: Double quote to prevent globbing and word splitting.
[info] 861-861: Double quote to prevent globbing and word splitting.
[info] 862-862: Double quote to prevent globbing and word splitting.
[info] 863-863: Double quote to prevent globbing and word splitting.
[info] 864-864: Double quote to prevent globbing and word splitting.
[info] 865-865: Double quote to prevent globbing and word splitting.
[info] 866-866: Double quote to prevent globbing and word splitting.
[info] 867-867: Double quote to prevent globbing and word splitting.
[info] 868-868: Double quote to prevent globbing and word splitting.
[info] 869-869: Double quote to prevent globbing and word splitting.
[info] 870-870: Double quote to prevent globbing and word splitting.
[info] 875-875: Double quote to prevent globbing and word splitting.
[info] 876-876: Double quote to prevent globbing and word splitting.
[warning] 900-900: This { is literal. Check expression (missing ;/\n?) or quote it.
[warning] 900-900: This } is literal. Check expression (missing ;/\n?) or quote it.
[style] 911-911: See if you can use ${variable//search/replace} instead.
[info] 911-911: Double quote to prevent globbing and word splitting.
[warning] 943-943: This { is literal. Check expression (missing ;/\n?) or quote it.
[warning] 943-943: This } is literal. Check expression (missing ;/\n?) or quote it.
[style] 952-952: Use $(...) notation instead of legacy backticks
...
.
[style] 960-960: See if you can use ${variable//search/replace} instead.
[info] 960-960: Double quote to prevent globbing and word splitting.
[warning] 981-981: This { is literal. Check expression (missing ;/\n?) or quote it.
[warning] 981-981: This } is literal. Check expression (missing ;/\n?) or quote it.
[style] 989-989: Use $(...) notation instead of legacy backticks
...
.
[warning] 1012-1012: This { is literal. Check expression (missing ;/\n?) or quote it.
[warning] 1012-1012: This } is literal. Check expression (missing ;/\n?) or quote it.
[style] 1022-1022: Use $(...) notation instead of legacy backticks
...
.
[info] 1030-1030: Double quote to prevent globbing and word splitting.
[warning] 1048-1048: This { is literal. Check expression (missing ;/\n?) or quote it.
[warning] 1048-1048: This } is literal. Check expression (missing ;/\n?) or quote it.
[style] 1057-1057: Use $(...) notation instead of legacy backticks
...
.
[info] 1068-1068: Double quote to prevent globbing and word splitting.
[info] 1072-1072: Double quote to prevent globbing and word splitting.
[info] 1073-1073: read without -r will mangle backslashes.
[info] 1073-1073: Double quote to prevent globbing and word splitting.
[warning] 1081-1081: This { is literal. Check expression (missing ;/\n?) or quote it.
[warning] 1081-1081: This } is literal. Check expression (missing ;/\n?) or quote it.
[style] 1091-1091: Use $(...) notation instead of legacy backticks
...
.
[info] 1099-1099: Double quote to prevent globbing and word splitting.
[info] 1100-1100: Double quote to prevent globbing and word splitting.
[info] 1111-1111: Double quote to prevent globbing and word splitting.
[info] 1115-1115: Double quote to prevent globbing and word splitting.
[info] 1116-1116: read without -r will mangle backslashes.
[info] 1116-1116: Double quote to prevent globbing and word splitting.
[warning] 1124-1124: This { is literal. Check expression (missing ;/\n?) or quote it.
[warning] 1124-1124: This } is literal. Check expression (missing ;/\n?) or quote it.
[style] 1134-1134: Use $(...) notation instead of legacy backticks
...
.
[info] 1142-1142: Double quote to prevent globbing and word splitting.
[info] 1143-1143: Double quote to prevent globbing and word splitting.
[style] 1189-1189: Check exit code directly with e.g. 'if ! mycmd;', not indirectly with $?.
[style] 1196-1196: Check exit code directly with e.g. 'if ! mycmd;', not indirectly with $?.
[info] 1205-1214: Expressions don't expand in single quotes, use double quotes for that.
[style] 1236-1236: Check exit code directly with e.g. 'if ! mycmd;', not indirectly with $?.
[info] 1240-1240: Double quote to prevent globbing and word splitting.
[style] 1241-1241: Check exit code directly with e.g. 'if ! mycmd;', not indirectly with $?.
[info] 1256-1256: Double quote to prevent globbing and word splitting.
[style] 1257-1257: Check exit code directly with e.g. 'if ! mycmd;', not indirectly with $?.
[error] 1287-1287: Double quote array expansions to avoid re-splitting elements.
[style] 1288-1288: Check exit code directly with e.g. 'if ! mycmd;', not indirectly with $?.
[info] 1295-1295: Double quote to prevent globbing and word splitting.
[info] 1297-1297: Double quote to prevent globbing and word splitting.
[info] 1304-1304: This - will be a regular '-' in this context.
[info] 1304-1304: This : will be a regular ':' in this context.
[info] 1304-1304: This / will be a regular '/' in this context.
[info] 1324-1324: Command appears to be unreachable. Check usage (or ignore if invoked indirectly).
[info] 1358-1358: Double quote to prevent globbing and word splitting.
[info] 1455-1455: Double quote to prevent globbing and word splitting.
[info] 1457-1457: Double quote to prevent globbing and word splitting.
[info] 1461-1461: Possible misspelling: DBMS may not be assigned. Did you mean dbms?
[info] 1471-1471: Double quote to prevent globbing and word splitting.
[info] 1472-1472: Double quote to prevent globbing and word splitting.
Additional comments not posted (1)
centreon/unattended.sh (1)
1176-1176
: Refactor the version comparison logic for clarity and maintainability.
…ion (#4139) Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
Description
Fixes # (issue)
Type of change
Target serie
How this pull request can be tested ?
Please describe the procedure to verify that the goal of the PR is matched. Provide clear instructions so that it can be correctly tested.
Any relevant details of the configuration to perform the test should be added.
Checklist
Community contributors & Centreon team