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

ip: fix case sensitivity of querying by nickname #1915

Merged
merged 1 commit into from
Aug 5, 2020

Conversation

ivar-it
Copy link
Contributor

@ivar-it ivar-it commented Aug 3, 2020

Description

Fixes problem with .ip not working when uppercase letters in queried nick.
The DB stores users in lowercase, so it does not get a match otherwise.

Ex. when quering user "GLaDER"(hiding successful query output because privacy)

Found in 7.0.4, running in docker.
Currently not linked to any issue.

| .ip GLaDER
| I'm not aware of this user.
| .ip glader
| [IP/Host Lookup] Hostname: XXXX | Location: XXX| Region: XXXX | City: XXXX | ISP: XXXXX

Checklist

  • I have read CONTRIBUTING.md
  • I can and do license this contribution under the EFLv2
  • No issues are reported by make qa (runs make quality and make test)
  • I have tested the functionality of the things this change touches

@ivar-it ivar-it changed the title Fixes problem with .ip not working when uppercase letters in queried nick. ip.py | Fixes problem with .ip not working when uppercase letters in queried nick. Aug 3, 2020
@dgw dgw changed the title ip.py | Fixes problem with .ip not working when uppercase letters in queried nick. ip: fix case sensitivity of querying by nickname Aug 3, 2020
Fixes problem with .ip not working when uppercase letters in nick.
The DB stores users in lowercase, so it does not get a match otherwise.
@dgw dgw added this to the 7.1.0 milestone Aug 3, 2020
@dgw dgw added the Bugfix Generally, PRs that reference (and fix) one or more issue(s) label Aug 3, 2020
Copy link
Member

@dgw dgw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks. We really should just make #1341 happen already.

@dgw dgw merged commit bd433f9 into sopel-irc:master Aug 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bugfix Generally, PRs that reference (and fix) one or more issue(s)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants