Skip to content
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 permissions for bot-owner-only commands that should be executable by server admins. #25

Open
official-Cromatin opened this issue Dec 24, 2021 · 3 comments

Comments

@official-Cromatin
Copy link

Ich wusste nicht so recht wo ich Feedback Hinterlassen sollte, deswegen mach ich das jetzt hier.

Ich benutze den Bot auf mehreren Servern und er leiset mir Gute Dienste,
wurde von anderen auch schon gefragt ob diese ihn auch Benutzen können (da ihnen die Möglichkeit fehlt ihn selbst zu Betreiben wollten sie das ich ihnen einen Einladungs Link Schicke), ich hatte es abgelehnt (auch aus dem Grund weil ich nicht weis wie das mit dem Bereitstellen ist) da ich ihre ID dann in die Config Eintragen müsste damit sie die Bot Owner Only Commands nutzen könnten.
Leider hätten sie dann auch zu meinen Servern vollen Zugang, was alles andere als Optimal wäre.

Mit ist allerdings die Idee gekommen das man das Problem löst indem die Person die den Bot auf einen Server einläd Automatisch der Owner (zumindest dieses Betreffenden Servers) ist.
(Die IDs die in der Config als Owner eingetragen werden haben vollen zugriff auf alle Server zu welchen der Bot eingeladen wurde)
Um allerdings die Rechte des Bots weitergeben zu können ohne ihn neu Einzuladen würde ich des weiteren einen weiteren Command festlegen (als BSP .transferownership @user kurz .to @user)

Diese können dann Natürlich ebenfalls sich auf der Seite Anmelden um Einstellungen am Bot für ihre Server Vorzunehmen
(Die Bot Config müsste dann Pro User Funktionieren)

Das wären meine Aktuellen Vorschläge die ich zur Verbesserung des Bots hätte

@expeehaa
Copy link
Collaborator

Freut mich, dass dir der Bot gefällt.

Botowner zu sein sollte einem eigentlich nur die Berechtigung zum Setzen/Ändern von serverunabhängigen, globalen Einstellungen geben. Von daher möchte ich deinen Vorschlag des Verteilens dieser Berechtigung per Befehl nicht umsetzen.
Stattdessen kannst du mir aber gerne die Befehle schicken, die ein Serveradmin ausführen können sollte, damit ich die notwendigen Berechtigungen ändern kann.

Ich weiß, dass das Problem existiert, hatte aber bisher keinen Grund, mich damit zu beschäftigen.

@official-Cromatin
Copy link
Author

Das wären Bot-Owner-Only Commands, aber diese hier wären wichtig (Alle anderen würden globale Einstellungen betreffen):

.edit
.logserver
.logignore
.logevents
.log
.loglist
.send (Nur auf dem Lokalem Server ausführtbar)
.birthdayremove
.birthdayroleremove
.birthdaymessagechannel
.birthdaymessagechannelremove
.award
.take
.startevent
.setrolemoney
.resetdailymoney
.removerolemoney
.setrolemoneypriority
.addxp
.setxp
.msgxprestrictionadd
.msgxprestrictionremove
.setrolelevelbinding
.removerolelevelbinding
.savechat
.activity

Dazu hab ich aber mehrere Lösungsansetze:

  1. Man kann die Commands die ich oben genannt hat nur ausführen wenn man den Bot eingeladen hat
    oder die Rechte mit .executebotowneronlycommandsadd @user kurz .ebooca @user zugesprochen bekommen hat (von der Person die Bot owner des Servers ist (das ist die Person die ihn eingeladen hat)), falls man das Team Verlässt und nicht Länger die Commands ausführen soll kann man einem nutzer mit .executebotowneronlycommandsremove @user kurz .eboocr @user die Rechte wieder aberkennen
    Um sich Personen mit diesen Rechten Auflisten zu Lassen kann man .executebotowneronlycommandslist kurz .eboocl Benutzen
    Um die Server Bot owner Rechte (die Person die den Bot eingeladen hat) weierzugeben sollte man sie mit .transferownership @user kurz .to @user an den Entsprechenden User weitergeben können.

  2. Man kann die Commands ausführen wenn man die Rolle mit dem Namen Bot Owner hat (Zusätzlich zu der Permissions Rolle)

@expeehaa
Copy link
Collaborator

Danke für die Liste. Bis auf die Geburtstagsbefehle haben die die falschen Berechtigungen, das ändere ich.

Geburtstage sind aktuell global, was ein gewisses Problem darstellt. Das beste wäre vermutlich, die auch serverabhängig zu machen. Weiß aber nicht, wann ich dazu komme.

@expeehaa expeehaa changed the title Mehrere Owner und Owner Rechte Übertragen Fix permissions for bot-owner-only commands that should be executable by server admins. Aug 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants