Skip to content
This repository was archived by the owner on May 10, 2025. It is now read-only.

Include top of the stacktrace in error embed for emblem command #208

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

FlaminSarge
Copy link

This is the only place that the exception is directly logged to clients with the intent for them to open an issue on GitHub, so may as well give them more information to submit.

The alternative to this would be:
Generate a GUID for the exception.
Log the exception + GUID.
Display the GUID to client in the ErrorEmbed, for them to log an issue on Discord/GitHub.

I do not know if the stack trace will contain any sensitive info from the bot host, hence the alternative proposal.

This is the only place that the exception is directly logged to clients with the intent for them to open an issue on GitHub, so may as well give them more information to submit.
@MoonieGZ
Copy link
Collaborator

MoonieGZ commented Jan 2, 2025

Hello, I greatly appreciate your contribution but this project is no longer in active development.

I forgot to make a note of this in the repo I apologize.

Will definitely be using this tip in future projects, thank you!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants