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.

Sign up for GitHub

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

Jump to bottom

Gracefully handle NS cert add myself <fp> #2128

Merged
merged 2 commits into from
Feb 14, 2024
Merged

Gracefully handle NS cert add myself <fp> #2128

merged 2 commits into from
Feb 14, 2024

Commits on Feb 14, 2024

  1. Gracefully handle NS cert add myself <fp> 

    A non-operator with the nick "mynick" attempts to register
a fingerprint to their authenticated account.

They /msg NickServ cert add mynick <fingerprint>

NickServ responds with "Insufficient privileges" because
they've accidentally invoked the operator syntax (to action
other accounts).

This patch allows the user to add the fingerprint if the client's
account is identical to the target account.

Signed-off-by: Matt Hamilton <[email protected]>
    @Eriner
    Eriner committed Feb 14, 2024
    Configuration menu
    Copy the full SHA
    9d53d9f View commit details
    Browse the repository at this point in the history

  2. Update nickserv.go 

    Compare the case-normalized target to Account()
    @slingamn
    slingamn authored Feb 14, 2024
    Configuration menu
    Copy the full SHA
    136dd1a View commit details
    Browse the repository at this point in the history