[Kamailio-Devel] [ openser-Patches-1840951 ] insert_body() function

SourceForge.net noreply at sourceforge.net
Mon Sep 15 19:45:44 CEST 2008


Patches item #1840951, was opened at 2007-11-29 14:34
Message generated for change (Comment added) made by miconda
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=743022&aid=1840951&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: Fixed
Priority: 5
Private: No
Submitted By: Nobody/Anonymous (nobody)
Assigned to: Daniel-Constantin Mierla (miconda)
Summary: insert_body() function

Initial Comment:
there is no way to insert a body for a message passing bye. therefore an insert_body() function is required. a suitable place would be the textops module.

patch as an attachment, ready for review.



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

>Comment By: Daniel-Constantin Mierla (miconda)
Date: 2008-09-15 20:45

Message:
Function included in trunk.

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

Comment By: Daniel-Constantin Mierla (miconda)
Date: 2008-09-11 18:38

Message:
Perhaps insert_body() is cleaner for this operation, and should do:
- check if body exists already and return false now, perhaps in the future
should create multi-part body
- optional second parameter to specify content-type, if not present assume
text/plain or so


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

Comment By: Nobody/Anonymous (nobody)
Date: 2008-09-11 17:19

Message:
IMO adding a body is useful. If the body replace functions can not be used
to replace the non existing body with a new body then we should have a new
function.

klaus

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

Comment By: Henning Westerholt (henningw)
Date: 2008-09-11 17:07

Message:
Hi Daniel,

i've updated the patch to trunk and moved this to the patch section. Do
you think this functionality is still useful?

Cheers,

Henning
File Added: textops.patch

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

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



More information about the Devel mailing list