Zimbra-Community / account-history

A Zimbra extension that allows the user to monitor their recent login and account activity. This is important to keep the user's account safe.
GNU General Public License v2.0
21 stars 2 forks source link

Remove sync.log from config.properties #24

Closed naisiew closed 4 years ago

naisiew commented 4 years ago

The installer will include audit.log and sync.log in /opt/zimbra/lib/ext/accountHistory/config.properties file . For speed, you can remove sync.log and the sync.log.yyyy-MM-dd.gz for better speed.

In my environment of around 1000 users, it takes about 120 sec to find all logs with sync.log and below 10 sec without sync.log.

barrydegraaff commented 4 years ago

users with zextras suite need the sync.log to see log-ins via ActiveSync.

I will see if it still needed.

On 23 Mar 2020, at 18:26, Naisiew Yeak notifications@github.com wrote:

The installer will include audit.log and sync.log in /opt/zimbra/lib/ext/accountHistory/config.properties file . For speed, you can remove sync.log and the sync.log.yyyy-MM-dd.gz for better speed.

In my environment of around 1000 users, it takes about 120 sec to find all logs with sync.log and below 10 sec without sync.log.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or unsubscribe.

naisiew commented 4 years ago

Oh didn’t realize that. My setup has Mobile NG in my 8.8.15 NE, which supposed to be same as Zextras mobile. I don’t see string “account=“ in sync.log. Is there special search string in sync.log?

barrydegraaff commented 4 years ago

no it used to be the same.

Does the log rotation work?

On 24 Mar 2020, at 02:36, Naisiew Yeak notifications@github.com wrote:

Oh didn’t realize that. My setup has Mobile NG in my 8.8.15 NE, which supposed to be same as Zextras mobile. I don’t see string “account=“ in sync.log. Is there special search string in sync.log?

— You are receiving this because you commented. Reply to this email directly, view it on GitHub, or unsubscribe.

naisiew commented 4 years ago

You mean does it read audit.log.yyyy-mm-dd.gz? Yes, as advertised. It reads today (audit.log) and yesterday's (audit.log.2020-03-23.gz) only.

barrydegraaff commented 4 years ago

I looked into it and ZSuite does now log correctly to audit.log. Which means we no longer need to parse sync.log.

The only thing sync.log provides on top of audit.log is an indication of the device type. Aka android or iphone. But IMHO that is little information for the performance hit.

I have released a new version that patches the installer/UI and readme.