scarletcafe / jishaku

A debugging and testing cog for discord.py rewrite bots.
https://jishaku.readthedocs.io/en/latest/
MIT License
541 stars 181 forks source link

NotOwner not raised when user isnt owner #188

Closed Motzumoto closed 2 years ago

Motzumoto commented 2 years ago

Summary

When someone who isnt the bot owner runs jsk it doesnt return a not owner error. It doesnt look to return anything

Reproduction steps

My error handler goes as follows: https://haste.powercord.dev/avulevowin (sentry error handling hidden due to privacy reasons(its at the very bottom of the error handler and i dont think its necessary to display it)) Get any account that isnt the bot owner and this is what gets returned: https://i.lunardev.group/9ppGRdY/direct

Expected results

I expect for it to return NotOwner; "You must own this bot to use JSK" or something along those lines. From what i remember it USED to do this.

Actual results

Returns nothing

Checklist

System information

I dont think this is an issue with my error handler as its just been rewritten, if you know how to fix it please let me know

scarletcafe commented 2 years ago

Nothing about this behavior has changed d.py or jishaku wise so this has to be some form of bug in your code somewhere. I can still easily see NotOwner raising matching your versions, so it's ultimately your job to figure out why your handler isn't picking it up.

image

I'd consider maybe looking into whether some of your other cases have MRO overlap with other exceptions (e.g. NotOwner is a CommandError, so handling CommandError first will shortcut the handling). You also abridged your code and it's possible the code you redacted could also be culprit somehow. (word of advice - you shouldn't really have it such that your code itself is sensitive - put your secrets elsewhere).

Regardless, pretty confident this isn't my bug so I'm closing this.

Motzumoto commented 2 years ago

You also abridged your code and it's possible the code you redacted could also be culprit somehow. (word of advice - you shouldn't really have it such that your code itself is sensitive - put your secrets elsewhere).

I hid the code because i believed it was unnecessary, its just a large function that gets called at an else at the very bottom of the file, so it shouldnt be causing any issues since it only gets called if all the conditionals dont get called