Bug 14212 - Add 'oldip' and 'newip' to different_ip message in auth.tt
Summary: Add 'oldip' and 'newip' to different_ip message in auth.tt
Status: NEW
Alias: None
Product: Koha
Classification: Unclassified
Component: Templates (show other bugs)
Version: unspecified
Hardware: All All
: P5 - low enhancement (vote)
Assignee: Owen Leonard
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-05-16 21:02 UTC by Barton Chittenden
Modified: 2023-08-05 10:52 UTC (History)
0 users

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Barton Chittenden 2015-05-16 21:02:12 UTC
When IP addresses are changing frequently, users will be logged out and the login screen will have the warning message "Error: IP address has changed. Please log in again"

This is confusing  to users, and it is often unclear to the users that this is a network issue, not a problem with with Koha. Furthermore, the library's IT staff may claim that the IP addresses have not changed, which puts the user in a very difficult position.

C4/Auth.pm defines the variables 'oldip' and 'newip' in addition to 'different_ip'.

I would suggest changing

[% IF ( different_ip ) %]
<div id="login_error"><strong>Error: </strong>IP address has changed, please log in again </div>
[% END %]

to

[% IF ( different_ip ) %]
<div id="login_error"><strong>Error: </strong>IP address has changed from '[% oldip %]' to '[% newip %]', please log in again </div>
[% END %]

This message gives users (and anyone supporting them) the evidence to back up the claim that the IP address has changed.
Comment 1 Katrin Fischer 2023-08-05 10:52:19 UTC
I could try and make this change, but is there a reason we might not want to show this information?