Re: ppg push - product-name attr.

Aarno Syvänen <[email protected]>
Newsgroups gmane.comp.mobile.kannel.devel
Message-ID <[email protected]>
Hi,

this is a bug in any case: Kannel should ignore attributes it does 
understand, expect in the case of fatal errors (phone number,
for instance). What specs are referring ? Kannel may even implement 
product-name ;)

And, of course, all testing tools are good. Even better are ppg testing 
tools not written by the writer of the code itself.

Aarno

On Wednesday, March 19, 2003, at 11:39 PM, Bas A. Schulte wrote:

> maybe another bug in ppg push handling. If my pap document contains a 
> product-name attribute (which is valid according to the spec I saw), 
> ppg fails to parse the request, this appears in the logfile:
>
> 2003-03-19 23:29:06 [14] WARNING: PPG: pap control entity erroneous, 
> the request unacceptable
>
> This piece of xml causes Kannel to fail:
>
> <?xml version="1.0"?>
> <!DOCTYPE pap PUBLIC "-//WAPFORUM//DTD PAP//EN"
>           "http://www.wapforum.org/DTD/pap_1.0.dtd">
> <pap product-name="blah">
> ...
>
> whilst this piece works:
>
> <?xml version="1.0"?>
> <!DOCTYPE pap PUBLIC "-//WAPFORUM//DTD PAP//EN"
>           "http://www.wapforum.org/DTD/pap_1.0.dtd">
> <pap>
> ...
>
> I looked through wap_push_ppg.c but boy, xml parsing in C *is* scary 
> ;) Not my cup of tea.
>
> BTW, I'm not totally sure this is actually a bug, it could be 
> something in a more recent version of the WAP/PAP spec. I checked in 
> WAP-247-PAP-20010429-a.pdf, which tells me this is an optional 
> (implied) attribute so I think Kannel should handle this.
>
> Regards,
>
> Bas.
>
> ps. I'm working on a simple commandline example java app that uses 
> Openwave's wap push library, freely available, might be good to add 
> this is a conformance and/or alternative testing tool? Kannel PPG does 
> work with the example PushIT application that comes with the lib 
> because that one does not generate the product-name attribute if it's 
> not filled in in the GUI. The commandline tool I'm using however uses 
> the lib's SimplePush class which sets it by default and you can't 
> override that (easily).
>
>
>
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.