mentioning any role crashes webhook logs

[solareon]

Summary

When passing any tag to the logger module with a webhook fails

Reproduction

Pass a log to the logger module with a webhook and a tag

Expected behavior

Webhook to post with tag

Actual behavior

Error on console and no webhook

Additional context

[script:qbx_smallreso] SCRIPT ERROR: @qbx_core/modules/logger.lua:55: attempt to concatenate a nil value (field '?')
[script:qbx_smallreso] > logPayload (@qbx_core/modules/logger.lua:55)
[script:qbx_smallreso] > fn (@qbx_core/modules/logger.lua:84)

Additionally the logger documentation should reflect that roles other than @everyone or @here need to be called using user/role ids or the user will not be tagged.

Last Updated

today

Custom Resources

n/a

Resource Rename

n/a

[Manason]

Do you know why this might be occurring? The code looks to protect against nil values here https://github.com/Qbox-project/qbx_core/blob/main/modules/logger.lua#L54-L55

[solareon]

It seems to be related to how it handles the payload.tags[i] which I guess doesn't look like a table when it's just a single value. Looking at the docs again this might be me being a bonehead not wrapping my payload tags in [].

[solareon]

Okay this was due to not wrapping the @everyone in {} like this {'@everyone'}. Doing that fixed it up.