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

Using firmer language in the token removal message to the user #267

Merged
merged 1 commit into from Jan 11, 2019

Conversation

Projects
None yet
2 participants
@SebastiaanZ
Copy link
Member

SebastiaanZ commented Jan 11, 2019

As there seems to be someone self-botting on our server to steal API tokens and wreck servers, I feel it's better to change the removal message to something that emphasizes the need to change the token. The old message could give someone the idea that our bot deleting the message mitigated the danger of accidentally posting it.

New message in action:

Firmer recommendation to change token

@heavysaturn
Copy link
Member

heavysaturn left a comment

Excellent change. Approved.

@heavysaturn

This comment has been minimized.

Copy link
Member

heavysaturn commented Jan 11, 2019

Since this is a purely cosmetic change (for an urgent matter), I'm just gonna merge it.

@heavysaturn heavysaturn merged commit f00c324 into master Jan 11, 2019

@sco1 sco1 deleted the token-detection-warning-strong-language branch Jan 11, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment