/[resiprocate]/main/repro/repro.config
ViewVC logotype

Log of /main/repro/repro.config

Parent Directory Parent Directory | Revision Log Revision Log


Links to HEAD: (view) (download) (annotate)
Sticky Revision:

Revision 11220 - (view) (download) (annotate) - [select for diffs]
Modified Sat Aug 30 14:15:39 2014 UTC (5 years, 2 months ago) by dpocock
File length: 39697 byte(s)
Diff to previous 11151
repro: Logging: configuring Syslog facility

Revision 11151 - (view) (download) (annotate) - [select for diffs]
Modified Tue Apr 22 16:27:35 2014 UTC (5 years, 6 months ago) by sgodin
File length: 39605 byte(s)
Diff to previous 11146
-added ability to install a SipMessageLoggingHandler that will get called for every inbound and outbound
 SIP Message on a particular transport - work started by Palladin
-added SipStack::setTransportSipMessageLoggingHandler to set logger on all added transports 
-added new Repro setting EnableSipMessageLogging to enable INFO level SipMessage logging
-ReproRunner: enasure we pass PrivateKeyPassPhrase when custom private key file is provided 
-ReproRunner: if transprot fails to create - don't bother processing it further



Revision 11146 - (view) (download) (annotate) - [select for diffs]
Modified Mon Apr 21 20:54:26 2014 UTC (5 years, 6 months ago) by sgodin
File length: 39445 byte(s)
Diff to previous 10929
-added ability to use Domain Private Key files that are encrypted with a pass phrase
-repro: disable auth-int authentication in repro by default - appears buggy
-repro: allow private key pass pharse to be configured as a global default or per advanced transport 
 -note:  storing this in clear text config is a really bad idea - we should improve this in the future, however
         this implementation shows how someone can use private key passwords in a more secured custom version 
         of repro 


Revision 10929 - (view) (download) (annotate) - [select for diffs]
Modified Tue Jan 28 18:24:44 2014 UTC (5 years, 9 months ago) by Dpocock
File length: 39112 byte(s)
Diff to previous 10925
repro: fix config syntax for PluginDirectory

Revision 10925 - (view) (download) (annotate) - [select for diffs]
Modified Mon Jan 27 18:55:14 2014 UTC (5 years, 9 months ago) by Dpocock
File length: 39110 byte(s)
Diff to previous 10914
repro: CookieAuthenticator: checking extension header against cookie

Revision 10914 - (view) (download) (annotate) - [select for diffs]
Modified Tue Jan 21 21:48:42 2014 UTC (5 years, 9 months ago) by Dpocock
File length: 38959 byte(s)
Diff to previous 10899
repro: repro.config: add notes about users.txt file format

Revision 10899 - (view) (download) (annotate) - [select for diffs]
Modified Tue Jan 21 15:04:08 2014 UTC (5 years, 9 months ago) by Dpocock
File length: 38711 byte(s)
Diff to previous 10849
repro: take HttpAdminPassword out of repro.config, use a htdigest file

Revision 10849 - (view) (download) (annotate) - [select for diffs]
Modified Sun Jan 12 10:43:30 2014 UTC (5 years, 10 months ago) by Dpocock
File length: 38610 byte(s)
Diff to previous 10826
repro: add StaticRealm config option to always challenge using a specified realm value

Revision 10826 - (view) (download) (annotate) - [select for diffs]
Modified Sat Jan 4 21:22:06 2014 UTC (5 years, 10 months ago) by Dpocock
File length: 38425 byte(s)
Diff to previous 10814
repro: certificate auth: look up AclStore in real time for TLS peer names

Revision 10814 - (view) (download) (annotate) - [select for diffs]
Modified Wed Jan 1 20:56:58 2014 UTC (5 years, 10 months ago) by Dpocock
File length: 38698 byte(s)
Diff to previous 10812
repro: add config option ChallengeThirdPartiesCallingLocalDomains

Revision 10812 - (view) (download) (annotate) - [select for diffs]
Modified Wed Jan 1 20:56:47 2014 UTC (5 years, 10 months ago) by Dpocock
File length: 38367 byte(s)
Diff to previous 10795
repro: add RADIUS support

Revision 10795 - (view) (download) (annotate) - [select for diffs]
Modified Tue Dec 31 01:04:59 2013 UTC (5 years, 10 months ago) by Dpocock
File length: 37507 byte(s)
Diff to previous 10793
repro: add support for TCPMinimumGCHeadroom

Revision 10793 - (view) (download) (annotate) - [select for diffs]
Modified Tue Dec 31 01:04:49 2013 UTC (5 years, 10 months ago) by Dpocock
File length: 36868 byte(s)
Diff to previous 10792
repro: add TCPConnectionGCAge config option for garbage collection

Revision 10792 - (view) (download) (annotate) - [select for diffs]
Modified Tue Dec 31 01:04:44 2013 UTC (5 years, 10 months ago) by Dpocock
File length: 36544 byte(s)
Diff to previous 10786
repro: configure User-Agent header for registrar, set default value with PACKAGE_VERSION

Revision 10786 - (view) (download) (annotate) - [select for diffs]
Modified Wed Dec 25 16:23:11 2013 UTC (5 years, 10 months ago) by Dpocock
File length: 36359 byte(s)
Diff to previous 10780
repro: add TlsTrustedPeers config option

Revision 10780 - (view) (download) (annotate) - [select for diffs]
Modified Tue Dec 24 01:26:26 2013 UTC (5 years, 10 months ago) by Dpocock
File length: 36086 byte(s)
Diff to previous 10723
resip: support WsCookieContext creation using factory pattern

Revision 10723 - (view) (download) (annotate) - [select for diffs]
Modified Mon Dec 9 15:44:03 2013 UTC (5 years, 11 months ago) by dpocock
File length: 35765 byte(s)
Diff to previous 10709
repro: add plugin DSO support

Revision 10709 - (view) (download) (annotate) - [select for diffs]
Modified Tue Dec 3 22:01:12 2013 UTC (5 years, 11 months ago) by dpocock
File length: 35096 byte(s)
Diff to previous 10702
repro: repro.config: fix sample LoggingInstanceName

Revision 10702 - (view) (download) (annotate) - [select for diffs]
Modified Wed Nov 27 16:58:16 2013 UTC (5 years, 11 months ago) by Dpocock
File length: 35094 byte(s)
Diff to previous 10700
repro: rename config option MessageSizeLimit to StreamMessageSizeLimit

Revision 10700 - (view) (download) (annotate) - [select for diffs]
Modified Wed Nov 27 15:23:22 2013 UTC (5 years, 11 months ago) by Dpocock
File length: 34937 byte(s)
Diff to previous 10561
repro: add new repro.config option MessageSizeLimit

Revision 10561 - (view) (download) (annotate) - [select for diffs]
Modified Sat Sep 28 07:48:11 2013 UTC (6 years, 1 month ago) by dpocock
File length: 34699 byte(s)
Diff to previous 10471
merge: b-gsoc-2013: WebSocket handshake cookie parsing and validation

Revision 10471 - (view) (download) (annotate) - [select for diffs]
Modified Wed Sep 11 12:54:31 2013 UTC (6 years, 2 months ago) by dpocock
File length: 34400 byte(s)
Diff to previous 10470
repro: make the CertificatePath optional with no default

Revision 10470 - (view) (download) (annotate) - [select for diffs]
Modified Wed Sep 11 11:38:21 2013 UTC (6 years, 2 months ago) by dpocock
File length: 34221 byte(s)
Diff to previous 10291
repro: allow specification of arbitrary domain cert/key PEM files

Revision 10291 - (view) (download) (annotate) - [select for diffs]
Modified Thu Jul 18 13:24:43 2013 UTC (6 years, 3 months ago) by Dpocock
File length: 33587 byte(s)
Diff to previous 10270
repro: enable IPv6 by default

Revision 10270 - (view) (download) (annotate) - [select for diffs]
Modified Thu Jul 18 07:57:59 2013 UTC (6 years, 3 months ago) by dpocock
File length: 33588 byte(s)
Diff to previous 10103
repro: added support for WebAdmin and CommandServer binding to specific IP addresses

Revision 10103 - (view) (download) (annotate) - [select for diffs]
Modified Fri Apr 5 09:08:36 2013 UTC (6 years, 7 months ago) by Dpocock
File length: 33253 byte(s)
Diff to previous 10102
repro: repro.config: more updates for WS/WSS support

Revision 10102 - (view) (download) (annotate) - [select for diffs]
Modified Thu Apr 4 22:30:43 2013 UTC (6 years, 7 months ago) by dpocock
File length: 32928 byte(s)
Diff to previous 10094
repro: repro.config, disable WS/WSS by default

Revision 10094 - (view) (download) (annotate) - [select for diffs]
Modified Thu Apr 4 22:02:03 2013 UTC (6 years, 7 months ago) by dpocock
File length: 32931 byte(s)
Diff to previous 10052
Merge b-webrtc into main

Revision 10052 - (view) (download) (annotate) - [select for diffs]
Modified Thu Mar 21 19:04:45 2013 UTC (6 years, 7 months ago) by dpocock
File length: 32741 byte(s)
Diff to previous 10022
repro: sample config: disable CommonNameMappings by default

Revision 10022 - (view) (download) (annotate) - [select for diffs]
Modified Tue Feb 26 18:49:36 2013 UTC (6 years, 8 months ago) by dpocock
File length: 32627 byte(s)
Diff to previous 10009
repro: Add supporting for starting as root and dropping privileges

Revision 10009 - (view) (download) (annotate) - [select for diffs]
Modified Fri Feb 22 19:27:44 2013 UTC (6 years, 8 months ago) by sgodin
File length: 32277 byte(s)
Diff to previous 10008
-ensure that we can accept IPv6 addresses in the Transport settings in repro.config

Revision 10008 - (view) (download) (annotate) - [select for diffs]
Modified Thu Feb 21 22:49:55 2013 UTC (6 years, 8 months ago) by dpocock
File length: 31963 byte(s)
Diff to previous 9828
repro: add LoggingInstanceName config option

Revision 9828 - (view) (download) (annotate) - [select for diffs]
Modified Sat Aug 4 12:26:51 2012 UTC (7 years, 3 months ago) by dpocock
File length: 31763 byte(s)
Diff to previous 9813
repro: CertificateAuthenticator: add CommonNameMappings config option to load CN mappings from a text file

Revision 9813 - (view) (download) (annotate) - [select for diffs]
Modified Thu Aug 2 17:38:19 2012 UTC (7 years, 3 months ago) by dpocock
File length: 30909 byte(s)
Diff to previous 9809
repro: add new EnumDomains config option for the corresponding SipStack property

Revision 9809 - (view) (download) (annotate) - [select for diffs]
Modified Thu Aug 2 17:37:19 2012 UTC (7 years, 3 months ago) by dpocock
File length: 30649 byte(s)
Diff to previous 9802
repro: eliminate code that automatically adds localhost, machine name, etc to the Domains config variable

Revision 9802 - (view) (download) (annotate) - [select for diffs]
Modified Fri Jul 13 15:56:28 2012 UTC (7 years, 4 months ago) by sgodin
File length: 30163 byte(s)
Diff to previous 9796
-repro config - disable TLS by default
-fix repro readme.txt - had old command line format stuff in it
-changes to repro windows installer:
 -Add repro.config
 -Add shortcut to repro.config on start menu
 -Add working directory to Repro shortcut so that config file is found

Revision 9796 - (view) (download) (annotate) - [select for diffs]
Modified Thu Jun 28 19:44:29 2012 UTC (7 years, 4 months ago) by sgodin
File length: 30166 byte(s)
Diff to previous 9791
-modified repro version to match resip version (1.9)
-log routes for registration Accounting 
-repro.config formatting and typo corrections
-fixed compilation issue in ContactInstanceRecord for some compilers

Revision 9791 - (view) (download) (annotate) - [select for diffs]
Modified Wed Jun 20 18:18:36 2012 UTC (7 years, 4 months ago) by sgodin
File length: 30134 byte(s)
Diff to previous 9785
-added ability to disable RegistrationRefresh accounting event

Revision 9785 - (view) (download) (annotate) - [select for diffs]
Modified Thu Jun 14 16:46:05 2012 UTC (7 years, 4 months ago) by sgodin
File length: 30012 byte(s)
Diff to previous 9784
-default via and routing headers on accounting events to disabled

Revision 9784 - (view) (download) (annotate) - [select for diffs]
Modified Thu Jun 14 16:24:30 2012 UTC (7 years, 4 months ago) by sgodin
File length: 30008 byte(s)
Diff to previous 9770
-added configuration so specify if routing and/or via headers headers should be included in call accounting events
 -four new config settings: SessionAccountingAddRoutingHeaders, SessionAccountingAddViaHeaders,
  RegistrationAccountingAddRoutingHeaders, RegistrationAccountingAddViaHeaders

Revision 9770 - (view) (download) (annotate) - [select for diffs]
Modified Wed Jun 13 18:30:58 2012 UTC (7 years, 4 months ago) by sgodin
File length: 29296 byte(s)
Diff to previous 9740
Add registration and call/session accounting
--------------------------------------------
-configured/enabled via configuration file

-Session Accounting - When enabled resiprocate will push a JSON formatted 
 events for sip session related messaging that the proxy receives,
 to a persistent message queue that uses berkeleydb backed storage.
 The following call events are logged:
   SessionCreated - INVITE passing authentiation was received
   SessionRouted - received INVITE was forward to a target
   SessionRedirected - session was 3xx redirected or REFERed
   SessionEstablished - there was 2xx answer to an INVITE (only generate for first 2xx)
   SessionCancelled - CANCEL was received
   SessionEnded - BYE was received from either end
   SessionError - a 4xx, 5xx, or 6xx response was sent to invitor

-Registration Accounting - When enabled resiprocate will push a JSON formatted 
 events for every registration, re-registration, and unregistration message
 received to a persistent message queue that uses berkeleydb backed storage.
 The following registration events are logged:
   RegistrationAdded - initial registration received
   RegistrationRefreshed - registration refresh received / re-register
   RegistrationRemoved - registration removed by client / un-register
   RegistrationRemovedAll - all contacts registration remove / unregister received

-Consuming Accounting Events:
 Users must ensure that the message queues are consumed, or they will grow without
 bound.  A queuetostream consumer process is provided, that will consume the 
 events from the message queue and stream them to stdout.  This output stream can
 be consumed by linux scripting tools and converted to database records or some
 other relevant representation of the data.  
 For example: 
 ./queuetostream ./sessioneventqueue > sessionstreamconsumer.py
 ./queuetostream ./regeventqueue > regstreamconsumer.py
 In the future a MySQL consumer may also be provided in order to update
 session accounting records in a MySQL database table.

Revision 9740 - (view) (download) (annotate) - [select for diffs]
Modified Thu May 31 16:33:12 2012 UTC (7 years, 5 months ago) by sgodin
File length: 26763 byte(s)
Diff to previous 9688
-added an ability to configure a different database instance for some of the repro database tables
 New configuration file settings, RuntimeMySQLServer and subsetting, were added to facilitate this.

 The Users and MessageSilo database tables are different from the other repro configuration
 database tables, in that they are accessed at runtime as SIP requests arrive.  It may be
 desirable to use BerkeleyDb for the other repro tables (which are read at starup time, then 
 cached in memory), and MySQL for the runtime accessed tables; or two separate MySQL instances 
 for these different table sets.  The new configuration settings allow you to achieve this.



Revision 9688 - (view) (download) (annotate) - [select for diffs]
Modified Fri May 18 19:16:03 2012 UTC (7 years, 5 months ago) by sgodin
File length: 26859 byte(s)
Diff to previous 9633
-simplify AsyncProcessorMessage by removing requirement to implement clone() and copy constructor - messages are never copied
-optimize pcre use in MessageSilo - we don't use replacement patterns to set REG_NOSUB flag properly
-added new repro setting to assume that first hop supports outbound

There are cases where the first hop in a particular network supports the concept of outbound
and ensures all messaging for a client is delivered over the same connection used for
registration.  This could be a SBC or other NAT traversal aid router that uses the Path 
header.  However such endpoints may not be 100% compliant with outbound RFC and may not 
include a ;ob parameter in the path header.  This parameter is required in order for repro
to have knowledge that the first hop does support outbound, and it will reject registrations
that appear to be using outboud (ie. instanceId and regId) with a 439 (First Hop Lacks Outbound
Support).  In this case it can be desirable when using repro as the registrar to not reject
REGISTRATION requests that contain an instanceId and regId with a 439.
If this setting is enabled, then repro will assume the first hop supports outbound 
and not return this error.

Revision 9633 - (view) (download) (annotate) - [select for diffs]
Modified Wed May 2 19:27:38 2012 UTC (7 years, 6 months ago) by sgodin
File length: 25886 byte(s)
Diff to previous 9622
-merge work from branches/b-repro-enh-20120205

-up repro version number 0.9
-added new Settings page to repro web interface to show current command line / files settings in use
 - will also display some some low level stack info and congestion stats (if enabled)
-added new config setting StatisticsLogInterval to specify how often statistics are dumped to 
 the log files
-cleaned up the WebAdmin constructor to take a Proxy object
-added new KeyValueStore class for highly efficient generic storage
 - keys are simple integer indexes into a vector and must be allocated 
   before they are used
 - currently supports storage and retrieval of the following types:
   Data, bool, short, unsigned short, int, unsigned int, and UInt64
-Converted ConfigParse storage from std::multimap to HashMultiMap
-removed print in TcpConnection that could end up printing garbage at the end of 
 messages that are not null terminated
-added KeyValueStore to three strategic locations in repro, allowing custom Processors 
 (Monkeys, Lemurs and Baboons) to store state scoped as follows:
     -Global Proxy Scope - Proxy::getKeyValueStore
     -Request Scope - RequestContext::getKeyValueStore
     -Target Scope - Target::getKeyValueStore
 Before this storage can be used you must statically allocate a storage key.
 See mFromTrustedNodeKey use in the IsTrustedNode class for an example.
-major changes to how repro is started up, to allow easier additions of
 custom startup logic, such as adding custom Processors (Monkeys, Lemurs and 
 Baboons) to the default Processor chains
  - almost all logic that was in repro.cxx has been moved out to a new class
    (ReproRunner), and split into smaller virtual methods that can be overridden
  - see comments at the top of repro.cxx for an example of how to add custom
    processors
-added option to enable some basic P-Asserted-Identity header handling
 - After auth is successful 
   - if P-Perferred-Identity header is present remove it
   - if no P-Asserted-Identity header is present, then add one
 - Removal of P-Asserted-Identity if Privacy header is set to "id"
   Note: Since we have no better mechanism to determine if destination is 
         trusted or not we assume that all destinations outside our domain 
         are not-trusted and will remove the P-Asserted-Identity header
-modified how ;lr param on routes are processed - moved logic from StaticRoute 
 to ResponseContext - this slightly changes the behaviour
 - previous implementation used to stop the processing chain as soon
   as the first matching route with a ;lr is found, new implementation
   does not stop Target processing if a ;lr is present
 - Multiple targets can now have a ;lr on them and be processed
   properly.  The ;lr parameter is now examined in 
   ResponseContext::beginClientTransaction, and if present the RequestUri
   is not altered and a Route header is added instead
-fixed a duplicate removeContact call in OutboundTargetHandler (a merge conflict
 victim)
-avoid unnecessary iteration through target list in StaticRoute when try to determine
 if auth is required
-added two new constructors to QValueTarget to make is easier to form targets from 
 a NameAddr or Uri only
-removed some unused code in ResponseContext:  addOutboundBatch and mOutboundMap
-renamed Target::targetPtrCompare to Target::priorityMetricCompare to be more descriptive
-implement proper q-value processing of contacts in a redirect response

-fix for a long standing issue in repro that started in rev6794, where
 repro can be over protective and issue 403 responses for legitimate 
 mid-dialog requests.  The issue occurs when a repro domain user forms
 a dialog with a user in another external domain.  Any mid-dialog
 requests coming from the external domain would get 403'd.  This 
 was due to the logic in the AmIResponsible monkey, and the fact that 
 such requests have the repro endpoints contact address in the 
 RequestUri (typically the endpoints IP address), so not belonging
 to repro's domain, and the From user is not being from repro's domain.

 The original code to only do these checks for out of dialog requests
 was reintroduced with the following comments:

         // only perform relay check for out-of-dialog requests
         // !bwc! Um, then all anyone has to do to get us to be their relay
         //       is throw in a spurious to-tag...
         //       This smells funny. I am commenting it out.
         // .slg. Putting code back in and clarifying the funny smell.....
         //       We only want to do this check for out of dialog requests, since 
         //       mid-dialog requests could be 403'd otherwise.  Consider
         //       an INVITE request from a repro domain user to a user in 
         //       another domain.  The resulting ACK/200, BYE or any other
         //       mid-dialog request coming from the remote domain, will contain
         //       the repro users contact address in the RequestUri and a 
         //       foreign domain in the from header.  We want to ensure these
         //       requests are not 403'd.  Byron's comment about an endpoint getting
         //       us to relay by placing a spurious to tag in the request still 
         //       stands. Perhaps we ought to be checking the To header domain in 
         //       this case - however that is also weak, since the To header is not
         //       used in routing and easily be set to a URI in our domain to trick
         //       repro into forwarding.  Note:  From header domain checking is
         //       stronger than To header domain checking, since if the domain is 
         //       ours, then it must pass Digest Authentication (at least for non 
         //       ACK and BYE requests).

-add ability for a repro admin to add manual / permanent registrations - such
 manually added registrations are persisted to the database, and loaded at startup
 - Manual registrations can be added on the Registration Web Page
-add display of registered contact's QValue on registrations web page
-added ability to specify a Path on new static registration feature
-cleanup some old hacks now that we have the ability to manually add registrations
  - ParallelForkStaticRoutes no longer combines StaticRoutes Targets and LocationServer
    Targets
  - Static Routes are no longer added as QValueTargets, they are now added as simple Targets.
    So they are no longer susceptible to the various QValue Settings - ie. QValueMsBeforeCancel.
  - Added new ContinueProcessingAfterRoutesFound setting:
    By default (false) we will stop looking for more Targets if we have found
    matching routes.  Setting this value to true will allow the LocationServer Monkey
    to run after StaticRoutes have been found.  In this case the matching
    StaticRoutes become fallback targets, processed only after all location server 
    Targets fail.

-added new Helper method:  Tuple getClientPublicAddress(const SipMessage& request)
  - look at Via headers, and finds the first public IP address closest to the sending
    client.
-added loopback address checking to Tuple::isPrivateAddress
-modified ServerRegistration:
 - Ensure that ContactInstanceRecord::mReceivedFrom is always populated - not just 
   in outbound use cases - added a new flag to indicate when flow routing is required
 - Added a new mPublicAddress flag member to ContactInstanceRecord - this will assist
   with an upcoming feature to do geo proximity routing
-added new ContactInstanceRecord::mPublicAddress to regsync process
-removed a chatty log line in DialogSetId
-used Symbols::COMMA in a few spots it wasn't being used in
-removed unimplemented targetCount method on ResponseContext
-renamed Helper::isSenderBehindNAT to isClientBehindNAT for consistency
-ensure we only skip the Monkey chain in StaticRoute if we actual found routes
-added missing license block to ForkControlMessage
-adding new Baboon:  GeoProximityTargetSorter
 If enabled, then this baboon can post-process the target list.  
 This includes targets from the StaticRoute monkey and/or targets
 from the LocationServer monkey.  Requests that meet the filter 
 criteria will have their Target list, flatened (serialized) and
 ordered based on the proximity of the target to the client sending
 the request.  Proximity is determined by looking for a 
 x-repro-geolocation="<latitude>,<longitude>" parameter on the Contact
 header of a received request, or the Contact headers of Registration
 requests.  If this parameter is not found, then this processor will
 attempt to determine the public IP address closest to the client or
 target and use the MaxMind Geo IP library to lookup the geo location.
-requires floating point support for geo proximity calculations
-resip TCP transports can crash repro on uncaught exception - if garbage is received
 on the socket, and there is no Content-Length header, then SipMessage::Exception can
 throw, and it was not caught with the existing ParseException catch handler.  Changed
 to catch BaseException instead.
-removed unsafe logging statement in DumTimeout - if DUM and stack are in different threads, then
 crash could occur
-enable Inserter functionality for collections that store pointers
 to items.  Use InsertP instead of Inserter for such collections.
 For example:  
 std::vector<std::string*> v;
 ... populate vector ...
 std::cout << InserterP(v) << std::endl;
-statictize strings in Inserter/InserterP
-add InserterP test cases to rutil/test/testInserter.cxx
-cleaned up Processor and ProcessorChain classes
 - added mName property in anticipation of a future capability to define processor 
   chains in the configuration file
 - simplified operator<< for processors, to use Name
-modified some Inserter uses to use new InserterP function
-allow custom repro implementations to add themselves to the RegistrarHandler so that
 registration messages can be processed and reacted to
-added a singleResultQuery API to MySqlDb
-series of interfaces to make is easier to implement new Asynchronous Monkey's / Processors that utilize a common thread pool
-removed getTransactionId from ForkControlMessage - method exists on base class, so it's not needed
-removed getTransactionId and tid() from UserInfoMessage - getTransactionId method exists on base class, so they are not needed
-made Worker and abstract base class
-modified ConfigParser to throw a resip style exception when failing to open configuration file
-minor cleanups to ConfigParser configuration file parsing routine

-added new RequestFilter monkey
 - allows user to configure conditions under which an inbound request should be rejected or not
 - allows two regular expression conditions that can be applied to any SIP message header:
   this includes the request-line, standard SIP headers and custom SIP headers.  If a header
   that can appear multiple time is specified, then each instance of the header is checked.
 - When conditions are met, allows the action carried out to be defined:
   - Accept - accepts this request and stops further processing in Request Filter monkey 
   - Reject - rejects this request with the provided SIP status code and reason text
   - SQL query - only available when MySQL support is compiled in - runs an arbitrary stored
     procedure or query, using replacement strings from the 2 condition regular expressions
     - query must return an empty string or "0" to instruct repro to Accept the request, or a 
       string containing "<SIP Reject Status Code>[, <SIP Reject Reason>]" to Reject the request
     - using the repro configuration file the SQL Query can be configured to operate on a
       completely different my SQL instance/server than the repro configuration
   - Filters are defined in the HTTP web interface via new Add Filter, Edit Filter and Show
     Filters web pages.  There is an ability to test the condition regular expressions from 
     the web page as well.
   - other Monkey settings are configured in the repro configuration file or via command line:
     DisableRequestFilterProcessor, RequestFilterDefaultNoMatchBehavior, 
     RequestFilterDefaultDBErrorBehavior, RequestFilterMySQLServer (and other mySQL related settings) 
   - can be used to implement a User Blocking functionality - ie. calls and instant messages 
     from user X to user Y should always be blocked, because user X is in user Y's block list
   - introduced new FilterStore configuration database table to store the Filters configured
     on the web pages

-added AsyncProcessorDispatcher / thread pool to repro that be shared by all AsyncProcessors
 - currently only used by new RequestFilter monkey
-cleaned up some implementation in AbstractDb to remove some code duplication
-catch config file parse, and missing file errors (exceptions) at repro startup
-RouteStore 
  - stop webpage from being able to add two routes with the same Key
  - optimized data fetch for displaying routes on web page
-Repro Web Interface
 - added new Add Filter, Edit Filter and Show Filters pages
 - made use of HTML tables consistent across all pages
 - made table backgrounds white to improved appearance
 - added title to right hand side pane
 - cleaned up formatting on many pages
 - ordered methods in source to match ordering on web page 
 - added warning to Domains page, that repro must be restarted

-added version information to startup logs and to display on web interface
-added bottom footer on web interface with link to www.resiprocate.org
-added new RequestFilter files to all Visual Studio projects and linux Makefile
-upped repro version to 0.9

-fix a bug in repro web interface, where fragmented HTTP messages were not being handled correctly
-fixed possible assert if a transport error is seen after trying to send an ACK message

-added optional MessageSilo support to repro
 - stores IM's (ie. SIP MESSAGE requests) for offline users
 - replays messages to users when they register (ie. come back online)
 - records are persisted to a database table, so they survive shutdowns
 - configurable filters exist for DestUri, MimeType, MessageBody size

-moved CommandServer from repro to reprolib
-added restart command to reprocmd - allows restarting repro to apply configuration changes while leaving 
 the in memory registration database in tact
-propagate db insert/update failures to callers - web interface now shows errors if record fails to update in db
-remove unused AbstractDb API's: writeRoute and writeFilter
-added ability for tables to have non-unique keys (ie. duplicate records)
-modified WorkerThread to support work that does not require a response to be queued back to the stack
-added database transaction support to BerekelyDb and MySQL implementations
-reorganized repro runner startup ordering so that DUM TU is started before Proxy and added new 
 virtual createDatastore method - in order to facility new Silo Monkey startup that requires access to 
 the Registrar object
-make RequestFilter monkey options - use config flag
-allow a DateCategory to be created from time_t

-build fixes for linux, add --with-geoip option to autotools configure
-added --with-popt to configure so that test projects can be build easily with popt support
-added autodetection of epoll support on unix and automatic addition of HAVE_EPOLL define

-added ability for repro to report a 404 error when attempting to reach a user that does not exist
 - previously repro would always send a 480 response when attempting to reach an AOR that wasn't
   registered
-removed unused SipStack parameter to DigestAuthenticator monkey

-allow reprocmd to run defaulting to 127.0.0.1:5081 for server connection

-added secondary database support to BerkeleyDb - allows tables with a secondary index
 - my sql implementation of AVPs with two indexes was added as well
-message silo records can now be deleted by referencing a unique key (ie. timestamp+tid)
 - transaction support is no longer required
-optimized data copies when reading records from BerkeleyDb
-increase MySQL AVP table size for value field from 1024 to 4096
-implemented logic to periodically cleanup Silo records that have expired


Revision 9622 - (view) (download) (annotate) - [select for diffs]
Modified Mon Apr 30 23:11:46 2012 UTC (7 years, 6 months ago) by dpocock
File length: 15161 byte(s)
Diff to previous 9613
repro/tls: support for mutual TLS/client certificate verification (config options to enable the TLS client auth monkey)

Revision 9613 - (view) (download) (annotate) - [select for diffs]
Modified Sat Apr 28 10:10:37 2012 UTC (7 years, 6 months ago) by dpocock
File length: 13884 byte(s)
Diff to previous 9612
repro/tls: add more documentation about root certs to config file

Revision 9612 - (view) (download) (annotate) - [select for diffs]
Modified Sat Apr 28 10:10:23 2012 UTC (7 years, 6 months ago) by dpocock
File length: 12862 byte(s)
Diff to previous 9545
repro/tls: add support for loading a directory of root certificates or a file containing a bundle of root certificates

Revision 9545 - (view) (download) (annotate) - [select for diffs]
Modified Mon Apr 16 12:14:38 2012 UTC (7 years, 6 months ago) by dpocock
File length: 12135 byte(s)
Diff to previous 9498
repro: create PID file

Revision 9498 - (view) (download) (annotate) - [select for diffs]
Modified Sat Apr 7 22:29:19 2012 UTC (7 years, 7 months ago) by dpocock
File length: 11992 byte(s)
Diff to previous 9407
Add a daemonize option to repro (copied from reTurn)

Revision 9407 - (view) (download) (annotate) - [select for diffs]
Modified Fri Feb 17 17:22:52 2012 UTC (7 years, 8 months ago) by sgodin
File length: 11906 byte(s)
Diff to previous 9395
Cleaned up the MySQL database store implementation for repro, it is now in a deployable state

-allow MySQL connection parameters to be specified in repro config
-cleaned up MySQL initialization, so we fail to start if there is a db connect error
-remove all generic throws from MySQL implementation
-implemented MySQL connection recovery on errors 
  - when connection errors occur on query, try to re-connect immediately (once)
-made the number of worker threads in the repro authgrabber dispatcher configurable via the repro configuration file
-optimized query building in MySQLDb class
-added ability for repro.config to provide a select statement that allows a1 password retrieval from an
 arbitrary database table on the MySQL server
-modified UserAuthGrabber to use UserStore::getUserAuthInfo, instead of UserStore::getUserInfo API - we don't
 use the other fields retrieved, so this will provide an optimization
-modified repro so that Registration authentication uses the same pool of
 worker threads that the DigestAuthenticator uses when looking up
 credentials from the database.  Previously only auth look ups for 
 non-REGISTER requests (ie. INVITE, SUBSCRIBE, etc.) would be done 
 in a manner that didn't block inbound message processing.  A 
 lengthy auth check for a REGISTER request would cause delays in 
 processing all other REGISTER requests. 
-added a new method to SipStack so that you can post to TU without
 needing to clone/copy the message 
-optimized the Worker thread to avoid making a copy of the Message when
 posting to the stack 
-added .sql script to create repro tables on MySQL
-modified configure script to allow enabling of MySQL support in repro.  
 Note:  berkeley-db4 support is always enabled so removed REPRO_DB option 
 from configure.  If MySQL is enabled, then either BerkeleyDb or MySQL can 
 be used, depending on the settings in repro.config

Other Misc Changes
-set SVN eol-style to native - for ConfigParse class
-added missing command to help text of reprocmd 







Revision 9395 - (view) (download) (annotate) - [select for diffs]
Modified Fri Feb 10 22:51:18 2012 UTC (7 years, 9 months ago) by sgodin
File length: 9602 byte(s)
Diff to previous 9392
-fix bug with specifying congestion settings in repro config
-default outbound support to false in config, since record route URI must be specified first

Revision 9392 - (view) (download) (annotate) - [select for diffs]
Modified Fri Feb 10 18:19:59 2012 UTC (7 years, 9 months ago) by sgodin
File length: 9603 byte(s)
Diff to previous 9372
-merged work from b-repro-enh-20120205 branch

-added a new Command interface/server to repro that operates over a TCP socket and uses
 XML formatted messaging
-implemented the following commands:
  - GetStackInfo
  - GetStackStats
  - ResetStackStats
  - LogDnsCache
  - ClearDnsCache
  - GetDnsCache
  - GetCongestionStats
  - SetCongestionTolerance
  - Shutdown
  - GetProxyConfig
-added new reprocmd executable that connects to repro via new command socket server and sends commands
 base on command line arguments
-moved XMLCursor class from resip/stack to rutil

-added operator<< for Statistics Payload
-added ability to encode Congestion stats to a stream 
-modified GeneralCongestionManager::updateFifoTolerance to allow updating
 all fifos by specifying fifoDescription as an empty string
-added current fifo behaviour and tolerance settings to congestion stats output
-renamed configuration setting XmlRpcPort to RegSyncPort to differentiate it with new Command Port setting
-add new TCP server (CommandServer) to repro that accepts commands via XML formatted messages
-allow SipStack statistics to be reset/zero'd out
-modified GetStackStats command to be able to retrieve statistics that are accurate at the time of the request, 
 instead of just returning the statistics as of the last statistics interval
-added operator<< to ConfigParse


Revision 9372 - (view) (download) (annotate) - [select for diffs]
Modified Thu Feb 2 20:58:39 2012 UTC (7 years, 9 months ago) by sgodin
File length: 9462 byte(s)
Diff to previous 9367
-allow congestion manager settings to be configured in repro configuration file

Revision 9367 - (view) (download) (annotate) - [select for diffs]
Modified Wed Feb 1 16:12:20 2012 UTC (7 years, 9 months ago) by sgodin
File length: 8097 byte(s)
Diff to previous 9289
-merge work from resip-b-TKLC-perf_work branch

Merge Notes:

Various optimizations of Data
=============================

Made Data smaller, without sacrificing functionality. Data is 20 (56 vs 36) 
bytes smaller on 64-bit libs, and 4 (36 vs 32) bytes smaller on 32-bit libs. 
This was accomplished by making mSize, mCapacity, and mShareEnum 4-bytes on 
64-bit platforms (mShareEnum could be one byte, but it turns out this imposes a 
detectable performance penalty), and by having mShareEnum do double-duty as a 
null-terminator for mPreBuffer (Borrow==0), instead of requiring an extra byte 
at the end of mPreBuffer.

Several very simple functions have been inlined.

Functionality enhancements to a couple of functions:

- Data::md5() has been changed to Data::md5(Data::EncodingType type=HEX); this 
allows the output of md5() to be encoded as hex or Base64, or not encoded at all 
(binary).

- Data::replace(const Data& match, const Data& target) has been updated to 
Data::replace(const Data& match, const Data& target, int max=INT_MAX); this 
allows the maximum number of replacements to be specified.

Lastly, a few specialized hashing and comparison functions have been added:

- Data::caseInsensitiveTokenHash(); this is a case-insensitive hash that assumes 
that the Data is an RFC 3261 token (eg; branch params). This character set has 
the property that no character is equal to any other character when bit 6 is 
masked out, except for the alphabetical characters. (For alphabetical 
characters, bit 6 specifies whether the character is upper/lower case) This 
means that we can mask out bit 6, and then use a case-sensitive hash algorithm 
on the resulting characters, without hurting the collision properties of the 
hash, and get a case-insensitive hash as a result. This hash function is based 
on the Hsieh hash.

- bool Data::caseInsensitiveTokenCompare(const Data& rhs); this is an equality 
comparison that assumes that both Datas are RFC 3261 tokens (eg; branch 
parameters). This function takes advantage of the same properties of the RFC 
3261 token character set as caseInsensitiveTokenHash(), by using a bitmask 
instead of a true lowercase() operation. This ends up being faster than 
strncasecmp().

- Data& schemeLowercase(); this is a variant of lowercase() that assumes the 
Data is an RFC 3261 scheme. This character set has the property that setting bit 
6 is a no-op, except for alphabetical characters (0-9, '+', '-', and '.' all 
already have bit 6 set). Setting bit 6 on a alphabetical character is equivalent 
to lower-casing the character. Note: There is no corresponding schemeUppercase() 
function, because clearing bit 6 will convert 0-9, '+', '-', and '.' into 
unprintable characters (well, '-' is turned into a CR, but you get the point).



Performance improvements to ParseBuffer
=======================================

- Most functions that returned a Pointer now return a much more lightweight
   CurrentPosition object.
- Allow some of the simpler functions to be inlined
- Integer parsing code is more efficient, and overflow detection is better


Performance enhancements to DnsUtil
===================================

- DnsUtil::inet_ntop(): For some reason, the stock system inet_ntop 
  is dreadfully inefficient on OS X. A dirt-simple hand-rolled 
  implementation was 5-6 times as fast. This is shocking. The Linux 
  implementation is plenty efficient, though, so we're using 
  preprocessor to activate the hand-rolled code.

- DnsUtil::isIpV4Address(): The implementation uses 
  sscanf(), which is pretty expensive. Hand-rolled some code that 
  is much faster.


Reduced the memory footprint associated with storing URIs
=========================================================
- Removed the AOR cacheing stuff from Uri; it was horrifically inefficient. Checking
  for staleness of the cache was nearly as expensive as regenerating the AOR from 
  scratch. Not to mention that the AOR cacheing stuff took up a whopping 148 bytes
  of space on 64-bit platforms (4 Datas, and an int).

- Reworked the host canonicalization cache to take up less space, and be faster.
  Previously, the canonicalized host was put in a separate Data. We now canonicalize
  in-place, and use a bool to denote whether canonicalization has been performed yet.
  This saves us 32 bytes.

- Changed Data Uri::mEmbeddedHeadersText to an auto_ptr<>, since in most cases Uris don't
  use it. Also use auto_ptr for mEmbeddedHeaders (was already a pointer, for consistency).


Change how branch parameters are encoded.
=========================================

Old format: z9hG4bK-d8754z-<branch>-<transportseq>-<clientData>-<sigcompCompartment>-d8754z-

New Format: z9hG4bK-524287-<transportseq>-<clientData>-<sigcompComprtment>-<branch>

This format encodes faster, parses faster (with _much_ simpler code), and takes up
less space on the wire. We may decide to tweak the new resip cookie; I chose 
something that we can use memcmp instead of strncasecmp with, but the token character
set has a bunch of characters that aren't alphanumeric we could use.

Also, some other small optimizations; avoid copies associated with calling
Data::base64encode()/base64decode() on empty Datas, and reorder the SIP cookie
comparisons to be more efficient.


State shedding modifications to TransactionState
================================================
In a number of cases, we were preserving state (in the form of SipMessages
and DnsResults) in cases where we did not really need them any more. For
example, once we have transmitted a response, there is no need
to preserve the full SipMessage for this response (the raw retransmit buffer
is sufficient). Also, INVITE requests do not need to be maintained once
a final response comes in (since there is no possibility that we'll need to
send a simulated 408 or 503 to the TU, nor will we need to construct a CANCEL
request using the INVITE, nor will we need to retransmit). Similarly, once we
have received a final response for a NIT transaction, we no longer need to
maintain the original request or the retransmit buffer. Lastly, if we are
using a reliable transport, we do not need to maintain retransmit buffers
(although we may need to maintain full original requests for simulated
responses and such).

This change has basically no impact on reliable NIT performance, but a huge
impact on non-reliable and INVITE performance. Prior to this change, either
NIT UDP or INVITE TCP testStack would exhaust main memory on my laptop (with
4GB of main memory), bringing progress to a complete halt on runs longer than
15 seconds or so. I did not bother trying INVITE UDP, but that works now too.


Reduction in buffer reallocations while encoding a SipMessage
=============================================================
TransportSelector now keeps a moving average of the outgoing message size,
which is used to preallocate the buffers into which SipMessages are encoded.

This ends up making a small difference in testStack when linked against google
malloc, but a larger difference when linked against OS X's (horrible) standard
malloc.


Multiple Threads in the Stack
=============================
Allow transaction processing, transport processing, and DNS processing to be 
broken off into separate threads.

- SipStack::run() causes the creation and run of three threads; a 
TransactionControllerThread, and TransportSelectorThread, and a DnsThread. You 
continue to use stuff like StackThread and EventStackThread to give cycles to 
the rest of the stack (mainly processing app timers and statistics logging); the 
SipStack is smart enough to unhook these three things from the normal event loop 
API when they have their own threads. In other words, to use the new 
multi-threaded mode, all you have to do is throw in a call to SipStack::run() 
before you fire up your normal SipStack processing, and a 
SipStack::shutdownAndJoinThreads() when you're done.

- In the Connection read/write code, process reads/writes until EAGAIN, or we 
run out of stuff to send. Gives a healthy performance boost on connection-based 
transports.

- In TransactionController, put transaction timers in their own fifo. This 
prevents timers from firing late when the state machine fifo gets congested. 
Also, process at most 16 TransactionMessages from the state machine fifo at a 
time, to prevent starving other parts of the system.

- Unhook the TransactionController's processing loop from that of the 
TransportSelector. This simplifies this API considerably, but required the 
addition of a new feature to Fifo. Fifo can now take an (optional) 
AsyncProcessHandler* that will be notified when the fifo goes from empty to 
non-empty. Actually pretty useful.

- Allow setPollGrp() to be called multiple times on the various classes that 
have this function. This allows the FdPollGrp to be re-set when the SipStack 
enters multithreaded mode.

- Added a "multithreadedstack" --thread-type option to testStack. Exercise this 
option in testStackStd.sh

- Added the ability to run any of the existing Transport objects in their own 
thread, by a combination of a new transport flag 
(RESIP_TRANSPORT_FLAG_OWNTHREAD), and a new TransportThread class. Added support 
for this mode to testStack using the --tf option. Also exercised this feature in 
testStackStd.sh.

- Installed SelectInterruptors at the TransportSelector, each Transport object, 
and the DnsStub (this last one required moving SelectInterruptor to rutil). This 
is critical to making multithreaded mode work in a performant manner, and 
imposes almost no performance penalty due to the way they are invoked.

- SipStack now creates its own SelectInterruptor if one is not supplied 
externally. This is because it is critical to be able to wake the 
TransactionController up when new work comes down from the TU, or from the 
transports.


New congestion-management framework
===================================
Notable features include:
* Allow testStack, tfm/repro/sanityTests, and repro to be run with a congestion 
   manager with the --use-congestion-manager flag.

* Efficient wait-time estimation in AbstractFifo; keeps track of how rapidly
   messages are consumed, allowing good estimates of how long a new message will
   take to be serviced. More efficient than the time-depth logic in 
   TimeLimitFifo, and a better predictor too.

* The ability to shed load at the transport level when the TransactionController
   is congested, in a very efficient manner, using new functionality in Helper
   and SipMessage (Helper::makeRawResponse() and 
   SipMessage::encodeSingleHeader())

* The ability to shed load coming from the TU when the TransactionController is 
   congested. This is crucial when congestion is being caused by a TU trying to 
   do too much.

* Changed the way load-shedding is handled for TransactionUsers to use the new
   API

* A flexible congestion-management API, allowing load-shedding decisions to be
   made in an arbitrary fashion.

* A generalized CongestionManager implementation that is powerful enough to be
   useful.

* The TransactionController will now defer retransmissions of requests if 
   sufficiently congested (ie; the response is probably stuck in mStateMacFifo)

* The TransactionController now determines its hostname with a single call to 
   DnsUtil::getLocalHostName() on construction, for use in 503s. Previously, it 
   would make this call every time a 503 was sent; this call blocks sometimes!

* Don't call DnsResult::blacklistLast() on a Retry-After: 0

* Several fixes the the processing loop in testStack that were causing 
   starvation of one type of work or another when congestion occurred.


Other Misc Enhancements
=======================
-Small efficiency improvement in Random::getCryptoRandom(int)
 Random::getCryptoRandom(unsigned int len) was implemented by calling 
 Random::getCryptoRandom() repeatedly, and collecting the return values 
 in a buffer. In the openssl case, we now use a single call to RAND_bytes().
-Use a priority_queue instead of a multiset for storing timers.
-Slight refactoring of Timer so that transaction timers and payload timers (ie; 
 timers that carry a Message*) are separate classes. Transaction timers no longer 
 have an unused Message* member, and payload timers no longer have the unused 
 transaction-id, Timer::Type, and duration. This saves a _lot_ of memory for apps 
 that use lots of app timers with long lifetimes.
-Less wasteful population of Call-IDs: 
 When generating Call-IDs, Helper was computing an md5 hash of the hostname and 
 some salt, hex-encoding it, and then Base64 encoding the hex data. We now Base64 
 encode the md5 hash directly. This is less computationally expensive, requires 
 less memory because the resulting string is half the size, and requires fewer 
 bytes on the wire.
-Make TransactionMap case-insensitive; Data::caseInsensitiveTokenHash() is fast
 enough that performance actually increases a little.
-std::bitset-based parsing in a number of places.
-Don't check whether the encoding tables are initted for every single
 character; check once before the encode operation begins. Also, checking
 the value of a static bool to determine whether an init has been carried
 out is pointless; that bool might not be initted yet, and it could have
 any value. The static init code now copes with both accesses to the encoding
 tables during static initialization, and from multiple threads during runtime.
-Don't bother generating a transaction identifier unless the parse fails
 to extract one.
-Some refactoring of the FdPollGrp stuff. Now is compatible with cares, using
 a bit of a hack. Also compatible with being driven with the old buildFdSet()/
 select()/process(FdSet&) call sequence, although this is now deprecated.
 Fixing these compatibility problems allowed us to switch over to using FdPollGrp
 in all cases, instead of having dual mode everywhere.
-Buffer classes for Fifo to reduce lock contention. Using them in a few places, will
 use them in more once we phase out TimeLimitFifo with the new congestion management
 code.
-Use the --ignore-case option for generation of ParameterHash.cxx, instead of the
 nasty sed rewriting we are using now. Should also be slightly faster, since gperf
 handles case-insensitive hashing more efficiently than our hack was.
-Adding a local memory pool to SipMessage, to cut down (dramatically) on
 heap allocation overhead. Some minor refactoring to free up wasted space
 in SipMessage as well (makes more room for the pool). Changing the way
 the start-line is stored to no longer use a full-blown ParserContainer+
 HeaderFieldValueList. Lots of opportunistic doxygen merging.
 Up to 20K NIT transactions per second on my machine, roughly a doubling
 in performance.


Bug Fixes
=========
-Use getaddrinfo() instead of the non-threadsafe gethostbyname().
-Remove unused (and non-threadsafe) Timer::mTimerCount/Timer::mId.
 Previously, all timers were assigned a "unique" (not really, more on that in a 
 moment) integer identifier. There is no place in the resip codebase that 
 actually uses this identifier in any way. For transaction timers, this 
 identifier is in principle unnecessary, since there is more than sufficient 
 identifier information present already (the transaction id and timer type). When 
 passing a Message* into the timer queue, a unique identifier already exists; the 
 Message* itself (if potential use of this Message* bugs you, you can always turn 
 it into a handle by applying some sort of transformation to it). This identifier 
 is unnecessary in every case I can think of. In addition, the values are 
 assigned simply by incrementing a global variable (Timer::mTimerCount), with no 
 threadsafety measures whatsoever, so it is not even guaranteed to be unique. 
 Because of all this, it has been removed. As a bonus, this saves some memory; 8 
 bytes per timer on 64-bit platforms, which adds up to around 3MB when testStack 
 steady state has close to 400000 timers in the timer queues at any given point. 
 This could be an even larger amount for TUs that schedule lots of long-lifetime 
 timers (Timer C, for instance).
-Get rid of a wasteful double-encode, in Message.cxx
-minor windows build fixes to avoid file in use errors when building dum test projects
-fix testDigestAuthentiation for recent lowercase nonce change
-fixed a nasty bug in NameAddr - where unknown parameters uri parameters on a 
 NameAddr/Uri with no angle brackets are treated as NameAddr parameters.  When this is
 done, the memory for these parameters was only a temporary Data object.
-fix bug in Data.  If Data is wrapping memory allocated externally (ie. Share mode = BORROW)
 and you start appending to it.  It is possible that the append method will write a NULL
 character off the end of the buffer.  Changed the resize condition to make the buffer
 larger 1 character sooner, to accommodate for this.

Revision 9289 - (view) (download) (annotate) - [select for diffs]
Modified Wed Sep 28 19:14:23 2011 UTC (8 years, 1 month ago) by sgodin
File length: 8134 byte(s)
Diff to previous 9286
-disable IPv6 as default in repro.config

Revision 9286 - (view) (download) (annotate) - [select for diffs]
Added Wed Sep 28 17:43:30 2011 UTC (8 years, 1 month ago) by sgodin
File length: 8133 byte(s)
-merge changes from branch b-repro-enh-20110914
 -adding new configuration mechanism for repro
  - removed use of popt
  - using a new name/value pair approach
  - allow reading of settings from a .config file
  - allows command line options - but is not backwards compatible with old popt command line format
  - added resip ExternalLogger to repro, so that Error logs are also 
    logged to the console when file logging is enabled
  - added some new configuration settings:  LogFileName, LogFileMaxBytes
 -updated TFM for changes to repro
 -updated windows projects for TFM from VS v7.1 to v9 (VS2008)

This form allows you to request diffs between any two revisions of this file. For each of the two "sides" of the diff, enter a numeric revision.

  Diffs between and
  Type of Diff should be a

webmaster AT resiprocate DOT org
ViewVC Help
Powered by ViewVC 1.1.27