#idempiere IRC log for Friday, 2018-10-12

*** nmicoud has joined #idempiere06:25
*** a42niem has joined #idempiere06:40
*** CarlosRuiz has joined #idempiere07:14
nmicoudHello CarlosRuiz ; could you have a look at https://groups.google.com/forum/?nomobile=true#!topic/idempiere/wPcDrDvNWTM please ? I'm not sure if this is a bug or not :-/07:23
CarlosRuizHi nmicoud07:24
CarlosRuizchecking07:24
nmicoudthanks07:24
CarlosRuizseems like the variable is in days - as the name intends07:36
CarlosRuizbut the program was developed thinking in hours07:36
nmicoudso the * 24 should be removed, right ?07:37
nmicoudAs it's an 'old' process, i fear to miss something07:37
CarlosRuizthe postgres part is also wrong07:39
nmicoudthat's consistent :)07:39
CarlosRuiz:)  not consistent - is wrong in another way  :D07:39
nmicoud:D07:39
CarlosRuiztesting here07:39
CarlosRuizif you have an interval of 4 days 11 hours 58 minutes07:40
CarlosRuizthe extract minutes returns 5807:40
CarlosRuizwhile the oracle version returns 647807:40
CarlosRuizthe sysconfig USER_LOCKING_MAX_ACCOUNT_LOCK_MINUTES is not explained07:43
*** a_ has joined #idempiere07:43
nmicoudAFAIR, is the time (in minute) where a user can't log after using wrong credentials07:44
CarlosRuizso, if I set in 60 - I cannot unlock the user until 1 hour passed - right?07:44
nmicoudyes07:44
CarlosRuizlooking Login.java - getClients - line 1304 and below07:48
CarlosRuizso - there is where the users are locked07:50
CarlosRuizwhen trying to login07:50
CarlosRuizthis is my interpretation:07:50
CarlosRuizsuppose you set USER_LOCKING_MAX_INACTIVE_PERIOD_DAY = 707:50
*** a_ has quit IRC07:50
CarlosRuizif you go to vacation for 10 days - your user is unlocked on your return07:51
CarlosRuizbut when you try to login the MAX_INACTIVE_PERIOD_DAY condition is reached and you are locked07:51
nmicoudright  so you should not go on vacation :p  or ask an admin to unlock you ?07:53
*** checked has joined #idempiere07:54
nmicoudin my case, it is set to 90 days07:55
CarlosRuizthen you call your administrator and he tries to unlock you but the process doesn't work because the program looks for hours (252) and compare with the days (10)07:55
nmicoudyep07:55
checkedhi07:55
nmicoudin my case, a user locks its account with wrong password 3 times.  it can be unlocked with process in the following hours. After a delay, this muse be done using sql07:56
nmicoud*must*07:56
CarlosRuizHi checked07:56
CarlosRuiznmicoud, but I wonder something08:08
CarlosRuizdatelastlogin is not updated when the user fails to login08:08
CarlosRuizso if we check for datelastlogin it won't work08:09
*** checked has quit IRC08:09
nmicoudwhy that ?08:10
CarlosRuizI need to do some tests08:15
CarlosRuizI have a suggested patch - do you want to try it in oracle?08:15
CarlosRuizand I test in postgres08:15
nmicoudsure08:15
CarlosRuizdid you open a JIRA?08:15
nmicoudnot yet, as i was not sure of the bug08:16
CarlosRuizok - can you please create one and I'll upload the suggested patch there08:17
nmicouddone08:17
nmicoud#380208:18
RecordingBotnmicoud: Error: "3802" is not a valid command.08:18
CarlosRuizuploaded a patch there - but I want to do some tests before - I think the process is maybe broken08:21
CarlosRuizlet me try the test with the user on vacations08:21
nmicoudsure08:21
nmicoudPerhaps write on the ticket what test were / should be conducted. To be sure it's ok on both oracle & postgres ?08:22
CarlosRuizah the admin can unlock one by one - but the part not working is the no-user08:28
nmicoudyes, when a single user is selected, there is no control of period/days08:29
nmicoudwhich is correct as he should know what he's doing08:30
CarlosRuizok - so that seems ok - users in vacations are not unlocked by the general process (user empty) - but can be unlocked one by one08:33
nmicoudyes08:33
nmicoudso your patch is correct then ?08:34
CarlosRuizyes - in postgres is working fine08:34
CarlosRuiztesting now the locked by wrong attempts08:35
nmicoudand is ok on oracle as that was what i tested yesterday08:35
CarlosRuizok - failed login attempts can be unlocked by the general reset locked with user empty08:37
CarlosRuiznow testing expired password08:37
CarlosRuizok - expired password doesn't lock the account - just force the change password panel08:44
CarlosRuizso - if working in oracle I'll commit it08:44
nmicoudis working also (i made lots of tests un may/june)08:45
CarlosRuizcommitted08:51
nmicoudthanks Carlos ; if you still have time, could you look at the #3801 ? (should be fast)08:51
*** Not-5c22 has joined #idempiere09:05
Not-5c22[iDempiere5.1] jenkins built #209 completed (success) http://ci.idempiere.org/job/iDempiere5.1/209/09:05
Not-5c22[iDempiere] CarlosRuiz_globalqss pushed 1 commit to release-5.1 [+0/-0/±1] https://bitbucket.org/idempiere/idempiere/commits/09:07
Not-5c22[iDempiere] nmicoud 356d53b - IDEMPIERE-3801 System user can't use saved queries09:07
Not-5c22[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-3801 status set to "Resolved" -assignee set to "Nicolas Micoud" -resolution set to "Fixed"09:07
Not-5c22[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-380109:07
nmicoudthanks09:07
CarlosRuizyw09:07
Not-5c22[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-3800 status set to "Resolved" -assignee set to "Diego Ruiz" -resolution set to "Fixed"09:26
Not-5c22[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-380009:26
Not-5c22[iDempiere] CarlosRuiz_globalqss pushed 1 commit to release-5.1 [+0/-0/±2] https://bitbucket.org/idempiere/idempiere/commits/09:26
Not-5c22[iDempiere] druiz 82be836 - IDEMPIERE-3800 Improvements to CSV Export - fix and integrate improvements from Heng Sin09:26
Not-5c22[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-2724 labels set to "Potential-Idea"09:30
Not-5c22[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-272409:30
Not-5c22[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-2724 status set to "Resolved" -resolution set to "Incomplete"09:30
Not-5c22[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-272409:30
Not-5c22[iDempiere5.1] jenkins built #210 completed (success) http://ci.idempiere.org/job/iDempiere5.1/210/09:34
Not-5c22[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-2725 status set to "Resolved" -resolution set to "Won't Fix"09:36
Not-5c22[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-272509:36
Not-5c22[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-2733 labels set to "Potential-Idea"09:39
Not-5c22[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-273309:40
Not-5c22[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-2733 status set to "Resolved" -resolution set to "Incomplete"09:40
Not-5c22[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-273309:40
Not-5c22[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-2734 status set to "Resolved" -resolution set to "Duplicate"09:42
Not-5c22[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-273409:42
*** Ayaz_ has joined #idempiere09:43
Not-5c22[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-2283 status set to "Resolved" -resolution set to "Fixed"09:45
Not-5c22[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-228309:45
*** Ayaz_ has quit IRC09:56
Not-5c22[iDempiere] CarlosRuiz_globalqss pushed 1 commit to default [+2/-0/±0] https://bitbucket.org/idempiere/idempiere/commits/10:02
Not-5c22[iDempiere] globalqss f049e62 - IDEMPIERE-2754 remove wrong test data.10:02
Not-5c22[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-2754 Fix Version set to "5.1z"10:02
Not-5c22[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-275410:02
Not-5c22[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-2754 status set to "Resolved" -assignee set to "Carlos Ruiz" -resolution set to "Fixed"10:02
Not-5c22[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-275410:02
Not-5c22[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-2793 status set to "Resolved" -assignee set to "Deepak Pansheriya" -resolution set to "Fixed"10:33
Not-5c22[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-279310:33
Not-5c22[iDempiere] CarlosRuiz_globalqss pushed 1 commit to default [+0/-0/±30] https://bitbucket.org/idempiere/idempiere/commits/10:33
Not-5c22[iDempiere] globalqss 5d80a8c - IDEMPIERE-2793 Extendability improvements / integrate patch from logilite (Deepak) - Changed private access modifier of members to 'Protected'10:33
Not-5c22[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-2793 Fix Version set to "5.1z"10:33
Not-5c22[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-279310:33
Not-5c22[iDempiere-mvn] jenkins built #41 completed (success) http://ci.idempiere.org/job/iDempiere-mvn/41/10:34
Not-5c22[iDempiere-mvn] jenkins built #42 completed (success) http://ci.idempiere.org/job/iDempiere-mvn/42/11:03
*** nmicoud_ has joined #idempiere12:25
*** nmicoud has quit IRC12:29
Not-5c22[IDEMPIERE] hieplq updated IDEMPIERE-2610 status set to "Closed" -resolution set to "Fixed"13:15
Not-5c22[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-261013:15
Not-5c22[IDEMPIERE] hieplq updated IDEMPIERE-3687 status set to "Closed" -resolution set to "Fixed"13:17
Not-5c22[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-368713:17
*** xapiens has quit IRC13:52
*** asim has joined #idempiere14:18
asimhello14:18
asimis any body here>?????14:19
*** asim has quit IRC14:20
*** nmicoud_ has quit IRC15:11
*** xapiens has joined #idempiere15:39
*** CarlosRuiz has quit IRC15:50
*** a42niem has quit IRC23:32

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!