Improper munging of IP addresses when used as IMAP or SMTP server address
Bug Summary
In server Settings, Geary attempts to munge IP addressses as though they were domain names. Additionally, the overall UX of this field is questionable, see below.
Your installation
Geary version: 40.0
Geary revision:
GTK version: 3.24.34
GLib version: 2.72.2
WebKitGTK version: 2.36.3
Desktop environment: GNOME
Distribution name: NixOS
Distribution release: 22.05 (Quokka)
Installation prefix: /nix/store/zj3ikd2j785gafgpjzk0bwmp8l1gdzz0-geary-40.0
Steps to reproduce
- From Geary homescreen, navigate as follows:
Hamburger menu -> Accounts -> -> 'Server Settings' button ->
- Set 'IMAP server' and/or 'SMTP server' to an IP address
Watch 'Service provider' field be updated to show 3/4 of an IPv4 address
- What did Geary do that was unexpected?
Instead of showing me the complete IP in the first text field, 'Service provider', it instead shows me three of the four integers that comprise an IPv4 address.
What did you expect to happen?
I would have expected it to either show me the whole IP.
While I'm too lazy to check the source, given this result I would expect the regex looks something like s/(\w+)((\.\w+){,3})/\2/
and probably works great on sites with names like mail.foo.bar
.
I imagine the intention was to give the user a sort of overview of what server you're talking about -- like if you're talking about imap.foo.bar and smtp.foo.bar, this must be the configuration page for email from foo.bar! Thanks for the help, but I'd suggest that if you can't make this feature work 100% of the time, you're going to leak user trust.
And, on a purely editorial note, the whole 'Service provider' field gives me UX badvibes, as it's uneditable (generated from the server names the user inputs), is not clearly demarcated from all the other fields (which are editable,) and, at least in Geary 40, is buggy. I do believe I spent a good 30s clicking on it and frowning over the rims of my glasses. Why can't I edit this field like the rest?
Then, when I realized it was a display label rather than an editable field, my default assumption was that Geary internally used a module that its devs call a 'service provider', and this module was at v69.208.95.