[Serdev] openser/ser - avoiding forks
reticent
tavis.lists at galaxytelecom.net
Thu Jun 16 05:45:19 UTC 2005
Section 10.3, Heading 8 (date header) has been implemented in the
textopts module by Jiri Kuthan, 2004-05-09 =P
Its just not documented anywhere,but if you look through the source code
for the textops module, its there
I sent an email to the list regarding this several monthes back,
although in retrospect i really should have submitted a patch to the
documentation for the "textops" module which i have now included with
this email (however it really should be reviewed as my description may
not be entirely accurate and could be elaborated on).
For reference, the function is called "append_time"
--
I really agree with Kristin Galway in respect to the current state of
documentation for SER, i've spent a large amount of time searching for
documentation only to discover that none exists leading me to spend time
digging through code and sending emails requesting help when a few lines
of text would have been (probably) all i needed.
Of course this does place the onus on us (considering that we are
complaining about it) to submit patches to the documentation.
So if (and i'm sure there are at least a few of us out there) you have
spent some time understanding/documenting features or procedures for
your own purposes that were not already documented (or where the
documentation was inadequate or non-existent) then take a little time
and submit a patch for the associated README file.
Jan Janak wrote:
>Hello,
>
>I am trying to reconcile patches and improvement suggestions that have been
>left unanswered on the mailing lists. I went through serusers and serdev
>mailing list archives from the beginning of 2004 till now.
>
>I was mainly focusing on submitted patches and non-trivial feature requests.
>Below is a list of issues that I was able to find in the mailing list archives.
>Given the number of messages that were sent to the lists in the last year
>I am pretty sure that the list below will miss many suggestions.
>
>If you submitted a patch or improvement and it was neither accepted nor rejected,
>and it is still relevant, please resubmit your proposal to serdev at lists.iptel.org
>mailing list or (preferably) create an issue in the bug tracking system at:
>
>http://bugs.sip-router.org
>
>Please re-submit also improvements sent in individual messages to developers.
>
>Summary
>-------
>
>When it comes to accepting improvements, there surely is space for improvements
>but the situation, in my opinion, is not as bad as it may seem from the recent
>discussions. Properly reported bugs get usualy fixed quickly. The more detailed
>description the faster the fix, so I think there are no big problems with that.
>
>The list at the end of this message contains patches and feature requests that
>I was able to find in the archives. From the list the following issues have
>not yet been closed:
>
>1, 4, 7, 18, 19, 21, 22, 23, 24, 25, 30, 31
>
>And from those only 1, 4, 18, 21, 22, 25, 31 contained patches that have
>been left unanswered. That 7 patches which still need to be processed.
>
>The following patches have in my opinion low importance:
>
> 1 - date header in REGISTER replies
> 18 - radius fix for acc
> 21 - check_to for digest credentials without database
>
>The following patches can be classified as important:
>
> 4 - AVP support in acc
> 22 - transactional auth replies
> 25 - Free TLS
> 31 - Xten improvements of pa
>
>In addition to that there are the following improvement suggestions which
>did not contain any patches: 7, 19, 23, 24
>
>So I was able to find 4 important and 3 less important patches that were not
>integrated in the last 1.5 year from the 31 items below. Also I created issues
>in the bug tracking system for all items below that are not yet closed.
>
>The winner among them in terms of e-mail volume is free TLS, of course.
>
> Jan.
>
>PS: Code back-porting is another issue and is not covered here.
>
>------------------------------------------------------------------------------
>
>SERUSERS - 2004
>---------------
>
>1)
> Date Header in REGISTER responses
> Feature request by TeleSIP, I have a patch for that already from
> Robert Sanders which will be integrated
> Status: OPEN
> Bug: SER-30
>
>2) PA interoperability with RTC, patch submitted by Klaus Darilion,
> integrated by Jamey HICKS
> Status: CLOSED
>
>3) branch=0 problem reported on 16 Jul 2004, closed by Jiri
> as "not a bug"
> Status: CLOSED
>
>4) AVP patch for acc module, submitted by Ramona on 31 Oct 2004
> Status: OPEN
> Bug: SER-31
>
>
>SERDEV - 2004
>-------------
>
>5)
> Video - related patch for nathelper, integrated by Maxim
> Status: CLOSED
>
>6)
> fix_nated_contact for nathelper (replied by Maxim)
> Status: CLOSED
>
>7)
> 16 Apr 2004
> Juha proposed adding npdi and pstn URI parameters (no patches)
> Status: OPEN
> Bug: SER-32
>
>8)
> Maxim submitted patch adding PIDs of all processes into the pid
> file, resolved by another means
> Status: CLOSED
>
>9)
> 20 Apr. 2004
> Alexander Mayhofer submitted patch for rtpproxy, commited by Maxim
> Status: CLOSED
>
>10)
> Test for realm in auth_radius too tight, reported by Cesar
> Hernandez, fixed by Jan
> Status: CLOSED
>
>11)
> 23 Apr 2004
> Multicast support patch, commited by Andrei
> Status: CLOSED
>
>12)
> 24 Apr 2004
> Postgres patch by Alexander Mayhofer, in CVS
> Status: CLOSED
>
>13)
> max_expires patch submitted by Jamey Hicks, in CVS
> Status: CLOSED
>
>14)
> start/stop commands fro SEMS into serctl by Klaus Darilion, rejected.
> Status: CLOSED
>
>15)
> 30 Jun 2004
> User agent support in xlog module by Alexander Mayhofer, in cvs
> Status: CLOSED
>
>16)
> 7 Jul 2004
> Patch for configurable user agent string by Maxim, resolved by other
> means
> Status: CLOSED
>
>17)
> 21 Jul 2004
> Patch for storing user agent string in location table, commited by
> Maxim
> Status: CLOSED
>
>18)
> 21 Jul 2004
> Radius-related patch for acc module
> Status: OPEN
> Bug: SER-33
>
>19)
> 2 Oct 2004
> Escaped uri characters are not interpreted properly, no patch
> Status: DEFERED
> Bug: SER-34
>
>20)
> 27 Oct 2004
> Mysql_ping patch by Dan Pascu, in cvs
> Status: CLOSED
>
>21)
> 10 Nov 2004
> uri_db patch to make it possible to use check_to and check_from
> without database (submitted by Marian Dumitru).
> Status: OPEN
> Bug: SER-35
>
>22)
> 25 Dec 2004
> Maxim's patch to make it possible send transactional replies from
> authentication modules
> Status: OPEN
> Bug: SER-36
>
>23)
> Additional header fields requested by Juha (server, refer-to)
> Status: OPEN
> Bug: SER-37
>
>
>SERUSERS - 2005
>---------------
>
>24)
> feature request route("string")
> Status: DEFERED
> Bug: SER-38
>
>25)
> Free TLS
> Status: OPEN
> Bug: SER-39
>
>26)
> 19 Apr 2005
> branch=0 in ACK problem, rejected by Jiri
> Status: CLOSED
>
>27)
> 20 Apr 2005
> mysql_ping patch for 0.8.14, not important enough to be commited
> to 0.8.14
> Status: CLOSED
>
>28)
> Backport of radius auth changes from unstable to stable, rejected
> Status: CLOSED
>
>
>SERDEV - 2005
>-------------
>
>29)
> Transaction replies in auth modules (by Maxim)
> Status: OPEN
> Bug: SER-36
>
>30)
> NAT support in usrloc and registrar, partially done
> Status: OPEN
> Bug: SER-40
>
>31)
> RFC3265,RFC3903 support in pa, I could not find the patch
> Status: OPEN
> Bug: SER-41
>
>_______________________________________________
>Serdev mailing list
>serdev at lists.iptel.org
>http://lists.iptel.org/mailman/listinfo/serdev
>
>
>
>
-------------- next part --------------
--- README.old Tue Aug 24 09:00:39 2004
+++ README Thu Jun 16 03:20:26 2005
@@ -35,7 +35,8 @@ Andrei Pelinescu-Onciul
1.3.7. append_to_reply(txt)
1.3.8. append_hf(hf)
1.3.9. append_urihf(prefix, suffix)
- 1.3.10. is_present_hf(hf_name)
+ 1.3.10. append_time()
+ 1.3.11. is_present_hf(hf_name)
1.4. Known Limitations
@@ -52,7 +53,8 @@ Andrei Pelinescu-Onciul
1-7. append_to_reply usage
1-8. append_hf usage
1-9. append_urihf usage
- 1-10. is_present_hf usage
+ 1-10. append_time usage
+ 1-11. is_present_hf usage
_________________________________________________________
Chapter 1. User's Guide
@@ -235,7 +237,19 @@ append_urihf("CC-Diversion: ", "\r\n");
...
_________________________________________________________
-1.3.10. is_present_hf(hf_name)
+1.3.10. append_time()
+
+ Should be used to insert a "Date:" header field into a response to
+ a REGISTER request (heading 8, section 10.3 of RFC3261 for reference)
+
+ Example 1-10. append_time usage
+...
+append_time();
+...
+
+ _________________________________________________________
+
+1.3.11. is_present_hf(hf_name)
Return true if a header field is present in message.
@@ -248,7 +262,7 @@ append_urihf("CC-Diversion: ", "\r\n");
* hf_name - Header field name.
- Example 1-10. is_present_hf usage
+ Example 1-11. is_present_hf usage
...
if (is_present_hf("From")) log(1, "From HF Present");
...
More information about the Serdev
mailing list