Capturing the return value of the bootstrap call and emitting a message when encountering a non-success value

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@941 9efc2488-bf62-4759-914b-345cdb29e865

Patch from Scott McKellar to clarify errors in bootstrapping of the server process

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@940 9efc2488-bf62-4759-914b-345cdb29e865

Patch from Scott McKellar with minor adjustments:

1. In an earlier patch to osrf_settings_host_value_object(), we
contrived to abort the program deliberately instead of crashing with
a segmentation violation.  The current patch applies the same
treatment to the similar function osrf_settings_host_value(),
although I don't know that anyone has ever reported a problem at
this spot.  I rearranged the error message a bit so that it would
be distinct from the other one.

2. If the config pointer is null, it's because osrf_settings_retrieve()
was unable to populate it, or was never called.  There are two ways
it might fail: it might not get an osrf_message from
osrf_app_session_request_recv(), or the osrf_message it gets might
not have the _result_content member populated.  I added a couple of
log messages to distinguish between these possibilities. Maybe they
will make diagnosis easier.

3. The latter change also plugs a small memory leak.  If we get an
osrf_message but it has no _result_content, the present code neglects
to free the osrf_message.

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@939 9efc2488-bf62-4759-914b-345cdb29e865

removed unused TEMPLATEDIR directory creation.  thanks, Dan S.

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@938 9efc2488-bf62-4759-914b-345cdb29e865

Patch from Scott McKellar:

1. I declared several functions as static, and moved their prototypes
from the header file to the implementation file.  In each case I
verified that nothing else in the code base calls it.

2. I removed the socket_send_nowait prototype from the header, and
commented it out in the implementation file.  I was tempted to
eliminate it entirely, but stayed my hand.  It is a trivial wrapper
for _socket_send(), and nothing calls it.

3. Nothing calls several other functions either.  However unlike
socket_send_nowait() they are non-trivial functions, and they appear
to be intended for external linkage, so I left them alone for
potential future use.

4. The identifier _socket_print_list is reserved for local scope,
and should not have external linkage, because it starts with an
underscore followed by a lower case letter.  I may submit a patch to
rename it some day if I ever get around to it, but that will require
a change in socket_test.c, which calls it.

5. I moved the definition of the RBUFSIZE macro from the header file
into the implmentation file.  It is an internal detail that no
other translation unit should know or care about.

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@937 9efc2488-bf62-4759-914b-345cdb29e865

Patch from Scott McKellar:

1. I made __osrfChatXMLErrorOcurred and __osrfChatClientSentDisconnect
static, and removed the leading underscores from their names.  No other
source files refer to them.

2. In osrfNewChatServer() I explicitly initialize the port member to
zero -- which is already happening implicitly due to the memset() in
safe_malloc().

3. Also in osrfNewChatServer(): the existing code populates the secret
member only if the secret parameter is not NULL, and leaves it
uninitialized otherwise.  However if the secret parameter were NULL
we would have already performed an early exit.  I populate the secret
member unconditionally, just as we do with the domain member.  I also
moved this assignment up with the other assignments, for a more
readable flow.

4. In osrfChatServerFree(), the existing code leaves several memory
references dangling, without freeing them.  I added code to free the
domain member, the deadNodes member, and finally the osrfChatServer
itself.

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@936 9efc2488-bf62-4759-914b-345cdb29e865

adding gateway parsing code.  so far, we have XML gateway handling using python minidom.

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@935 9efc2488-bf62-4759-914b-345cdb29e865

adding sample makefile and install.conf file for opensrf installation

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@934 9efc2488-bf62-4759-914b-345cdb29e865

ported XML parsing from libxml2 to Python's builtin xml.dom.minidom to reduce dependenies

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@933 9efc2488-bf62-4759-914b-345cdb29e865

Patch from Scott McKellar; cleaning up several error conditions

http://list.georgialibraries.org/pipermail/open-ils-dev/2007-June/001241.html

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@932 9efc2488-bf62-4759-914b-345cdb29e865

Patch from Scott McKellar;  Adds calls to strerror() where appropriate in
OpenSRF/src/utils/socket_bundle.c.

Applied with some additional changes to clean up log messages and correct an
instance of printf'ing a getpid as an int instead of a long.

http://list.georgialibraries.org/pipermail/open-ils-dev/2007-May/001148.html

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@930 9efc2488-bf62-4759-914b-345cdb29e865

removed some error lines printed to stderr.  not used and they also have the potential of overwriteing errno

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@929 9efc2488-bf62-4759-914b-345cdb29e865

slimmed some logging.  capturing errno locally after the recv() call so it is not replaced by a different system call before the check

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@928 9efc2488-bf62-4759-914b-345cdb29e865

no longer freeing return value from strerror since it should not be modified

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@926 9efc2488-bf62-4759-914b-345cdb29e865

dueling patches of pedantism

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@925 9efc2488-bf62-4759-914b-345cdb29e865

added some more descriptive logging.  removed old signal handling code.  no longer freeing return value from strerror since it should not be modified

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@924 9efc2488-bf62-4759-914b-345cdb29e865

Change suggested by Scott McKellar; handle c-app children death without a
signal handler.

http://list.georgialibraries.org/pipermail/open-ils-dev/2007-May/000955.html

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@923 9efc2488-bf62-4759-914b-345cdb29e865

Patch from Dan Scott; adds -rpath=$(LIBDIR) to the default LDFLAGS for
OpenSRF, obviating the need to set up non-standard system search paths
for any libs that OpenSRF loads that either don't have a lib prefix
or live outside the normal system library paths.

http://list.georgialibraries.org/pipermail/open-ils-dev/2007-May/001123.html

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@922 9efc2488-bf62-4759-914b-345cdb29e865

swapped the formats -- typo

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@921 9efc2488-bf62-4759-914b-345cdb29e865

Patch from Scott McKellar; reasonable and non-intrusive diagnostic message in
the case that we've been given on port or unix socket path to connect over.

http://list.georgialibraries.org/pipermail/open-ils-dev/2007-May/001109.html

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@920 9efc2488-bf62-4759-914b-345cdb29e865

Patch from Scott McKellar; allows the compiler to warn us of missing casts
coming from undefined safe_malloc()s, NULL-initialize pointers (\0 != NULL).

http://list.georgialibraries.org/pipermail/open-ils-dev/2007-May/001098.html

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@919 9efc2488-bf62-4759-914b-345cdb29e865

Several combined cleanup patches from Scott McKellar:

 * remove leading underscores from static vars and functions
 * make static functions, well, static
 * tighten up the definition of funcs that take 0 params

http://list.georgialibraries.org/pipermail/open-ils-dev/2007-May/001095.html
http://list.georgialibraries.org/pipermail/open-ils-dev/2007-May/001101.html
http://list.georgialibraries.org/pipermail/open-ils-dev/2007-May/001112.html

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@918 9efc2488-bf62-4759-914b-345cdb29e865

At Scott McKellar's suggestion, using (long) cast and %ld format for getpid()

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@917 9efc2488-bf62-4759-914b-345cdb29e865

list.georgialibraries.org/pipermail/open-ils-dev/2007-May/001092.html
Patch from Scott McKellar to:

 * Make default config object pointer static (module local)
 * Clean up variable name to remove leading underscores
 * Remove think-o which would deallocate the default config when
   loading any other
 * Fix error log message

Applied with minor change to the log message fix

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@916 9efc2488-bf62-4759-914b-345cdb29e865

XSD from Dan Scott for validating the ~/.srfsh.xml config file.
http://list.georgialibraries.org/pipermail/open-ils-dev/2007-May/001030.html

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@915 9efc2488-bf62-4759-914b-345cdb29e865

From Dan Scott: EOL Whitespace is now stripped in the perl config file,
hopefully saving some debugging time in the future.

Details at: http://list.georgialibraries.org/pipermail/open-ils-dev/2007-May/001049.html

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@914 9efc2488-bf62-4759-914b-345cdb29e865

ignore non-leaf text nodes

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@913 9efc2488-bf62-4759-914b-345cdb29e865

compatability functions for strnlen and strndup

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@912 9efc2488-bf62-4759-914b-345cdb29e865

moving this to strn_compat

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@911 9efc2488-bf62-4759-914b-345cdb29e865

cut-n-paste error in logged message -- thanks scottmck

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@910 9efc2488-bf62-4759-914b-345cdb29e865

strndup() and strnlen() implementations from Jeroen Ruigrok van der Werven
See bug #551 for original details -- http://open-ils.org/cgi-bin/bugzilla/show_bug.cgi?id=551

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@909 9efc2488-bf62-4759-914b-345cdb29e865

reordering breakage; const qualifier in header

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@908 9efc2488-bf62-4759-914b-345cdb29e865

Failure cleanups suggested by Scott McKellar; modified from patch at
http://list.georgialibraries.org/pipermail/open-ils-dev/2007-May/001006.html

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@907 9efc2488-bf62-4759-914b-345cdb29e865

const-ification

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@906 9efc2488-bf62-4759-914b-345cdb29e865

Correcting previous comment.  The right mailing list pointer is at:
http://list.georgialibraries.org/pipermail/open-ils-dev/2007-May/001002.html

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@905 9efc2488-bf62-4759-914b-345cdb29e865

stderr output cleanup
Patch supplied by Scott McKellar
http://list.georgialibraries.org/pipermail/open-ils-dev/2007-May/001006.html

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@904 9efc2488-bf62-4759-914b-345cdb29e865

added some inline comments

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@903 9efc2488-bf62-4759-914b-345cdb29e865

added math bench code.  added shutdown method to disconnect from jabber.  updated makefile to used fetched version of the stax jar

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@902 9efc2488-bf62-4759-914b-345cdb29e865

changed log level on what really should not be a warning message

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@901 9efc2488-bf62-4759-914b-345cdb29e865

returning OK status in respond method, instead of no status at all.  most code uses respond_complete, which is why this has not been an issue in the past.  writing new client libs exposed it, though

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@900 9efc2488-bf62-4759-914b-345cdb29e865

broke network-object logic out of the JSON parsing code

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@899 9efc2488-bf62-4759-914b-345cdb29e865

committing scott's safer MALLOC macro and a new buffer_release function patches

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@898 9efc2488-bf62-4759-914b-345cdb29e865

added a make target to sample makefile for fetching jar/source dependencies to ease deployment

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@897 9efc2488-bf62-4759-914b-345cdb29e865

rethinking deployment

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@896 9efc2488-bf62-4759-914b-345cdb29e865

updating to jdk1.5 build

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@895 9efc2488-bf62-4759-914b-345cdb29e865

added memcache lib to makefile

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@894 9efc2488-bf62-4759-914b-345cdb29e865

committing added jars

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@893 9efc2488-bf62-4759-914b-345cdb29e865

added a memcache client and test

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@892 9efc2488-bf62-4759-914b-345cdb29e865

added a settings server client and test.  made the global config object wrapped in a method instead of using static methods all around

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@891 9efc2488-bf62-4759-914b-345cdb29e865

implemented client bootstrap.  created a status object to model connection statuses.  remodeled the client test code to run arbitrary methods

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@890 9efc2488-bf62-4759-914b-345cdb29e865

fixed some doc strings.  added sample makefile

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@886 9efc2488-bf62-4759-914b-345cdb29e865

added a lot of documentation

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@885 9efc2488-bf62-4759-914b-345cdb29e865

implemented enough of the stack/session logic to make stateless requests and receive responses

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@884 9efc2488-bf62-4759-914b-345cdb29e865

adding more base objects, started on stack layer. added config parser

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@883 9efc2488-bf62-4759-914b-345cdb29e865

added JSON reading ability, including serializable objects

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@882 9efc2488-bf62-4759-914b-345cdb29e865

added support for opensrf-serializable objects and JSON output of those objects. updated some JSON test code

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@881 9efc2488-bf62-4759-914b-345cdb29e865

added initial opensrf stack layer objects.  more tuning/testing of the jabber layer

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@880 9efc2488-bf62-4759-914b-345cdb29e865

printf family format fixup found (har har) and suggested by Scott McKellar -- %lf -> %f
(no need for the precision and/or scale of %Lf)

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@879 9efc2488-bf62-4759-914b-345cdb29e865

fixed some logic bugs with the body parsing, added some comments

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@878 9efc2488-bf62-4759-914b-345cdb29e865

added the woodstox stax jar files

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@877 9efc2488-bf62-4759-914b-345cdb29e865

added java jabber layer

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@876 9efc2488-bf62-4759-914b-345cdb29e865

added some more index sanity checks.  added const qualifier where necessary

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@875 9efc2488-bf62-4759-914b-345cdb29e865

added size check to constructor

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@874 9efc2488-bf62-4759-914b-345cdb29e865

committing scott m's buffer protection patch

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@872 9efc2488-bf62-4759-914b-345cdb29e865

Exiting with a usable error message rather than segfaulting
 - patch provided by Scott McKellar
 - http://list.georgialibraries.org/pipermail/open-ils-dev/2007-April/000727.html

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@866 9efc2488-bf62-4759-914b-345cdb29e865

Avoiding memory leaks in string opperations
  - patch provided by Scott McKellar
  - http://list.georgialibraries.org/pipermail/open-ils-dev/2007-April/000724.html

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@864 9efc2488-bf62-4759-914b-345cdb29e865

completely untested and possibly broken settings extention to retrieve defaults when host specific fails

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@863 9efc2488-bf62-4759-914b-345cdb29e865

made new child creation more robust

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@862 9efc2488-bf62-4759-914b-345cdb29e865

Typo reported by Dan Scott

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@860 9efc2488-bf62-4759-914b-345cdb29e865

removed some old code and the unnecessary select-as-sleep calls

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@859 9efc2488-bf62-4759-914b-345cdb29e865

applied patch from Dan to provide an example usage

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@858 9efc2488-bf62-4759-914b-345cdb29e865

forcing int on timeout values in process and timed_read

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@853 9efc2488-bf62-4759-914b-345cdb29e865

added an info log line to log the called method

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@852 9efc2488-bf62-4759-914b-345cdb29e865

removing install of deprecated files

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@850 9efc2488-bf62-4759-914b-345cdb29e865

removing deprecated files

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@849 9efc2488-bf62-4759-914b-345cdb29e865

removed the deprecated rest_gateway config; added comment on router name changes

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@848 9efc2488-bf62-4759-914b-345cdb29e865

added "use Net::Domain" in the package that calls hostfqdn()

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@847 9efc2488-bf62-4759-914b-345cdb29e865

spacing adjustment

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@846 9efc2488-bf62-4759-914b-345cdb29e865

adjusting comment in example config re bug #518

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@845 9efc2488-bf62-4759-914b-345cdb29e865

using Net::Domain instead of the shell to get the fqdn of the current host

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@844 9efc2488-bf62-4759-914b-345cdb29e865

added more portable (perl) hostname fetching to pass to the C code, updated docs

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@843 9efc2488-bf62-4759-914b-345cdb29e865

lowered some logging

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@841 9efc2488-bf62-4759-914b-345cdb29e865

refactored the unix::server config handling by getting rid of the old temp config files
and passing config params directly to the net::server run() method

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@840 9efc2488-bf62-4759-914b-345cdb29e865

applying spelling fixes and making db names/passwords match accross default config files thanks to patch by Dan Scott.  Thanks, Dan!

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@838 9efc2488-bf62-4759-914b-345cdb29e865

removing unused log section

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@832 9efc2488-bf62-4759-914b-345cdb29e865

added the <client> block to the gateway.  moving everything to "localhost" for consistency

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@831 9efc2488-bf62-4759-914b-345cdb29e865

do a kill -9 after kill -s INT to make sure no stubborn processes stick around

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@830 9efc2488-bf62-4759-914b-345cdb29e865

removing old, unused child_finish_hook

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@822 9efc2488-bf62-4759-914b-345cdb29e865

added child exit handling code and example

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@821 9efc2488-bf62-4759-914b-345cdb29e865

fixed bool parsing bug -- off by one on the string size enforcement

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@817 9efc2488-bf62-4759-914b-345cdb29e865

added a is-true wrapper for bool objects

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@816 9efc2488-bf62-4759-914b-345cdb29e865

added timeout url param to set the request timeout

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@815 9efc2488-bf62-4759-914b-345cdb29e865

Added support for the opensrf XID, which is a transaction string that's passed
around via the jabber message and inserted into each logged message to relate
activity accross different processes

implemented a child onExit handler for the C stack for cleaning up db connections, etc.

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@814 9efc2488-bf62-4759-914b-345cdb29e865

removed libjudy dependencies from makefiles

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@813 9efc2488-bf62-4759-914b-345cdb29e865

reduced hash size (just now comitting, been like this for a while on dev)

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@812 9efc2488-bf62-4759-914b-345cdb29e865

adding python libs

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@811 9efc2488-bf62-4759-914b-345cdb29e865

printf format fix

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@809 9efc2488-bf62-4759-914b-345cdb29e865

any unhandled char below ascii 32 will now be encoded as utf8 (\uXXXX)

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@808 9efc2488-bf62-4759-914b-345cdb29e865

removed big hash/array dependencies to remove libjudy requirements

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@807 9efc2488-bf62-4759-914b-345cdb29e865

replaced (printf style) format buffer calls with non-format calls to prevent issues with embedded % symbols

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@806 9efc2488-bf62-4759-914b-345cdb29e865