naviserver-devel — Developer discussion

Re: [naviserver-devel] [naviserver-commits] naviserver ChangeLog, 1.615, 1.616

[Vlad S. <vl...@cr...>]

Excellent work Stephen!!

Stephen Deasey wrote:
> Update of /cvsroot/naviserver/naviserver
> In directory sc8-pr-cvs16.sourceforge.net:/tmp/cvs-serv25435
> 
> Modified Files:
> 	ChangeLog 
> Log Message:
> 	* nsd/connio.c:
> 	* tests/encoding.test: Use a more robust check for buffer full condition
> 	as Tcl may not fill it all the way when it complains of lack of space.
> 	Fixes bug reported by Neophytos Demetriou, Thanks.
> 
> 
> Index: ChangeLog
> ===================================================================
> RCS file: /cvsroot/naviserver/naviserver/ChangeLog,v
> retrieving revision 1.615
> retrieving revision 1.616
> diff -C2 -d -r1.615 -r1.616
> *** ChangeLog	24 May 2007 20:16:09 -0000	1.615
> --- ChangeLog	24 May 2007 20:23:24 -0000	1.616
> ***************
> *** 14,17 ****
> --- 14,22 ----
>   	* nsd/adpcmds.c: Remove tabs and re-indent.
>   
> + 	* nsd/connio.c:
> + 	* tests/encoding.test: Use a more robust check for buffer full condition
> + 	as Tcl may not fill it all the way when it complains of lack of space.
> + 	Fixes bug reported by Neophytos Demetriou, Thanks.
> + 
>   2007-05-23  Stephen Deasey  <sd...@us...>
>   
> 
> 
> -------------------------------------------------------------------------
> This SF.net email is sponsored by DB2 Express
> Download DB2 Express C - the FREE version of DB2 express and take
> control of your XML. No limits. Just data. Click to get it now.
> http://sourceforge.net/powerbar/db2/
> _______________________________________________
> naviserver-commits mailing list
> nav...@li...
> https://lists.sourceforge.net/lists/listinfo/naviserver-commits
> 

Re: [naviserver-devel] NaviServer IOBUFSZ

[Vlad S. <vl...@cr...>]

.adp with ns_apd_puts works, but .tcl with ns_return does not

telnet 0 8080
Trying 0.0.0.0...
Connected to 0.
Escape character is '^]'.
GET /a.tcl HTTP/1.0

HTTP/1.0 200 OK
MIME-Version: 1.0
Accept-Ranges: bytes
Date: Thu, 24 May 2007 01:27:55 GMT
Server: NaviServer/4.99.2
Content-Type: text/plain; charset=utf-8
Connection: close

Connection closed by foreign host.

Vlad Seryakov wrote:
> Just tested latest CVS version, it works with outputcharset just fine
> 
> Neophytos Demetriou wrote:
>> Try:
>>
>> ns_return 200 text/html [encoding system]-[string repeat X 10000]
>>
>> With:
>>
>> ns_param        hackcontenttype         true
>> ns_param        outputcharset           utf-8
>> ns_param        urlcharset              utf-8
>> ns_param  preferredcharsets             { utf-8 iso8859-1 }
>>
>> and then try the same thing by commenting out the outputcharset line.
>> Without outputcharset, it works. With outputcharset, it doesn't (data >
>> 8192).
>>
>> I've placed a logging statement just before Ns_ConnSetRequiredHeaders in
>> ReturnCharData (return.c):
>>
>> * Without outputcharset - ReturnCharData: len=10007 sendRaw=1 hlen=10007
>>
>> * With outputcharset - ReturnCharData: len=10007 sendRaw=0 hlen=10007
>>
>>> e.g., for a current project, we use the server the way Zoran said once
>>> on the list: use it out of the box and rely on the Unicode encoding.
>>> The strange thing is, that, when on the other hand explicitely
>>> verbosely setting the outputcharset (et.al) parameter to utf-8, it
>>> breaks.
>> True. The only reason I mention this here is because it works for
>> aolserver. I suppose there is a good reason that things are done
>> differently.
>>
>>
>> Best wishes,
>> Neophytos
>>
>>
>>
>>
>> Vlad Seryakov wrote:
>>> Can you provide more info how to reproduce it, i used test page with n > 
>>> 8192 and it works fine
>>>
>>> Neophytos Demetriou wrote:
>>>> Dear Vlad,
>>>> Dear Zoran,
>>>>
>>>> I came across some strange behavious when sendRaw is 0 (usually when you
>>>> specify an encoding in the configuration file, i.e. hackcontenttype
>>>> true, etc) I get a blank page instead of the actual data.
>>>>
>>>> I've experimented with [string repeat X $n] and I realized that this
>>>> happens whenever the size of the returned data  exceeds  IOBUFSZ (i.e.
>>>> $n > 8192).
>>>>
>>>> Maybe I'm missing something obvious here (*) but I assume there's some
>>>> kind of bug in Ns_ConnWriteVChars.
>>>>
>>>> Best wishes,
>>>> Neophytos
>>>>
>>>>
>>>> PPS. It's already 6 o' clock in the morning. I need to get some sleep
>>>> and I'll try to check it out again later in the afternoon.
>>>>
>>>> PS. I'm using a cvs checkout from 2007-05-19.
>>>>
>>
>> -------------------------------------------------------------------------
>> This SF.net email is sponsored by DB2 Express
>> Download DB2 Express C - the FREE version of DB2 express and take
>> control of your XML. No limits. Just data. Click to get it now.
>> http://sourceforge.net/powerbar/db2/
>> _______________________________________________
>> naviserver-devel mailing list
>> nav...@li...
>> https://lists.sourceforge.net/lists/listinfo/naviserver-devel
>>
> 

-- 
Vlad Seryakov
vl...@cr...
http://www.crystalballinc.com/vlad/

Re: [naviserver-devel] NaviServer IOBUFSZ

[Vlad S. <vl...@cr...>]

Just tested latest CVS version, it works with outputcharset just fine

Neophytos Demetriou wrote:
> Try:
> 
> ns_return 200 text/html [encoding system]-[string repeat X 10000]
> 
> With:
> 
> ns_param        hackcontenttype         true
> ns_param        outputcharset           utf-8
> ns_param        urlcharset              utf-8
> ns_param  preferredcharsets             { utf-8 iso8859-1 }
> 
> and then try the same thing by commenting out the outputcharset line.
> Without outputcharset, it works. With outputcharset, it doesn't (data >
> 8192).
> 
> I've placed a logging statement just before Ns_ConnSetRequiredHeaders in
> ReturnCharData (return.c):
> 
> * Without outputcharset - ReturnCharData: len=10007 sendRaw=1 hlen=10007
> 
> * With outputcharset - ReturnCharData: len=10007 sendRaw=0 hlen=10007
> 
>> e.g., for a current project, we use the server the way Zoran said once
>> on the list: use it out of the box and rely on the Unicode encoding.
>> The strange thing is, that, when on the other hand explicitely
>> verbosely setting the outputcharset (et.al) parameter to utf-8, it
>> breaks.
> 
> True. The only reason I mention this here is because it works for
> aolserver. I suppose there is a good reason that things are done
> differently.
> 
> 
> Best wishes,
> Neophytos
> 
> 
> 
> 
> Vlad Seryakov wrote:
>> Can you provide more info how to reproduce it, i used test page with n > 
>> 8192 and it works fine
>>
>> Neophytos Demetriou wrote:
>>> Dear Vlad,
>>> Dear Zoran,
>>>
>>> I came across some strange behavious when sendRaw is 0 (usually when you
>>> specify an encoding in the configuration file, i.e. hackcontenttype
>>> true, etc) I get a blank page instead of the actual data.
>>>
>>> I've experimented with [string repeat X $n] and I realized that this
>>> happens whenever the size of the returned data  exceeds  IOBUFSZ (i.e.
>>> $n > 8192).
>>>
>>> Maybe I'm missing something obvious here (*) but I assume there's some
>>> kind of bug in Ns_ConnWriteVChars.
>>>
>>> Best wishes,
>>> Neophytos
>>>
>>>
>>> PPS. It's already 6 o' clock in the morning. I need to get some sleep
>>> and I'll try to check it out again later in the afternoon.
>>>
>>> PS. I'm using a cvs checkout from 2007-05-19.
>>>
> 
> 
> -------------------------------------------------------------------------
> This SF.net email is sponsored by DB2 Express
> Download DB2 Express C - the FREE version of DB2 express and take
> control of your XML. No limits. Just data. Click to get it now.
> http://sourceforge.net/powerbar/db2/
> _______________________________________________
> naviserver-devel mailing list
> nav...@li...
> https://lists.sourceforge.net/lists/listinfo/naviserver-devel
> 

-- 
Vlad Seryakov
vl...@cr...
http://www.crystalballinc.com/vlad/

Re: [naviserver-devel] [naviserver-commits] modules/nsdbpg Makefile, 1.3, 1.4

[Vlad S. <vl...@cr...>]

Yes, no harm here but it is just more clear how those variables are 
assigned and what are defaults.

Stephen Deasey wrote:
> On 5/23/07, Vlad Seryakov <ser...@us...> wrote:
>> Log Message:
>> Updateed makefiles to conditionally set NAVISERVER env variable
>>
>> ! NAVISERVER = /usr/local/ns
>>
>>   MODNAME    = nsdbpg
>> --- 30,36 ----
>>   #
>>
>> ! ifndef NAVISERVER
>> !     NAVISERVER  = /usr/local/ns
>> ! endif
> 
> These are make variables. You override them on the command line like this:
> 
>   make NAVISERVER=/this/overrides/usr/local/ns
> 
> -------------------------------------------------------------------------
> This SF.net email is sponsored by DB2 Express
> Download DB2 Express C - the FREE version of DB2 express and take
> control of your XML. No limits. Just data. Click to get it now.
> http://sourceforge.net/powerbar/db2/
> _______________________________________________
> naviserver-devel mailing list
> nav...@li...
> https://lists.sourceforge.net/lists/listinfo/naviserver-devel
> 
*

Re: [naviserver-devel] NaviServer IOBUFSZ

[Zoran V. <zv...@ar...>]

Am 23.05.2007 um 11:36 schrieb Neophytos Demetriou:

> PS. Please note that I had to disable nstrace (i.e. I'm using init.tcl
> from 4.99.1 and removed the nstrace file) but I don't see how the two
> can be related.

They are not related at all.

BTW: I plan (soon) to rewrite that problematic part (interp inits)
to be entirely based on traces instead of introspection as this is
much cleaner and easier to maintain and understand. The ttrace I will
leave inside for some special uses.

Cheers
Zoran

Re: [naviserver-devel] NaviServer IOBUFSZ

[Neophytos D. <k2...@ph...>]

Please also note that it returns 0 bytes:
    [ <=>         ] 0             --.--K/s

09:34:27 (0.00 B/s) - `index.html' saved [0]

- Neophytos

Neophytos Demetriou wrote:
> Hi Stephen,
> 
>> Just to be clear though for this particular case: when outputcharset
>> is not set in the config file it works, and when it is it doesn't --
>> do you mean just that your browser shows a blank page or have you
>> observed something more specific, like an incorrect Content-Length
>> header, etc.?
> 
> Both :)
> 
> Here's the output of a "wget -S" when outputcharset is not set in the
> config file:
> 
>   HTTP/1.0 200 OK
>   Set-Cookie: xxxxxxxxxxxxxx
>   MIME-Version: 1.0
>   Accept-Ranges: bytes
>   Date: Wed, 23 May 2007 09:34:32 GMT
>   Server: NaviServer/4.99.2
>   Content-Type: text/html; charset=utf-8
>   Connection: close
>   Length: unspecified [text/html]
> 
> Best wishes,
> Neophytos
> 
> PS. Please note that I had to disable nstrace (i.e. I'm using init.tcl
> from 4.99.1 and removed the nstrace file) but I don't see how the two
> can be related.
> 
> -------------------------------------------------------------------------
> This SF.net email is sponsored by DB2 Express
> Download DB2 Express C - the FREE version of DB2 express and take
> control of your XML. No limits. Just data. Click to get it now.
> http://sourceforge.net/powerbar/db2/
> _______________________________________________
> naviserver-devel mailing list
> nav...@li...
> https://lists.sourceforge.net/lists/listinfo/naviserver-devel

Re: [naviserver-devel] NaviServer IOBUFSZ

[Neophytos D. <k2...@ph...>]

Hi Stephen,

> Just to be clear though for this particular case: when outputcharset
> is not set in the config file it works, and when it is it doesn't --
> do you mean just that your browser shows a blank page or have you
> observed something more specific, like an incorrect Content-Length
> header, etc.?

Both :)

Here's the output of a "wget -S" when outputcharset is not set in the
config file:

  HTTP/1.0 200 OK
  Set-Cookie: xxxxxxxxxxxxxx
  MIME-Version: 1.0
  Accept-Ranges: bytes
  Date: Wed, 23 May 2007 09:34:32 GMT
  Server: NaviServer/4.99.2
  Content-Type: text/html; charset=utf-8
  Connection: close
  Length: unspecified [text/html]

Best wishes,
Neophytos

PS. Please note that I had to disable nstrace (i.e. I'm using init.tcl
from 4.99.1 and removed the nstrace file) but I don't see how the two
can be related.

Re: [naviserver-devel] NaviServer IOBUFSZ

[Stephen D. <sd...@gm...>]

On 5/23/07, Neophytos Demetriou <k2...@ph...> wrote:
> Try:
>
> ns_return 200 text/html [encoding system]-[string repeat X 10000]
>
> With:
>
> ns_param        hackcontenttype         true
> ns_param        outputcharset           utf-8
> ns_param        urlcharset              utf-8
> ns_param  preferredcharsets             { utf-8 iso8859-1 }
>
> and then try the same thing by commenting out the outputcharset line.
> Without outputcharset, it works. With outputcharset, it doesn't (data >
> 8192).
>
> I've placed a logging statement just before Ns_ConnSetRequiredHeaders in
> ReturnCharData (return.c):
>
> * Without outputcharset - ReturnCharData: len=10007 sendRaw=1 hlen=10007
>
> * With outputcharset - ReturnCharData: len=10007 sendRaw=0 hlen=10007
>
> > e.g., for a current project, we use the server the way Zoran said once
> > on the list: use it out of the box and rely on the Unicode encoding.
> > The strange thing is, that, when on the other hand explicitely
> > verbosely setting the outputcharset (et.al) parameter to utf-8, it
> > breaks.
>
> True. The only reason I mention this here is because it works for
> aolserver. I suppose there is a good reason that things are done
> differently.


Actually no, this is probably broken...

Just to be clear though for this particular case: when outputcharset
is not set in the config file it works, and when it is it doesn't --
do you mean just that your browser shows a blank page or have you
observed something more specific, like an incorrect Content-Length
header, etc.?

I'm taking a look at this. Thanks for reporting.

Re: [naviserver-devel] NaviServer IOBUFSZ

[Neophytos D. <k2...@ph...>]

Try:

ns_return 200 text/html [encoding system]-[string repeat X 10000]

With:

ns_param        hackcontenttype         true
ns_param        outputcharset           utf-8
ns_param        urlcharset              utf-8
ns_param  preferredcharsets             { utf-8 iso8859-1 }

and then try the same thing by commenting out the outputcharset line.
Without outputcharset, it works. With outputcharset, it doesn't (data >
8192).

I've placed a logging statement just before Ns_ConnSetRequiredHeaders in
ReturnCharData (return.c):

* Without outputcharset - ReturnCharData: len=10007 sendRaw=1 hlen=10007

* With outputcharset - ReturnCharData: len=10007 sendRaw=0 hlen=10007

> e.g., for a current project, we use the server the way Zoran said once
> on the list: use it out of the box and rely on the Unicode encoding.
> The strange thing is, that, when on the other hand explicitely
> verbosely setting the outputcharset (et.al) parameter to utf-8, it
> breaks.

True. The only reason I mention this here is because it works for
aolserver. I suppose there is a good reason that things are done
differently.


Best wishes,
Neophytos




Vlad Seryakov wrote:
> Can you provide more info how to reproduce it, i used test page with n > 
> 8192 and it works fine
> 
> Neophytos Demetriou wrote:
>> Dear Vlad,
>> Dear Zoran,
>>
>> I came across some strange behavious when sendRaw is 0 (usually when you
>> specify an encoding in the configuration file, i.e. hackcontenttype
>> true, etc) I get a blank page instead of the actual data.
>>
>> I've experimented with [string repeat X $n] and I realized that this
>> happens whenever the size of the returned data  exceeds  IOBUFSZ (i.e.
>> $n > 8192).
>>
>> Maybe I'm missing something obvious here (*) but I assume there's some
>> kind of bug in Ns_ConnWriteVChars.
>>
>> Best wishes,
>> Neophytos
>>
>>
>> PPS. It's already 6 o' clock in the morning. I need to get some sleep
>> and I'll try to check it out again later in the afternoon.
>>
>> PS. I'm using a cvs checkout from 2007-05-19.
>>
> 

Re: [naviserver-devel] NaviServer IOBUFSZ

[Bernd E. <eid...@we...>]

Hi,

isn't that still an open wound, the organisation of the encoding part in 
general?

e.g., for a current project, we use the server the way Zoran said once on the 
list: use it out of the box and rely on the Unicode encoding. The strange 
thing is, that, when on the other hand explicitely verbosely setting the 
outputcharset (et.al) parameter to utf-8, it breaks.

Out of the blue, the mentioned Ns_ConnWriteVChars thing is in the context of 
the scatter-gather-interface... - Stephen?

> Can you provide more info how to reproduce it, i used test page with n >
> 8192 and it works fine
>
> Neophytos Demetriou wrote:
> > Dear Vlad,
> > Dear Zoran,
> >
> > I came across some strange behavious when sendRaw is 0 (usually when you
> > specify an encoding in the configuration file, i.e. hackcontenttype
> > true, etc) I get a blank page instead of the actual data.
> >
> > I've experimented with [string repeat X $n] and I realized that this
> > happens whenever the size of the returned data  exceeds  IOBUFSZ (i.e.
> > $n > 8192).
> >
> > Maybe I'm missing something obvious here (*) but I assume there's some
> > kind of bug in Ns_ConnWriteVChars.
> >
> > Best wishes,
> > Neophytos
> >
> >
> > PPS. It's already 6 o' clock in the morning. I need to get some sleep
> > and I'll try to check it out again later in the afternoon.
> >
> > PS. I'm using a cvs checkout from 2007-05-19.

Re: [naviserver-devel] [naviserver-commits] modules/nsdbpg Makefile, 1.3, 1.4

[Stephen D. <sd...@gm...>]

On 5/23/07, Vlad Seryakov <ser...@us...> wrote:
>
> Log Message:
> Updateed makefiles to conditionally set NAVISERVER env variable
>
> ! NAVISERVER = /usr/local/ns
>
>   MODNAME    = nsdbpg
> --- 30,36 ----
>   #
>
> ! ifndef NAVISERVER
> !     NAVISERVER  = /usr/local/ns
> ! endif

These are make variables. You override them on the command line like this:

  make NAVISERVER=/this/overrides/usr/local/ns

Re: [naviserver-devel] NaviServer IOBUFSZ

[Vlad S. <vl...@cr...>]

Can you provide more info how to reproduce it, i used test page with n > 
8192 and it works fine

Neophytos Demetriou wrote:
> Dear Vlad,
> Dear Zoran,
> 
> I came across some strange behavious when sendRaw is 0 (usually when you
> specify an encoding in the configuration file, i.e. hackcontenttype
> true, etc) I get a blank page instead of the actual data.
> 
> I've experimented with [string repeat X $n] and I realized that this
> happens whenever the size of the returned data  exceeds  IOBUFSZ (i.e.
> $n > 8192).
> 
> Maybe I'm missing something obvious here (*) but I assume there's some
> kind of bug in Ns_ConnWriteVChars.
> 
> Best wishes,
> Neophytos
> 
> 
> PPS. It's already 6 o' clock in the morning. I need to get some sleep
> and I'll try to check it out again later in the afternoon.
> 
> PS. I'm using a cvs checkout from 2007-05-19.
> 

-- 
Vlad Seryakov
vl...@cr...
http://www.crystalballinc.com/vlad/

Re: [naviserver-devel] Release

[F. <kul...@se...>]

On Mon, 21 May 2007 10:59:09 -0400
Mike <nee...@gm...> wrote:

> On 5/21/07, Bernd Eidenschink <eid...@we...> wrote:
> > > Just (dist-)compiled on an Ubuntu 7.04, tomorrow I'll give it a try on a
> > > SuSE Enterprise Linux (all 32bit).
> >
> > tomorrow never dies...
> > I just dist-compiled on the SuSE Enterprise and it was all smooth jazz.
> 
> Anyone test on a FreeBSD box yet?  The last time Zoran helped me try,
> there were a couple of hickups, but we did resolve them.  Just want to
> make sure the changes made it back into the main tree.  If someone
> would like access to a FreeBSD machine, just send me a mail, and I'll
> arrange it.

On NetBSD I test it. It works like perfect, with WebDAV support. I wil test the compilations method on FreeBSD today.

Feri

Re: [naviserver-devel] Release

[Mike <nee...@gm...>]

On 5/21/07, Bernd Eidenschink <eid...@we...> wrote:
> > Just (dist-)compiled on an Ubuntu 7.04, tomorrow I'll give it a try on a
> > SuSE Enterprise Linux (all 32bit).
>
> tomorrow never dies...
> I just dist-compiled on the SuSE Enterprise and it was all smooth jazz.

Anyone test on a FreeBSD box yet?  The last time Zoran helped me try,
there were a couple of hickups, but we did resolve them.  Just want to
make sure the changes made it back into the main tree.  If someone
would like access to a FreeBSD machine, just send me a mail, and I'll
arrange it.

Re: [naviserver-devel] Release

[Bernd E. <eid...@we...>]

> Just (dist-)compiled on an Ubuntu 7.04, tomorrow I'll give it a try on a
> SuSE Enterprise Linux (all 32bit).

tomorrow never dies... 
I just dist-compiled on the SuSE Enterprise and it was all smooth jazz.

Bernd.

Re: [naviserver-devel] [naviserver-commits] naviserver ChangeLog, 1.593, 1.594

[Zoran V. <zv...@ar...>]

Am 21.05.2007 um 12:12 schrieb Stephen Deasey:

>
> Because [ns_logctl severity ?] is a Tcl API and Ns_LogLevel() is a  
> C one?

Ah! I thought this was the Tcl usage-case. So you actually extended
the C-API to achieve what [ns_logctl severity ?] is doing. Allright.
Then all is clear.

Cheers
Zoran

Re: [naviserver-devel] [naviserver-commits] naviserver ChangeLog, 1.593, 1.594

[Stephen D. <sd...@gm...>]

On 5/21/07, Zoran Vasiljevic <zv...@ar...> wrote:
>
> Am 21.05.2007 um 11:59 schrieb Stephen Deasey:
>
> > On 5/21/07, Zoran Vasiljevic <zv...@ar...> wrote:
> >>
> >> Am 20.05.2007 um 01:50 schrieb Stephen Deasey:
> >>
> >>> + 2007-05-20  Stephen Deasey  <sd...@us...>
> >>> +
> >>> +     * include/ns.h:
> >>> +     * nsd/log.c (Ns_LogLevel): Add the Ns_LogLevel() function
> >>> which
> >>> returns
> >>> +     true if the given log severity level is enabled. Sometimes the
> >>> message
> >>> +     you need to log is expensive to construct, so you'd like to
> >>> construct it
> >>> +     only if it's needed.
> >>> +
> >>
> >> Doesn't
> >>
> >>     ns_logctl severity level
> >>
> >> return what you need?
> >>
> >> Welcome to lexxsrv running at /usr/local/aw/bin/nsd (pid 631)
> >> NaviServer/4.99.2 for osx built on May  5 2007 at 12:16:43
> >> CVS Tag: $Name:  $
> >> lexxsrv:nscp 1> ns_logctl severity notice
> >> 1
> >> lexxsrv:nscp 2> ns_logctl severity debug
> >> 0
> >> lexxsrv:nscp 3>
> >
> >
> > Yes.
> >
> > So you might do:
> >
> > if {[ns_logctl severity debug]} {
> >     ns_log debug "check yourself: [before_you_wreck_yourself
> > $a_huh_huh_huh_huh]"
> > }
> >
> > Otherwise the expensive function will be substituted into the log
> > message regardless of whether it actually gets logged or not. You
> > wouldn't use it all the time.
> >
> > Ns_LogLevel() is the equivalent for C code.
> >
> > (Should be called Ns_LogSeverity() ?)
>
> I do not understand. What is the purpose of your newly implemented
> feature/call when [ns_logctl severity] does already what you wanted?


Because [ns_logctl severity ?] is a Tcl API and Ns_LogLevel() is a C one?

So the above example would become:

if (Ns_LogLevel(Debug) {
    Ns_Log(Debug, "check yourself: %s", BeforeYouWreckYourself(...));
}

Saving a call to BeforeYouWreckYourself() in the (common) case where
Debug level logging is not enabled.

Re: [naviserver-devel] [naviserver-commits] naviserver ChangeLog, 1.593, 1.594

[Zoran V. <zv...@ar...>]

Am 21.05.2007 um 11:59 schrieb Stephen Deasey:

> On 5/21/07, Zoran Vasiljevic <zv...@ar...> wrote:
>>
>> Am 20.05.2007 um 01:50 schrieb Stephen Deasey:
>>
>>> + 2007-05-20  Stephen Deasey  <sd...@us...>
>>> +
>>> +     * include/ns.h:
>>> +     * nsd/log.c (Ns_LogLevel): Add the Ns_LogLevel() function  
>>> which
>>> returns
>>> +     true if the given log severity level is enabled. Sometimes the
>>> message
>>> +     you need to log is expensive to construct, so you'd like to
>>> construct it
>>> +     only if it's needed.
>>> +
>>
>> Doesn't
>>
>>     ns_logctl severity level
>>
>> return what you need?
>>
>> Welcome to lexxsrv running at /usr/local/aw/bin/nsd (pid 631)
>> NaviServer/4.99.2 for osx built on May  5 2007 at 12:16:43
>> CVS Tag: $Name:  $
>> lexxsrv:nscp 1> ns_logctl severity notice
>> 1
>> lexxsrv:nscp 2> ns_logctl severity debug
>> 0
>> lexxsrv:nscp 3>
>
>
> Yes.
>
> So you might do:
>
> if {[ns_logctl severity debug]} {
>     ns_log debug "check yourself: [before_you_wreck_yourself
> $a_huh_huh_huh_huh]"
> }
>
> Otherwise the expensive function will be substituted into the log
> message regardless of whether it actually gets logged or not. You
> wouldn't use it all the time.
>
> Ns_LogLevel() is the equivalent for C code.
>
> (Should be called Ns_LogSeverity() ?)

I do not understand. What is the purpose of your newly implemented
feature/call when [ns_logctl severity] does already what you wanted?
Can you give a simple usage example?

Cheers
Zoran 

Re: [naviserver-devel] [naviserver-commits] naviserver ChangeLog, 1.593, 1.594

[Stephen D. <sd...@gm...>]

On 5/21/07, Zoran Vasiljevic <zv...@ar...> wrote:
>
> Am 20.05.2007 um 01:50 schrieb Stephen Deasey:
>
> > + 2007-05-20  Stephen Deasey  <sd...@us...>
> > +
> > +     * include/ns.h:
> > +     * nsd/log.c (Ns_LogLevel): Add the Ns_LogLevel() function which
> > returns
> > +     true if the given log severity level is enabled. Sometimes the
> > message
> > +     you need to log is expensive to construct, so you'd like to
> > construct it
> > +     only if it's needed.
> > +
>
> Doesn't
>
>     ns_logctl severity level
>
> return what you need?
>
> Welcome to lexxsrv running at /usr/local/aw/bin/nsd (pid 631)
> NaviServer/4.99.2 for osx built on May  5 2007 at 12:16:43
> CVS Tag: $Name:  $
> lexxsrv:nscp 1> ns_logctl severity notice
> 1
> lexxsrv:nscp 2> ns_logctl severity debug
> 0
> lexxsrv:nscp 3>


Yes.

So you might do:

if {[ns_logctl severity debug]} {
    ns_log debug "check yourself: [before_you_wreck_yourself
$a_huh_huh_huh_huh]"
}

Otherwise the expensive function will be substituted into the log
message regardless of whether it actually gets logged or not. You
wouldn't use it all the time.

Ns_LogLevel() is the equivalent for C code.

(Should be called Ns_LogSeverity() ?)

Re: [naviserver-devel] [naviserver-commits] naviserver ChangeLog, 1.593, 1.594

[Zoran V. <zv...@ar...>]

Am 20.05.2007 um 01:50 schrieb Stephen Deasey:

> + 2007-05-20  Stephen Deasey  <sd...@us...>
> +
> + 	* include/ns.h:
> + 	* nsd/log.c (Ns_LogLevel): Add the Ns_LogLevel() function which  
> returns
> + 	true if the given log severity level is enabled. Sometimes the  
> message
> + 	you need to log is expensive to construct, so you'd like to  
> construct it
> + 	only if it's needed.
> +

Doesn't

    ns_logctl severity level

return what you need?

Welcome to lexxsrv running at /usr/local/aw/bin/nsd (pid 631)
NaviServer/4.99.2 for osx built on May  5 2007 at 12:16:43
CVS Tag: $Name:  $
lexxsrv:nscp 1> ns_logctl severity notice
1
lexxsrv:nscp 2> ns_logctl severity debug
0
lexxsrv:nscp 3>

Re: [naviserver-devel] [naviserver-commits] naviserver/nsd returnresp.c, 1.2, 1.3

[Stephen D. <sd...@gm...>]

Yeah, I'm just saying that in general we need flexible header
handling. Printf conditional add and printf replace are actually
probably the most useful. We should probably add it..


On 5/20/07, Vlad Seryakov <vl...@cr...> wrote:
> The idea here i may not explained that i am going to include Digest auth
> handler and in that handler it will assign www-authenticate header,
> because in case of Digest it is much more complicated. So using standard
> response handler it will check, if no header it will assume Basic
> authentication model, if exists, then it will use whatever supplied
>
> Stephen Deasey wrote:
> > Hmm, this is a bit like this:
> >
> > http://sourceforge.net/mailarchive/forum.php?thread_name=a9250c0b0702221615x5c0ab5e8gf8983c7c912a7520%40mail.gmail.com&forum_name=naviserver-devel
> >
> > Maybe we need the full set of add, conditonaly add and replace for the
> > printf version?
> >
> >
> > On 5/20/07, Vlad Seryakov <ser...@us...> wrote:
> >> Update of /cvsroot/naviserver/naviserver/nsd
> >> In directory sc8-pr-cvs16:/tmp/cvs-serv11302/nsd
> >>
> >> Modified Files:
> >>         returnresp.c
> >> Log Message:
> >>         * nsd/returnresp.c: In Ns_ConnReturnUnauthorized check if header
> >>         WWW-Authenticate already exists, this is in anticipation of new
> >>         Digest auth method to be implemented soon.
> >>
> >>
> >>
> >> Index: returnresp.c
> >> ===================================================================
> >> RCS file: /cvsroot/naviserver/naviserver/nsd/returnresp.c,v
> >> retrieving revision 1.2
> >> retrieving revision 1.3
> >> diff -C2 -d -r1.2 -r1.3
> >> *** returnresp.c        16 May 2007 16:04:09 -0000      1.2
> >> --- returnresp.c        20 May 2007 00:24:29 -0000      1.3
> >> ***************
> >> *** 331,340 ****
> >>       int         result;
> >>
> >> !     Ns_DStringInit(&ds);
> >> !     Ns_DStringVarAppend(&ds, "Basic realm=\"",
> >> !                         connPtr->servPtr->opts.realm, "\"", NULL);
> >> !     Ns_ConnSetHeaders(conn, "WWW-Authenticate", ds.string);
> >> !     Ns_DStringFree(&ds);
> >> !
> >>       if (ReturnRedirect(conn, 401, &result)) {
> >>           return result;
> >> --- 331,341 ----
> >>       int         result;
> >>
> >> !     if (Ns_SetIGet(conn->outputheaders, "WWW-Authenticate") == NULL) {
> >> !         Ns_DStringInit(&ds);
> >> !         Ns_DStringVarAppend(&ds, "Basic realm=\"",
> >> !                             connPtr->servPtr->opts.realm, "\"", NULL);
> >> !         Ns_ConnSetHeaders(conn, "WWW-Authenticate", ds.string);
> >> !         Ns_DStringFree(&ds);
> >> !     }
> >>       if (ReturnRedirect(conn, 401, &result)) {
> >>           return result;
> >>
> >>
> >> -------------------------------------------------------------------------
> >> This SF.net email is sponsored by DB2 Express
> >> Download DB2 Express C - the FREE version of DB2 express and take
> >> control of your XML. No limits. Just data. Click to get it now.
> >> http://sourceforge.net/powerbar/db2/
> >> _______________________________________________
> >> naviserver-commits mailing list
> >> nav...@li...
> >> https://lists.sourceforge.net/lists/listinfo/naviserver-commits
> >>
> >
> > -------------------------------------------------------------------------
> > This SF.net email is sponsored by DB2 Express
> > Download DB2 Express C - the FREE version of DB2 express and take
> > control of your XML. No limits. Just data. Click to get it now.
> > http://sourceforge.net/powerbar/db2/
> > _______________________________________________
> > naviserver-devel mailing list
> > nav...@li...
> > https://lists.sourceforge.net/lists/listinfo/naviserver-devel
> >
>
> --
> Vlad Seryakov
> vl...@cr...
> http://www.crystalballinc.com/vlad/
>
> -------------------------------------------------------------------------
> This SF.net email is sponsored by DB2 Express
> Download DB2 Express C - the FREE version of DB2 express and take
> control of your XML. No limits. Just data. Click to get it now.
> http://sourceforge.net/powerbar/db2/
> _______________________________________________
> naviserver-devel mailing list
> nav...@li...
> https://lists.sourceforge.net/lists/listinfo/naviserver-devel
>

Re: [naviserver-devel] [naviserver-commits] naviserver/nsd returnresp.c, 1.2, 1.3

[Vlad S. <vl...@cr...>]

The idea here i may not explained that i am going to include Digest auth 
handler and in that handler it will assign www-authenticate header, 
because in case of Digest it is much more complicated. So using standard 
response handler it will check, if no header it will assume Basic 
authentication model, if exists, then it will use whatever supplied

Stephen Deasey wrote:
> Hmm, this is a bit like this:
> 
> http://sourceforge.net/mailarchive/forum.php?thread_name=a9250c0b0702221615x5c0ab5e8gf8983c7c912a7520%40mail.gmail.com&forum_name=naviserver-devel
> 
> Maybe we need the full set of add, conditonaly add and replace for the
> printf version?
> 
> 
> On 5/20/07, Vlad Seryakov <ser...@us...> wrote:
>> Update of /cvsroot/naviserver/naviserver/nsd
>> In directory sc8-pr-cvs16:/tmp/cvs-serv11302/nsd
>>
>> Modified Files:
>>         returnresp.c
>> Log Message:
>>         * nsd/returnresp.c: In Ns_ConnReturnUnauthorized check if header
>>         WWW-Authenticate already exists, this is in anticipation of new
>>         Digest auth method to be implemented soon.
>>
>>
>>
>> Index: returnresp.c
>> ===================================================================
>> RCS file: /cvsroot/naviserver/naviserver/nsd/returnresp.c,v
>> retrieving revision 1.2
>> retrieving revision 1.3
>> diff -C2 -d -r1.2 -r1.3
>> *** returnresp.c        16 May 2007 16:04:09 -0000      1.2
>> --- returnresp.c        20 May 2007 00:24:29 -0000      1.3
>> ***************
>> *** 331,340 ****
>>       int         result;
>>
>> !     Ns_DStringInit(&ds);
>> !     Ns_DStringVarAppend(&ds, "Basic realm=\"",
>> !                         connPtr->servPtr->opts.realm, "\"", NULL);
>> !     Ns_ConnSetHeaders(conn, "WWW-Authenticate", ds.string);
>> !     Ns_DStringFree(&ds);
>> !
>>       if (ReturnRedirect(conn, 401, &result)) {
>>           return result;
>> --- 331,341 ----
>>       int         result;
>>
>> !     if (Ns_SetIGet(conn->outputheaders, "WWW-Authenticate") == NULL) {
>> !         Ns_DStringInit(&ds);
>> !         Ns_DStringVarAppend(&ds, "Basic realm=\"",
>> !                             connPtr->servPtr->opts.realm, "\"", NULL);
>> !         Ns_ConnSetHeaders(conn, "WWW-Authenticate", ds.string);
>> !         Ns_DStringFree(&ds);
>> !     }
>>       if (ReturnRedirect(conn, 401, &result)) {
>>           return result;
>>
>>
>> -------------------------------------------------------------------------
>> This SF.net email is sponsored by DB2 Express
>> Download DB2 Express C - the FREE version of DB2 express and take
>> control of your XML. No limits. Just data. Click to get it now.
>> http://sourceforge.net/powerbar/db2/
>> _______________________________________________
>> naviserver-commits mailing list
>> nav...@li...
>> https://lists.sourceforge.net/lists/listinfo/naviserver-commits
>>
> 
> -------------------------------------------------------------------------
> This SF.net email is sponsored by DB2 Express
> Download DB2 Express C - the FREE version of DB2 express and take
> control of your XML. No limits. Just data. Click to get it now.
> http://sourceforge.net/powerbar/db2/
> _______________________________________________
> naviserver-devel mailing list
> nav...@li...
> https://lists.sourceforge.net/lists/listinfo/naviserver-devel
> 

-- 
Vlad Seryakov
vl...@cr...
http://www.crystalballinc.com/vlad/

Re: [naviserver-devel] [naviserver-commits] naviserver/nsd returnresp.c, 1.2, 1.3

[Stephen D. <sd...@gm...>]

Hmm, this is a bit like this:

http://sourceforge.net/mailarchive/forum.php?thread_name=a9250c0b0702221615x5c0ab5e8gf8983c7c912a7520%40mail.gmail.com&forum_name=naviserver-devel

Maybe we need the full set of add, conditonaly add and replace for the
printf version?


On 5/20/07, Vlad Seryakov <ser...@us...> wrote:
> Update of /cvsroot/naviserver/naviserver/nsd
> In directory sc8-pr-cvs16:/tmp/cvs-serv11302/nsd
>
> Modified Files:
>         returnresp.c
> Log Message:
>         * nsd/returnresp.c: In Ns_ConnReturnUnauthorized check if header
>         WWW-Authenticate already exists, this is in anticipation of new
>         Digest auth method to be implemented soon.
>
>
>
> Index: returnresp.c
> ===================================================================
> RCS file: /cvsroot/naviserver/naviserver/nsd/returnresp.c,v
> retrieving revision 1.2
> retrieving revision 1.3
> diff -C2 -d -r1.2 -r1.3
> *** returnresp.c        16 May 2007 16:04:09 -0000      1.2
> --- returnresp.c        20 May 2007 00:24:29 -0000      1.3
> ***************
> *** 331,340 ****
>       int         result;
>
> !     Ns_DStringInit(&ds);
> !     Ns_DStringVarAppend(&ds, "Basic realm=\"",
> !                         connPtr->servPtr->opts.realm, "\"", NULL);
> !     Ns_ConnSetHeaders(conn, "WWW-Authenticate", ds.string);
> !     Ns_DStringFree(&ds);
> !
>       if (ReturnRedirect(conn, 401, &result)) {
>           return result;
> --- 331,341 ----
>       int         result;
>
> !     if (Ns_SetIGet(conn->outputheaders, "WWW-Authenticate") == NULL) {
> !         Ns_DStringInit(&ds);
> !         Ns_DStringVarAppend(&ds, "Basic realm=\"",
> !                             connPtr->servPtr->opts.realm, "\"", NULL);
> !         Ns_ConnSetHeaders(conn, "WWW-Authenticate", ds.string);
> !         Ns_DStringFree(&ds);
> !     }
>       if (ReturnRedirect(conn, 401, &result)) {
>           return result;
>
>
> -------------------------------------------------------------------------
> This SF.net email is sponsored by DB2 Express
> Download DB2 Express C - the FREE version of DB2 express and take
> control of your XML. No limits. Just data. Click to get it now.
> http://sourceforge.net/powerbar/db2/
> _______________________________________________
> naviserver-commits mailing list
> nav...@li...
> https://lists.sourceforge.net/lists/listinfo/naviserver-commits
>

Re: [naviserver-devel] WebDAV support

[Vlad S. <vl...@cr...>]

There is webdav module in AOLServer repository that can be used with 
naviserver

Kulcsár Ferenc wrote:
> Hello!
> 
> Can I use naviserver as WebDAV server? Is it a WebDAV modul for naviserver? It is a "must have" for my compani.
> 
> Regards,
> Feri
> 
> 
> -------------------------------------------------------------------------
> This SF.net email is sponsored by DB2 Express
> Download DB2 Express C - the FREE version of DB2 express and take
> control of your XML. No limits. Just data. Click to get it now.
> http://sourceforge.net/powerbar/db2/
> _______________________________________________
> naviserver-devel mailing list
> nav...@li...
> https://lists.sourceforge.net/lists/listinfo/naviserver-devel
> 

[naviserver-devel] WebDAV support

[F. <kul...@se...>]

Hello!

Can I use naviserver as WebDAV server? Is it a WebDAV modul for naviserver? It is a "must have" for my compani.

Regards,
Feri