Enhancement of timestamp logging

Geaaru geaaru at gmail.com
Fri Feb 24 15:05:56 CET 2017


Sorry, stupid web browser cache that doesn't refresh github pages and
unlucky and moreover I haven't received any notifications from github
about your commit.

I see only now your patch.

Can I send you a new pull request also on 3.1.x and 4.0.x with same
target?

Thank you for your support. Sorry again.

On Fri, 2017-02-24 at 08:13 -0500, Alan DeKok wrote:
> On Feb 24, 2017, at 5:09 AM, Geaaru <geaaru at gmail.com> wrote:
> > 
> > 
> > Yes, I know that code is public, I sent a pull request on github
> > (see
> > pull request #1917).
> 
>   You didn't check github and read the patch I put in?
> 
>   You didn't check the pull request to see the patch I put you?
> 
> > 
> > But FWIS on xlat.c file (on tree 3.0.x) or on xlat_eval.c (on tree
> > 4.0.x) there isn't a management of nanoseconds or milliseconds on
> > logging messages.
> > 
> > I don't find where could be currently used a timestamp with
> > milliseconds or nanosecods on logging.
> > 
> > FWIS currently logging timestamp with milliseconds or nanoseconds
> > is
> > not supported by freeradius. There is a '%T' option but
> > milliseconds is
> > set statically to 0000000.
> > 
> > I currently patched my freeradius code to permit a more efficient
> > timestamp logging, let me known if you want patch this on official
> > tree.
> 
>   If you go read github as I request, and read your pull request,
> you'll see that a fixed patch is already in.
> 
>   Alan DeKok.
> 


More information about the Freeradius-Devel mailing list