[Devel] [ openser-Bugs-1675036 ] PA module: NOTIFY Message body contains invalid XML document

[SourceForge.net]

Bugs item #1675036, was opened at 2007-03-06 18:05
Message generated for change (Comment added) made by miconda
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=743020&aid=1675036&group_id=139143

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: modules
Group: ver devel
>Status: Closed
>Resolution: Accepted
Priority: 5
Private: No
Submitted By: Nobody/Anonymous (nobody)
Assigned to: Daniel-Constantin Mierla (miconda)
Summary: PA module: NOTIFY Message body contains invalid XML document

Initial Comment:
PA module prepares invalid XML document as a Message Body for NOTIFY request: resulting XML document consists of duplicates of <geopriv><location-info><civilAddress> and <\status> tags as location section is not prepared by pidf_add_location() routine properly. Please find attached file with a patch fixed given error.

----------------------------------------------------------------------

>Comment By: Daniel-Constantin Mierla (miconda)
Date: 2007-04-04 17:34

Message:
Logged In: YES 
user_id=1246013
Originator: NO

Patch applied in 1.1.x. Newer releases should shift to new presence
implamentation.

----------------------------------------------------------------------

Comment By: Kovalevich Victor (vkc1974)
Date: 2007-03-13 15:16

Message:
Logged In: YES 
user_id=490489
Originator: NO

I agree, PRESENCE module looks to be a better solution. Solution I have
been involved is designed to use PA module services. We are going to
migrate to PRESENCE but not now.
Bug I have described looks to be caused by usual misprints - it is
reproduced on 1.1.x branch as well.

----------------------------------------------------------------------

Comment By: Kovalevich Victor (vkc1974)
Date: 2007-03-12 12:50

Message:
Logged In: YES 
user_id=490489
Originator: NO

I agree, PRESENCE module looks to be a better solution. Solution I have
been involved is designed to use PA module services. We are going to
migrate to PRESENCE but not now.
Bug I have described looks to be caused by usual misprints - it is
reproduced on 1.1.x branch as well.

----------------------------------------------------------------------

Comment By: Kovalevich Victor (vkc1974)
Date: 2007-03-07 12:38

Message:
Logged In: YES 
user_id=490489
Originator: NO

I agree, PRESENCE module looks to be a better solution. Solution I have
been involved is designed to use PA module services. We are going to
migrate to PRESENCE but not now.
Bug I have described looks to be caused by usual misprints - it is
reproduced on 1.1.x branch as well.

----------------------------------------------------------------------

Comment By: Daniel-Constantin Mierla (miconda)
Date: 2007-03-07 12:08

Message:
Logged In: YES 
user_id=1246013
Originator: NO

Could you try new presence modules? PA is marked as deprecated, more
functionality being provided by new presence and pua* modules:
http://openser.org/dokuwiki/doku.php/presence:presence-module

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=743020&aid=1675036&group_id=139143