Fix the output report directory name

The reports were stored in a directory named 000000000-000000000 instead
of the correct date covered by the report.

Improve the message helping the user to set the translation up

It isn't easy to figure out where sarg searches for sarg.mo. Running sarg
like sarg -x -v display the usual version number and a message giving
enough information to know where to put the message file.

Display locale information for debugging purpose

Running sarg -x -v display the version number but also the directory where
the message file is looked for, the package name and the currently
selected locale.

Update the French translation.

Fetch the simplified Chinese translation from the TP

That translation has been released with sarg for a long time but it was
only accepted by the translation project today.

Add a comment about a puzzling piece of code

The trick to store the same string only once is not clear and deserves a
little comment.

Sarg.conf with the topuser_fields documentation

The default configuration file now describes the accepted columns for the
top user report.

Display a column with the user's IP address in the top user report

In sarg.conf, topuser_fields accept the new USERIP parameter to display
a new column with the user's IP address. It is only useful if the IP
address is replaced by a name or resolved to a host name.

Merge branch 'master' of ssh://git.code.sf.net/p/sarg/code

Add the Brazilian Portuguese translation

Many thanks to Marcio Costa for this work.

Fix a problem with log file names storage

Log file names are stored in a tree to save space. They have to be merged
back when the full file name is required. But an error prevented the file
name from being reconstructed if two files were stored in different
directories and one node of the tree has a lower depth than the next
one.

Merge the tiny charset name convertor function

There is no need to have a distinct source file for that tiny global
function when it can be stored in getconf.c and be static.

Split a long line.

Rewrite the index tree rebuilding

When index_tree is set to "date", the reports are stored in a tree based
on their content's date. The existing code was somewhat broken.

In addition, the directory size used to be computed and displayed in the
years index file. It is now computed and displayed in the months and days
indexes.

Close some open files in email.c

This is patch #38 submitted by T.M. Abraham. Many thanks.

Replace some more overlooked index.html constants

See commit 56b97279b03d9ff758528f6f2d6eeeff4cd3f09e for the first part of
this change.

Delete old reports when index_tree is set to "date"

Sarg goes to great length to make sure no unrelated directories are
deleted.

A list of every report directory is build. It is sorted in ascending order
on the last modification time. If too many directories are present, the
oldest ones are deleted including the possibly remaining empty parent
directory.

The index.html file is not rebuild to account for the deleted reports yet.

Re-enable the deletion of old reports

The old report deletion has been commented out for testing. It is now
reinstated.

Use a global constant instead of the "index.html" file name in the code

The name of the html index file is defined once in the source code as a
global constant instead of littering the source code with a hard coded
litteral string.

Display the period covered by the logs even if no records were found

Displaying the start and end dates found in the log helps in diagnosing
why the requested date was not found.

Verbose output with correct period covered by the logs

Previous versions reported the period covered by the log but it was, in
fact, the period extracted from the log limited to the requested range.

Now, two distinct messages are displayed in verbose mode. The first is the
period really covered by the log files. The second is the range taken into
account to produce the report.

Improve old reports deletion

Use the last modification time to sort the old reports directories. The
previously used ctime is not the creation time but the time at which the
directory meta data was last modified.

Creating the old report list don't use temporary files nor the external
sort command. Everything is kept and sorted in memory.

The code make sure the report directory looks like one of our report
directory. This is a safety to avoid deleting an unrelated directory.

Currently this safety mechanism only detects directories created with in
index_tree set to "file".

Merge branch 'master' of ssh://git.code.sf.net/p/sarg/code

Ensure the ChangeLog has been updated

I used to always forget to update the ChangeLog before releasing sarg. It
won't happen if the archive script check that the ChangeLog contains an
entry for the current version and release date.

Check the release date when archiving the project

The release date is in the cmake script and in the source code. The script
to package sarg now check that the two dates are identical.

Conflicts:
archive.sh

Archive script check for version number without cat

There is no need to check the version number with a cat. Sed can read the
file by itself.

Archive script with better error messages

When the version number isn't correct, the error message gives a clearer
indication about what is wrong.

Conflicts:
archive.sh

Explicitly link against the math library

This patch was suggested by Diego Elio Pettenò on May 25, 2011 and
implemented in commit b966a3a224de2a6ea591458bec47c1bb4e9b3745 but the
patch had not been applied to the CMake build file.

Fix _FORTIFY_SOURCE redefinition error

Some systems define _FORTIFY_SOURCE by default. It conflicts with our use
of that macro when --enable-extraprotection is passed to the configuration
script.

The solution is to undefine it before redefining it with value 2.

If a higher _FORTIFY_LEVEL level ever appears, it will have to be used in
this program to benefit from the highest protection possible.

Help in diagnosing the number of old reports to keep

The lastlog and lastlog1 files are stored in the temporary directory
instead of the output directory and they are not deleted when sarg exits
if the user asked not to delete temporary files.

Some debug messages give indication about what is happening with respect
to the old reports deletion.

Clean up sarg-php HTML and report more errors

The original HTML was the bare minimum and no error messages were printed.
This patch should fix it if only someone could test it.

Sarg.conf search exclude_codes in the default directory

The default sarg.conf was looking for exclude_codes at the wrong place.

Thanks to José Vicente Mondejar for reporting this bug.

May fix some errors in sarg-php

Sarg-php was in a sorry state but as Jose Vicente Mondejar kindly provided
a translation, I felt obliged to try to improve things a bit.

I can't test it on my computer. I hope I didn't broke it more than it was.

Fix a link error with libpcre

Libpcre was linked at the beginning of the link command. It was fine with
some compilers but not with others.

Libraries are supposed to be linked after every *.o file that use them or
some linker (gold?) reports errors about undefined references to functions.

Thanks to Micha Voss for reporting this problem and testing the solution.

Clang compatibility fix

Option to not fail on integers sign comparison must come after the extra
warnings have been enabled or it is ignored.

Report LDAP status at the end of the configuration script

As was already done with gd and pcre, the lack of ldap support is
reported at the end of the configuration script.

Fix a message text in configure.

Check that every required ldap header file is available

LDAP support is only enabled if every required header file is detected on
the target system.

Process correctly run together options

If sarg is ran with option -l combined with any other option such as -xl,
then the -l option is mistaken as --lastlog and does not produce the
expected result (which is to use the access.log file passed as argument).

It has been fixed by requiring that any long option is préfixed with a
double dash.

It breaks the compatibility with old versions but nobody should notice.

Thanks to Leonardo Rodrigues for reporting this bug.

Merge branch 'master' of ssh://git.code.sf.net/p/sarg/code

Fix several possible sprintf buffer overflows

At least one of those sprintf was producing a segfault when a very long
elapsed time was encountered.

Thanks to Ricardo Fregati for reporting this bug and Marc Cesarine for
solving it.

Apply a recommendation from gcc

Gcc suggested to put parentheses around an operator to clarify a if
statement.

Use a string to decide the content of the per_user_limit file

Instead of a numerical value 0/1, the strings "ip" or "id" are used in the
per_user_limit option to decide if the file contains the IP address or the
user's ID.

An aliased IP address is not an IP address any more

If the user's IP address is aliased, it must not be considered to be an
IP address to be resolved by the DNS.

Fix the aliasing of an IPv4 address

The comparison was not working properly and the line of code was a little
bit obfuscated.

Add an option to create the per_user_limit files

Sarg.conf allows the user to either always create a per_user_limit file
even if it is empty or only create it if some user crosses the limit.

Add a function to look a user up by IP address

The function is currently unused but I might have had some use for it so I
left it in the code.

Create multiple per_user_limit files

Each file can have a different limit.

For each file, it is possible to write the user's ID or the user's IP
address.

If the files exists, they are purged.

Don't generate the date/time report page if it isn't used

The temporary file may still be created but it will be deleted before sarg
exits.

As a consequence, if temporary files are kept, the temporary directory may
hold more files than were actually used.

Don't segfault if date_time_by is empty

Thanks to nuphero for reporting this bug and helping in tracking it down.

Remove bashisms from configure.in

Tests using == as equality operator are not portable. They are replaced by
a single =.

Thanks to Renato Botelho for reporting this bug.

Fix the reporting of downloaded files

The reporting of downloaded files was in fact reporting the denied accesses
since a previous changes in version 2.4.

Fix some compiler warnings

Unused variables.

Merge branch 'master' of ssh://git.code.sf.net/p/sarg/code

Conflicts:
html.c

Avoid rereading the user_limit file each time a user is added

Each time a user reach the download limit, he/she is added to the
user_limit file only once. Instead of rereading the whole file to see if
the user is already there, a flag is kept in memory to track already
limited users.

Compare correctly a per user limit greater than 2GB

A user limit greater than 2GB would sometime list the user into the
limit file even though the user does not exceed the limit.

Thanks to RodinM for reporting this bug.

Iconv converts the terminating NUL in the LDAP string

The LDAP string was not completely converted and was lacking the
terminating zero.

Apply patch 36 submitted by Dayel Zabin

The patch converts the string returned by a LDAP search into a user
selected character set.

The patch has been slightly modified to avoid memory leaks and report
errors the sarg's way.

Build alias.c with cmake

Update the build date

Don't force the use of libpcre

Libpcre was always included in alias.c to do some testing but it breaks
the compilation when libpcre isn't available. It has been removed.

Suppress warning when compiling with mingw

Mingw doesn't accept the __attribute__ option. It is removed when
compiling with that compiler.

Merge branch 'master' of ssh://git.code.sf.net/p/sarg/code

Report a proper message if a log file cannot be read

Previous versions used to report that the file doesn't exist if the user
doesn't have read access.

Thanks to Henry Lin for reporting this bug.

Report a proper message if a log file cannot be read

Previous versions used to report that the file doesn't exist if the user
doesn't have read access.

Thanks to Henri Lin for reporting this bug.

Make sarg as the default target

This bug was introduced after listing the source files dependencies.

Fix some messages from the configure script

Get rid of the hard coded echo instructions.

Add a user alias

It is similar to the hostalias file but is applied to the user ID.

Proper include dependencies in Makefile

The Makefile build the necessary source files when a header file is
changed.

Use constant strings for the IP address and user name read from the log file

Fix an error when processing NTLM user formatted name

An uninitialized variable was used to split the user name when the format
was declared as NTLM.

Update po files

Disambiguate the string "time" for the translators

The same word is used whether the value is a wall-clock time or a duration.
Translators may need to translate them with different words.

Rewrite the function to format numbers with a suffix KMGT

The function use less string manipulation functions for one or two bytes
strings. The bytes are copied one at a time. It makes the code more
readable.

Suffixes up to yotta are supported.

A comma is used when configured to to so with use_comma in sarg.conf.

Work around a bug in mingw's snprintf

snprintf doesn't limit the output string to the requested size (or at least
not strictly). The formatted numbers displayed in the report were wrong.

Test for an absolute path on Windows

The code to test for an absolute path on Windows was not compiled because
the constant to enable that code is not supported.

Install sarg.exe on Windows

Use the executable extension discovered by the configuration script when
installing sarg.

Revert to an old version of the Chinese translation

Several merge conflict tags has gone unnoticed. Since the po file hasn't
been modified for a long time, it is safe to revert to the old version.

Some characters had been damaged too. This version looks good from the
po point of view.

Remove merge tags from the po files

A merge conflict in the po files must have gone unnoticed and the conflict
tags were committed. They have now been removed.

Ugly patch to link on Windows

Library ws2_32 is required when compiling on Windows but the autoconf macro
to test for a library doesn't work due to the special declaration of the
functions in that library.

Link against an external libintl

Without this library, the linker complains about missing libintl_gettext
and the like if gettext is not part of libc.

Suppress warnings from the compiler for unused variables

Fix a warning about an uninitialized variable

Don't complain if pcre.h is not available

The file can be compiled even if pcre is not available.

Check for winsock.h

The header file is required to compile sarg on windows but its
availability was not checked in the configuration script.

Update program build date

Write the access times in one html page

Web sites access times were written in a file named after the user's name
and the visited site URL. There were one file per site, they were small and
numerous. In fact, they could be so numerous that they could fill the disk
inode table.

Moreover, with a file name made of the user's name and the visited site
URL, the manufactured file name could be more than 256 characters resulting
in an OS error on Windows and Linux.

To fix these two problems, the access times are all grouped inside one web
page with anchors to directly scroll to the relevant site. The html file
name is fixed (tt.html) and doesn't depend on the user's name or web
site URL.

Make sure the configure script is up to date when packaging the project

The configure script must exist and have been updated by autoreconf before
sarg can be released.

Packaging script accepts a tag with a translation suffix

When a translation is submitted after the version has been tagged, a new
tag is added with the language as a suffix. It must be accepted by the
packaging script.

Update the date in the header of every C file

Mangled file name is really unique

Due to an error, the file name created by mangling the user ID was not
using the suffix that was supposed to make it unique. Therefore, user's
names made of several consecutive non alphanumeric characters were not
unique.

The error manifested itself as an error message saying that the
_.user_unsort file could not be sorted. It occurred after the first
user assigned to the _ file name had been sorted and its file deleted.
Therefore any subsequent user sharing the _ file name would fail.

This patch also makes sure no original user name might possibly ends up
being the same as a mangled name.

Use less memory to store the user strings

Use the string buffer object to store the strings corresponding to the
users. It takes much less memory as the strings only take the exact amount
of memory instead of allocating a fixed size buffer as big as the biggest
expected string.

Don't ignore lines starting with a space in access.log

Sarg was ignoring any access.log line starting with a space character. I
see no reasons for this limitation. Therefore, I simply removed it.

Any restriction on the processed lines should be placed in the specific
format engine that requires it.

Explain why some input log lines are ignored

When verbose mode is enabled, a listing explaining how many lines have
been excluded is displayed. Every reason to ignore a line is listed. It
should make it easier to figure out why the report is not generated.

Make sure the total number of lines read is big enough

Use an unsigned long int to store the number of lines read from the access
log file.

Get rid of the limit on the number of input files

Previous versions were limited to 255 access log files with a maximum
length of 1024 characters. That limit has been alleviated.

Replace the "getword loop" messages

The "getword loop" messages are unclear. They don't explain what went
wrong.

The new messages explain that the end of a word or number was not found.

Fix a message string

Make a translator friendly message out of pieced together words

The "generated by" message written at the page bottom was made out of
words assembled together during the page generation. It was not possible
to translate that message.

Include Danish translation