Tickets :: [#14478] Enhanced function of horde-active-sessions

6.0.0-alpha10

5/14/25

Summary	Enhanced function of horde-active-sessions
Queue	Horde Base
Queue Version	5.2.12
Type	Enhancement
State	Feedback
Priority	1. Low
Owners
Requester	frank.richter (at) hrz (dot) tu-chemnitz (dot) de
Created	09/23/2016 (3155 days ago)
Due
Updated	11/01/2016 (3116 days ago)
Assigned
Resolved
Milestone
Patch	Yes

11/01/2016 03:18:41 PM	frank (dot) richter (at) hrz (dot) tu-chemnitz (dot) de	Comment #7 New Attachment: horde-active-sessions[1]	Reply to this comment
[Show Quoted Text - 28 lines][Hide Quoted Text] - The -ll output should probably stay as-is, if people already rely on the output, scripting it in some way - -lll should be used instead for the more detailed information - -ll could still receive the optional user parameter. Ok. - You may want to switch to Horde_Argv because the parameter parsing is getting complicated: https://www.horde.org/libraries/Horde_Argv/docs/README In my first try it does'nt accept -ll exception 'Horde_Argv_OptionException' with message 'invalid long option string '-ll': must start with --, followed by non-dash' in /usr/share/pear/Horde/Argv/Option.php:323 Ah, okay. Then we probably have to stick with the current argument parsing, to maintain backward compatibility. - You should check the return value of $sessionHandler->destroy() Is it boolean? Or shall I use a try ... catch? It's a boolean. - You should only output the number of sessions with the -l* flags. Ok. Attached my next try. Frank

10/14/2016 02:41:42 PM	Jan Schneider	Comment #6	Reply to this comment
[Show Quoted Text - 15 lines][Hide Quoted Text] - The -ll output should probably stay as-is, if people already rely on the output, scripting it in some way - -lll should be used instead for the more detailed information - -ll could still receive the optional user parameter. Ok. - You may want to switch to Horde_Argv because the parameter parsing is getting complicated: https://www.horde.org/libraries/Horde_Argv/docs/README In my first try it does'nt accept -ll exception 'Horde_Argv_OptionException' with message 'invalid long option string '-ll': must start with --, followed by non-dash' in /usr/share/pear/Horde/Argv/Option.php:323 Ah, okay. Then we probably have to stick with the current argument parsing, to maintain backward compatibility. - You should check the return value of $sessionHandler->destroy() Is it boolean? Or shall I use a try ... catch? It's a boolean. - You should only output the number of sessions with the -l* flags. Ok. Frank

10/14/2016 01:59:26 PM	frank (dot) richter (at) hrz (dot) tu-chemnitz (dot) de	Comment #5	Reply to this comment
- The -ll output should probably stay as-is, if people already rely on the output, scripting it in some way - -lll should be used instead for the more detailed information - -ll could still receive the optional user parameter. Ok. - You may want to switch to Horde_Argv because the parameter parsing is getting complicated: https://www.horde.org/libraries/Horde_Argv/docs/README In my first try it does'nt accept -ll exception 'Horde_Argv_OptionException' with message 'invalid long option string '-ll': must start with --, followed by non-dash' in /usr/share/pear/Horde/Argv/Option.php:323 - You should check the return value of $sessionHandler->destroy() Is it boolean? Or shall I use a try ... catch? - You should only output the number of sessions with the -l* flags. Ok. Frank

10/12/2016 12:38:22 PM	Jan Schneider	Comment #4 State ⇒ Feedback	Reply to this comment
A few comments: - The -ll output should probably stay as-is, if people already rely on the output, scripting it in some way - -lll should be used instead for the more detailed information - -ll could still receive the optional user parameter. - You may want to switch to Horde_Argv because the parameter parsing is getting complicated: https://www.horde.org/libraries/Horde_Argv/docs/README - You should check the return value of $sessionHandler->destroy() - You should only output the number of sessions with the -l* flags.

09/23/2016 05:55:40 PM	ddangel0 (at) rosario (dot) gov (dot) ar	Comment #3	Reply to this comment
[Show Quoted Text - 13 lines][Hide Quoted Text] * extended information on -ll: user, timestamp, IP, Id, Browser * added -ll user to get information on this user * added -r user to remove a session I attached all the changed horde-active-sessions script. It works for me, pleae review. Regards, i tested with redis and only show me the list of all sessions. Frank forget this comment?IT WORKS !!!!! thanks men

09/23/2016 05:20:30 PM	ddangel0 (at) rosario (dot) gov (dot) ar	Comment #2	Reply to this comment
* extended information on -ll: user, timestamp, IP, Id, Browser * added -ll user to get information on this user * added -r user to remove a session I attached all the changed horde-active-sessions script. It works for me, pleae review. Regards, i tested with redis and only show me the list of all sessions. Frank

09/23/2016 01:19:16 PM	frank (dot) richter (at) hrz (dot) tu-chemnitz (dot) de	Comment #1 Priority ⇒ 1. Low Type ⇒ Enhancement Summary ⇒ Enhanced function of horde-active-sessions Queue ⇒ Horde Base Milestone ⇒ Patch ⇒ Yes New Attachment: horde-active-sessions State ⇒ New	Reply to this comment
* extended information on -ll: user, timestamp, IP, Id, Browser * added -ll user to get information on this user * added -r user to remove a session I attached all the changed horde-active-sessions script. It works for me, pleae review. Regards, Frank