Performance tweak to the logging routines.

_osrfLogToFile() is no longer a variadic function.  We always sent it the
same format string, and there was no point in sending a variable-length
parameter list that never actually varied in length.

Now we send it a fixed-length parameter list -- and thereby avoid two
calls to vsnprintf() that had been hidden in the VA_LIST_TO_STRING macro.

Also: added doxygen-style comments to several of the functions.

M    src/libopensrf/log.c

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

Fix a bug in md5sum() (which only affected code compiled
with debugging turned on).

md5sum() builds an md5 message digest in a buffer.  Originally
it used memset() to initialize the buffer with binary zeroes.
At some point the call to memset() was replaced with the
osrf_clearbuf() macro.  When compiled in debugging mode,
osrf_clearbuf() fills the buffer with exclamation points;
otherwise it reverts to the original memset().

In this case the use of osrf_clearbuf is inappropriate, because
we use strcat() to build the message digest, two bytes at a
time.  We don't need to use memset(), but the first byte needs
to be initialized to a nul byte so that strcat() will work as
intended.  Hence:

1. Remove the call to osrf_clearbuf().

2. Put a nul byte at the beginning of the buffer.

Also, I made the buffer smaller.  There's no reason for it
to be 256 bytes long.

M    src/libopensrf/utils.c

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

Small performance tweak.

Replaced two occurrences of jsonParseString( "[]" ) with the
equivalant (and faster) call to jsonNewObject( JSON_ARRAY ).

M    src/libopensrf/osrf_application.c
M    src/srfsh/srfsh.c

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

Performance tweak to message handling.

In five locations in osrf_message.c, we were turning a jsonObject into
a JSON string, only to parse it again back into a jsonObject:

-- osrf_message_add_object_param()
-- osrfMessageToJSON() (two occurrences)
-- osrf_message_deserialize() (two occurrences)

That's silly.

This new version copies the original jsonObject directly, with no round
trip through a JSON string.  We use jsonObjectDecodeClass() to do the
copy, in order to make sure that all class hints are decoded into
classnames.

(Until recently, jsonObjectDecodeClass() would remove classnames that were
already present.  That's presumably the reason for the curious and
inefficient two-step procedure that we were using up till now.)

Also: In two locations (in osrf_message_add_object_param() and
osrf_message_add_param())  We used jsonParseString() to parse a
hard-coded "[]".  I replaced those calls with equivalent (and faster)
calls to jsonNewObjectType( JSON_ARRAY );

Also: in osrf_message_set_result_content() I eliminated a pointless test
for nullity of the json_string variable.  An earlier test already
guarantees that json_string is not NULL.

M    src/libopensrf/osrf_message.c

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

Created a new function osrfHashExtract().  It extracts an item with a
given key from an osrfHash, without destroying it, leaving the rest
of the osrfHash intact.

M    include/opensrf/osrf_hash.h
M    src/libopensrf/osrf_hash.c

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

Tweak jsonObjectDecodeClass so that it preserves classnames
that are already present.  The previous version would add
new classnames, if it found them encoded, but would drop
any old ones.

At present this change will have no effect.  Apart from a
couple of recursive calls, the only function that calls
jsonObjectDecodeClass() is jsonParseString().  In that case,
we pass a freshly parsed jsonObject that can't possibly
have any classnames yet.

However this change will enable us to use this function
elsewhere (specifically in osrf_message.c), resulting in
simpler and faster code.

M    src/libopensrf/osrf_json_tools.c

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

Enhance the performance of the recursive descent JSON parser,
mainly the jsonParse() function.

1. The old version would create a jsonObject and then copy it, with
possible modifications, via a call to jsonObjectDecodeClass(), in
order to decode class hints into the classname member.  Finally, it
would throw away the original jsonObject.

The copying operation is expensive, and the new version eliminates
it.  When decoding is desired (which is nearly always), execution
passes through a parallel version of the get_hash() function, which
does the decoding on the fly.

In my benchmarking, the new version reduces the parsing time by
around 35 - 45 percent.  It is now at least twice as fast as the
older jsonParseString() function, which uses a finite state
machine instead of recursive descent.

2. In get_number(): instead of allocating and destroying a
temporary growing_buffer, use the one available in the
Parser structure.

3. In osrf_json.h: Applied some pedantic corrections to the
doxygen comments for the old parser.

--------

In all cases tested, the new version produces results identical
to those of the old version.  The results are also identical to
those of the older parser, apart from error detection.

M    include/opensrf/osrf_json.h
M    src/libopensrf/osrf_parse_json.c

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

Change the return type of jsonObjectGetString so that it
is a pointer to const char, instead of to non-const char.

We don't want the calling code to be able to modify the innards
of the jsonObject, at least not by this back door.

I have already examined all uses of this function and modified
them where necessary to avoid compile problems.

M    include/opensrf/osrf_json.h
M    src/libopensrf/osrf_json_object.c

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

1. Add doxygen-style comments to document functions in the
recursive-descent JSON parser.  Tidy up or correct existing
comments here and there.

2. In report_error(): add the const qualifier to the
third parameter.

M    include/opensrf/osrf_json.h
M    src/libopensrf/osrf_parse_json.c

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

We routinely grab a chunk of configuration file that doesn't
apply to the router process.  When we detect the spurious hit,
we throw it away and go on to the next one.

We had been detecting the spurious hit after forking, thus
wasting a fork.

Now we detect the spurious hit before forking, so that we
don't waste a fork.

M    src/router/osrf_router_main.c

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

Comment out an error message added in the previous patch.

The error message reported what looks like an error condition:
the config file doesn't provide transport info.  However what
appears to be happening is that the program tries to read
several different sections of the config file, of which some
are relevant and some aren't, due to some overloading of the
<routers> tag.  As a result it routinely spawns irrelevant
children, only to watch them die.

There's got to be a better way, but for now I'll just
suppress the error message.

M    src/router/osrf_router_main.c

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

Let's give OpenSRF a spin on Ubuntu Karmic

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

1. Eliminated some memory leaks by replacing calls to
jsonObjectFindPath() (whose return values weren't getting
freed) with calls to jsonObjectGetKey().  This change will
also speed up the configuration a bit by avoiding the
cloning of objects.

2. Changed setupRouter so that it returns void instead of int.
We weren't looking at the return value anyway.  Since the
function normally enters an infinite loop, any return
represents some kind of error.

3. If the configuration file has no transport information,
issue a message before exiting (to standard error, since we
don't have a log file yet).

4. Add doxygen-style comments for documentation.

M    src/router/osrf_router_main.c

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

1. Changed osrfLogFacilityToInt() so that it accepts
a const pointer.

2. Added the const qualifier to various variables.

3. In osrf_router_main.c: Removed three inappropriate
calls to free().  Some memory leaks remain, where we
fetch some cloned jsonObjects from jsonObjectFindPath()
and don't free them.

M    include/opensrf/log.h
M    src/router/osrf_router_main.c
M    src/gateway/osrf_http_translator.c
M    src/libopensrf/osrf_prefork.c
M    src/libopensrf/log.c
M    src/libopensrf/osrf_application.c

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

And place headers in /prefix/include/opensrf/ again, fixing regression to /prefix/include/

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

W h i t e s p a c e matters when defining automake targets

Headers weren't being installed as the include_HEADER target wasn't being parsed.

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

1. Add doxygen markup for documentation.

2. In jsonNewObjectType(): explicitly initialize a JSON_BOOL to false, instead
of implicitly relying on the expectation that a NULL pointer is represented
by all-bits-zero.

3. In jsonObjectExtractIndex(): set the parent pointer to NULL in the
extracted jsonObject.

M    include/opensrf/osrf_json.h
M    src/libopensrf/osrf_json_object.c

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

LogServer was removed from distro but not from MANIFEST - fixed

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

correct the test for modifying the *.xml.example files

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

POD/docs cleanup

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

100% test coverage

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

WIP

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

WIP

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

more test

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

sync

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

stowing WIP

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

WIP

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

Make building & installing the core components of OpenSRF optional.

For now, this will be of most interest to those interested in installing only
the JavaScript files from OpenSRF. If you disable the core components in
configure, then dependency checking for those core components is skipped and
the JavaScript files will be installed to the desired location.

For example:

./autogen.sh
./configure --prefix=/openils --sysconfdir=/openils/conf --disable-core
sudo make install

... will result in only the OpenSRF JavaScript files being installed in
/openils/lib/javascript/ (although at the moment, the other directories
will still be created).

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

Changed the signature of osrfStringArrayGetString():

1. It receives a pointer to osrfStringArray.  That pointer
is now a pointer to const.

2. It returns a character pointer.  That pointer is now a
pointer to const.

I have already examined all existing calls to this function,
and modified them where necessary.

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

In osrf_json_gateway.c: added the const qualifier to three
different local variables, all named "str".

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

In apacheGetParamValues() and apacheGetFirstParamValue():
applied const qualifier to nkey.

Also: changed the ensuing NULL tests to look at nkey
instead of key, since key has already been verified as
non-NULL.

Actually it looks like nkey is guaranteed to be non-NULL
as well, but that depends on the correct functioning
of osrfStringArray; so I left the NULL test, as
revised, in place.

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

In osrfAppSessionClientInit(): added a const
qualifier to domain.

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

In osrfSystemBootstrap(): added a const qualifier
to appname.

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

In osrfBigHashFree: added a const qualifier to key,
and removed a pointless cast.

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

In osrfStringArrayRemove(): fixed a bug whereby we would decrement
the size member even if no string was actually removed.

Added Doxygen-style comments to document everything.

Removed a few comments in the header so that they wouldn't
override comments in the implementation file.

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

In osrfNewHash(): specify a size for the osrfList used as a hash table,
so as to avoid wasting memory.

In osrfHashSet(): rearranged the logic a bit for clarity; no change in
behavior.

In osrfHashIteratorNext(): added a bit of protection against a corrupted
iterator.

Throughout: added Doxygen-style comments for documentation.  Removed
some comments from the header so that they wouldn't override more
complete comments from the implementation file.

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

initial support for capturing bytes read from gateway response

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

Use working Perl syntax in example print hostname command.

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

Add a linefeed to the Perl "print hostname" command

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

1. Moved the declaration of the osrf_host_config struct out of the header.
Nothing outside of osrf_settings.c needs access to any members of this
struct.

2. Made the osrf_settings_new_host_config function static; removed its
prototype from the header.

3. Made the "config" pointer static.

4. Rearranged osrf_settings_free_host_config a bit to protect against
attempts to free the cached config twice.

5. Finished adding doxygen-style markup comments.

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

Tweaking comments for doxygen:

Moved the overview from the implementation file to the header.

Deleted some comments from the header so that they wouldn't
override more complete comments from the implementation file.

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

Enhanced the error messages from osrf_settings_host_value() and
osrf_settings_host_value_object() to include the requested
search path.  That should make it easier to identify the caller
and thereby figure out what went wrong.

Also: added a couple of Doxygen-style comments.

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

for consistency with other config files and ease of install automation, set the default password to 'password'

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

assume 'yes' to fetching dependencies; reduce the verbosity of package downloads

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

1. In osrfListSetDefault(): install the standard function
free() as the default item-freeing callback function,
instead of the gossamer-thin and useless wrapper for it,
osrfListVanillaFree().  This change eliminates a layer of
function call overhead.

2. Eliminate osrfListVanillaFree() as neither used nor useful.

3. Add doxygen-style comments to document every function.

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

1. A minor streamlining of va_list_to_string(), to eliminate
a layer of copying.

2. Corrected some faulty comments for doxygen.

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

repaired partial change from 'class' to 'pkg'.  added use strict/warnings

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

1. Add comments to be recognized by doxygen or other such
documentation generators.

2. Change the values returned by buffer_fadd(), buffer_add(),
buffer_add_n(), and buffer_add_char() to make them more
consistent.  In practice we never pay any attention to the
return values anyway.

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

changed 'my' globals to 'our' globals for testing purposes

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

first tests

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

more cleanup and documentation in progress

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

yanking LogServer test file

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

removing use of LogServer

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

removing _json_hint_to_class

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

shuffling around, adding POD stubs

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

removing OSRF::Utils::LogServer at berick's suggestion

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

rearranging things a bit, docs. no actual code changes.

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

new test files for Utils breakout

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

removed $VERSION construct which caused undef warnings when not under svn, and only occurred in 2 modules

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

method prettyJSON doesnt exist anymore

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

ditto

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

that last version change was rather poorly timed, as it turns out

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

local doc updates for recent changes. wondering how much point there is in keeping these around, long-term

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

preëmptive version bump

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

'make check' works as promised. perl tests moved to 'check' target

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

updating MANIFEST for Module::Build-ness

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

added separate install fail diagnostic message

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

removing files no longer needed

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

'make' will now just plain make again without proceeding to install.
Added Build to files swept by 'make clean'

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

Going back to dumping a script (unbreaking non-root make vs. make install)

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

Added DISTCLEANFILES to make "make distclean" work better

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

AC_CONFIG_MACRO_DIR added at the behest of libtoolize

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

Added distclean-local: target and DISTCLEANFILES to make 'make distclean' work superawesomely
Added ACLOCAL_AMFLAGS at the behest of libtoolize

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

Build.PL built; build/test/install all selfcontained and automatic now

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

M::B changes

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

M::B changes

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

Extended the JSON_INIT_CLEAR macro to avoid segfaults.

Scenario: converting a JSON_BOOL, with a value of true, to a JSON_HASH or
JSON_ARRAY.  The true value (in a union with an osrfHash* and an osrfList*)
was being interpreted as a non_NULL pointer and deferenced.  Oops.

With this change, we clear the boolean value (by nullifying one of the
unioned pointers) whenever changing from a JSON_BOOL to anything else.

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

Performance tweak to jsonIterator.

Instead of storing a malloc'd copy of the key of a JSON_HASH entry,
just store a const pointer to the key string stored in the
internal osrfHash.  That way we don't have to do a malloc and
free every time we bump the iterator.

This change also requires the addition of a couple of const qualifiers
in the client code.

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

set the 'from' address in outbound jabber messages

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

set the 'from' address in outbound xmpp messages

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

reduce unnecessary calls to fcntl() while reading from the socket

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

check NULL-ness on hash the key before calling find_item

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

This is a performance tweak to the osrfHashGet function, a widely
used utility function.

The old version accepted a variable number of arguments: a pointer
to an osrfHash, a string optionally containing printf-style
format specifiers, and addtional parameters as needed to fill in
the blanks.

In practice, none of the code ever uses the printf-style formatting.
We always pass exactly two parameters.  We burn CPU cycles scanning
the string for format specifiers and never find any.

I eliminated the unused variable parameters and turned osrfHashGet()
into a simple two-parameter function.  Just in case anybody ever
wants it, I also cloned the original version into a new function
named osrfHashGetFmt, which accepts a variable number of arguments
as before.

Note that, since the signature of the function is changing,
it is necessary to recompile any open-ils programs that
call it, namely:

oils_dataloader.c
oils_event.c
oils_idl-core.c
oils_cstore.c
dump_idl.c

The Makefiles apparently don't recognize this dependency.

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

Add two new commands, "open" and "close", to open and close ongoing
connections to specified services.  (Otherwise a connection is created
and destroyed for every request.)

Main purpose: to be able to use srfsh for database operations that require
transactions.

Warning: typically the <keepalive> value will need to be increased for
the service in question in opensrf.xml, or else the server will close
the connection before you can type your request.

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

Fix a bug whereby, if there was only one <service> entry for the
public router in opensrf_core.xml, the service would fail to register.

Root cause: The code was expecting to see a list of services in a
JSON_ARRAY.  But if there's only one service, it's represented as a
JSON_STRING.

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

until we make the router / C processes write their own PID file, put a short sleep in after startup to give each time to show up in the 'ps' output.  this should help prevent lingering router processes after service stop

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

call cleanup instead of running the death callback directory when server session expires

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

added session callback support with example method.  added method-not-found handling.  added network socket flush for clearing stale inbound data.

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

added a session_data disctionary to ServerSession class for storing per-session data.  fixed bug where no params caused serer-side confusion.  added exmample of session_data usage to example app

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

1. If an input line ends in a backslash, remove the backslash
and append the following line as a continuation.

2. Disable the readline library's special treatment of
horizontal tabs, which by default trigger file name
completion (which is not useful for srfsh).

3. In calls to strtok(): accept horizontal tabs as
delimiters, since they are no longer suppressed
by readline().

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

teach interval_to_seconds about HH:MM:SS format time intervals

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

teach interval_to_seconds about negative intervals

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

flush_socket now flushes data through the xml stream parser to prevent the destorying the validity of the stream.  now discarding RESULT messages passed to server sessions.

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

don't try to 'kill' an empty pid / prevents warnings

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

define OSRF_HOSTNAME globally to prevent 'unbound variable' error

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

Use autotools to set the default values in opensrf-perl.pl to what was passed in to ./configure

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

Use autotools a bit more to our advantage:
  * In osrf_ctl.sh, use the configured location for osrf_config by default
  * Differentiate between prefix and exec_prefix to respect configure options
  * Make configure output slightly more consistent

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

try finding osrf_config in the users PATH first to prevent the use of 'find' when possible (find + NFS = watching paint dry)

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

opensrf.py can now read the opensrf.settings file for managing apps.  added start/stop_all actions.  support for service name syslog ident

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

until full settings support is added, comment that out for now.  added example keepalive

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