SettingCommand does not check for permissions #8

Closed
opened 2022-03-29 12:32:41 +00:00 by oko · 4 comments
Owner

Anyone can change any value on the settings commands. This is obviously a security issue.

Anyone can change any value on the settings commands. This is obviously a security issue.
oko added the
bug
help wanted
labels 2022-03-29 12:32:41 +00:00

Curious as to what permission would be designated as the one to determine whether or not you could utilize that command. I could always make a setting to designate an administrator role.

Curious as to what permission would be designated as the one to determine whether or not you could utilize that command. I could always make a setting to designate an administrator role.
Author
Owner

I think the safest route is just requiring admin. I would hope the one adding the bot is smart enough to properly configure it anyways, right?

I think the safest route is just requiring admin. I would hope the one adding the bot is smart enough to properly configure it anyways, right?

Figures, i'll have it require admin momentarily.

Figures, i'll have it require admin momentarily.

done

done
Laika closed this issue 2022-03-29 20:55:00 +00:00
Laika added this to the Kiafumi Functionality project 2022-04-04 07:32:43 +00:00
Sign in to join this conversation.
No milestone
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: oko/kiafumi#8
No description provided.