Patch from Scott McKellar; use camelCase instead of under_score function names

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

Patch from Scott McKellar; use camelCase instead of under_score function names

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

Patch from Scott McKellar; use camelCase instead of under_score function names

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

Patch from Scott McKellar which adds some const-correctness.

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

added a static method to see if there is already a globally connected cache client

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

repaired stderr call

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

forcing int-ness on timeout value

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

changed osrf python version from 1.2. to 1.0 to match opensrf

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

Use distutils to install Python into the standard location
Correct a syntax error in http_translator.py

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

added a flag to encode/decode flattened data as json

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

added a multisession class for agregating requets into a single recv manager

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

added an xml flattener similar to the java xml flattener

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

added internal log level.  added some crucial log lines.  logging thread id

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

Continue the march towards a pedantic 1.0 python API.

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

added iscomplete method

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

parsing logfile settings and initing logger

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

Add rudimentary locale support to srfsh.py

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

adding a basic mutlisession manager

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

Add some basic locale support.
Revert a dailyWTF that caused massive CPU & memory consumption.

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

Further pedanticize logging method names.

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

Move towards Pythonic API style conventions (as informed by pylint)

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

created a simplified XSL processing class with test

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

xml.py conflicts with other xml module namespace (boo). Renaming.

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

This file will hold only OpenSRF XML functions.
osrfObjectFindPath will be moved to the net_obj module.

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

adding a generic xml flattening utility with test

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

Patch from Scott McKellar to allow E as well as e for scientific notation

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

Patch from Scott McKellar to correct some problems with _jsonParserError(),
which constructs and issues a message about a parsing error:

The problems arise in the course of extracting a fragment of JSON text to
provide the context of the error.

1. The code starts by picking the beginning and end of the fragment to
extract.  In order to avoid beginning before the start of the string,
the original code goes through a loop, incrementing an index until it
is non-negative.  A similar loop corrects for an ending beyond the
end of the string.

These loops do the job, but to my eyes they look silly.  I replaced
them by assigning the corrected values directly, when corrections
are in order.

2. There is an off-by-two error in calculating the size of the buffer
needed to hold the fragment.  To begin with, we miscalculate the
length of the fragment.  If the fragment extends from character 30
through character 40, there are 11 characters in the fragment, not
10.  Then we neglect to add another byte for a terminal nul.

The result is that the last two characters in the intended fragment
are not displayed.  If the character in error is the last or the
next to last character in the string, it doesn't get displayed as
part of the fragment, leading to likely bafflement.

I corrected both these errors, embiggening the buffer by two.

3. The original code copies the fragment into the buffer by calling
snprintf().  Besides being needlessly inefficient, snprintf() is
dangerous in this context.  If the copied fragment contains a
format specifier such as "%s" or "%d", sprintf goes off looking for
a non-existent parameter, resulting in a mangled message or worse.

I replaced the snprintf() with a memcpy() and a terminal nul.

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

Patch from Scott McKellar to speed up parts of the growing_buffer API; return values from buffer add/reset API brought into a consistent state for proper return value checks

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

Patch from Scott McKellar to clean up const-iness and simplify the parser internals

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

arg ... add tracking in both "new" functions

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

tracking actual list length based on adds and uses

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

track malloc()-created objects

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

Patch from Scott McKellar which adds a free-object-pool for jsonObjects, reducing malloc/free overhead by a large degree

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

created external script for downloading java dependencies.  added new install.conf settings for where dependency jars are kept and whether or not to build the java libs

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

Minor code cleanup:
Avoid stepping on str's toes by renaming the variable 'str' to string.
Explicitly call osrf.json and str methods rather than polluting global name space.

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

Added support for "migratable" opensrf services.

See http://list.georgialibraries.org/pipermail/open-ils-dev/2007-October/001904.html

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

returning if config is not defined since next line of code tries to use config

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

removed unnecessary log line

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

parsing requests so the translator can return OK and stop waiting on DISCONNECT messages.  added some debug logging.

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

removing confusing dead code

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

added cache delete.  updated some logging

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

added the proof-of-concept http multipart translation code.  still needs request parsing to handle client disconnect messages and more testing with non-multipart

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

added a default timeout cache setting

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

adding a caching api.  requires memcache: ftp://ftp.tummy.com/pub/python-memcached/

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

added error message. re-tabbed to 4 spaces

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

removed debug statement

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

in addition to pushing the received message to the callback, we return the message from recv()

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

Patch from Scott McKellar:

1. Shift toward a camel case style of identifiers.

2. Make functions static when they are not called from elsewhere.

3. Populate all members in newly-allocated structs.

4. Add a couple of consts.

Details:

1. We have two redundant typdefs. I replaced all references to the
lower_case_with_underscores version with references to the camelCase
versions (while keeping the obsolescent typedefs themselves):

   osrf_app_request ==> osrfAppRequest
   osrf_app_session ==> osrfAppSession

2. I deleted the prototypes of two functions that are nowhere
referenced or even defined:

   _osrf_app_session_get_request()
   _osrf_app_session_push_request()

3. I eliminated osrf_app_session_make_locale_req(), replacing it
with the equivalent osrfAppSessionMakeLocaleRequest function.  No
other file references the former.

4. I made the following functions static, and removed their
prototypes from the header, since none is referenced from any other
file:

   osrfAppSessionMakeLocaleRequest()
   osrfAppSessionSendBatch()
   all remaining functions with a leading underscore

5. I explicitly initialized the stateless and session_locale members
of osrfAppSession.

6. I added the const qualifier to a couple of parameters of
osrfAppSessionStatus().

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

removing out-of-date jabber2d patches

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

added ability to parse attributes and elements containing no children or data to the xml2object parser

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

replaced str() calls with unicode()

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

repaired variable name

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

removing some debug logging

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

propogate locale from server to client

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

thinko, sorry

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

Cleaned up exception messages so they will not be bloated with newlines
(which are rendered as '\n' in syslog and local apache logs, anyway) and
layers of "Mess:" tags.  Logging the ISO timestamp instead of the default
output of Perl's localtime()

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

debug logging

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

thinko, sorry

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

still working on the perl locale stuff

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

for-loop initial declaration issue; realigned some const-ification

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

pushing locale through in the perl

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

uber-patch from Scott McKellar cleans up large amounts of const-correctness issues and static-ifies several areas of unused header-based code; several cleanups to use simpler APIs such as buffer_release

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

removed hard-coded reference to python 2.4

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

added ability to log to a file

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

changed imports to be fully qualified

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

And let's not double-log messages. Sigh.

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

Format the date and time with leading zeroes to be easy on the eyes, and easier to parse.

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

Correct name of prototype for osrf_message_get_last_locale()

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

Enable srfsh.py to load with the standard .srfsh.xml (no plugins)

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

make the perl local support match the C version

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

Avoid an "undefined variable" Perl warning when we receive no parameters.

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

Convert a few routine log messages from error to info level.

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

Make Logger a little more robust by setting a default loglevel
and ensuring that we don't try to regex an undefined value.

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

fixed infinite loop bug by hopping out of the loop if timeout gets down to 0, but the code still allows the the loop to occur once if timeout is initially 0

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

Patch from Scott McKellar to provide more const correctness to functions using osrfJSON objects

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

Add a top-level javascript-install target

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

Patch from Scott McKellar to increase const correctness in the JSON parser.

Also made the top level legacy_jsonParseString[Fmt] API const-correct, though this is removed within those wrapper calls to avoid potentially breaking the legacy code.

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

Patch from Scott McKellar to clean up some const qualification and initialize a pointer to NULL.

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

added new constructor to build from class hints

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

Patches from Scott McKellar covering:

1. Eliminating a const-removing cast from jsonObjectToJSON().  This
cast is no longer necessary now that a recent patch has changed the
signature of jsonObjectEncodeClass().

2. Moving the JSON_INIT_CLEAR macro out of the header file and into
the implementation file.  No other file invokes this macro -- nor
could it, since the macro refers to two static functions within
osrf_json_object.c.  Strictly speaking another file could provide
other functions with the same signatures, but I doubt that any such
usage was ever intended.

3. Adds some sanity checking to prevent undefined behavior when
a floating point payload is not representable as an int.

4. Streamlines the serialization of hashes and arrays by
eliminating a layer of allocation and deallocation.

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

Deliberately break applications relying on pre-OpenSRF 1.0 behaviour.
JSON_v0.js represents "legacy" JSON format, while JSON_v1.js represents "modern" JSON format.

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

Start installing the Python libraries as well.
Destination directory is named python25 because
Python3000 is just around the corner...

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

returning explicit true/false on data received

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

Install the JavaScript libraries into an OpenSRF library,
in prepration for OpenSRF applications being able to use
one known good JavaScript library for OpenSRF communication.
(I'm looking at you, EG, and your two copies of JSON.js)

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

I'm pretty sure we want $log here, as in all the other calls.
Perhaps we can get rid of the $logger import in the header?

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

Enable -l flag to work without an argument

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

More typos, as I read through logs.

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

adding -l for localhost support

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

Add -D_LARGEFILE64_SOURCE flag to CGLAGS to avoid compile problems on 32-bit platforms.

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

Fix some typos found while trying to improve osrf error messages.

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

provide a warning-level default logger if none is defined

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

Patch that:

1) Creates safe_calloc, suggested by Scott McKellar which includes
memset, as calloc does.  safe_malloc will eventually lose its call
to memset.

2) Creates a macro in utils.h called osrf_clearbuf which

 * under CLFAGS=-DNDEBUG fills the buffer with !s and a trailing nul
 * otherwise (currently) uses memset to fill with nuls

The secondary behavior should be changed to a no-op after no more
problems arise under NDEBUG mode.  I reversed the suggested semantics
because I'm not ready to completely break trunk.  To break everything
(AKA find where we should be providing a terminal nul) just compile
like this:

 $ CLFAGS=-DNDEBUG make clean all

3) replaces all memsets (excepting the ones in safe_?alloc) that act
on char bufs with said macro, so they can be spotted and improved to
deal with nul terminators where needed.  I didn't touch any memsets on
struct pointers.

4) made jid_get_*() from src/libopensrf/transport_message.c safe for
use with NDEBUG mode.  They were depending on the target buffer that
the caller passes in to be nul-filled (they use strncpy/memcpy which
don't guarantee terminal nul) -- now they provide their own terminal
nul.

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

Broad patch from Dan Scott to move towards better memory management:

 * bzero->memset (with sizeof) - except when followed immediately by
   snprintf(), in which case the call was deleted completely
 * sprintf->snprintf (with sizeof) - for the C99-guaranteed
   null-terminated string and avoidance of overwrites
 * fgets (with sizeof) - because in at least one case "len - 1" was
   being used for the length, even though fgets is defined to retrieve 1
   byte less than the requested length - so the code was effectively
   retrieving 2 bytes less than the allocated buffer
 * in 2 places in srfsh.c, increased buffer size by 1 to allow for
   null terminator
 * various typo fixes

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

Patch from Dan Scott to:

1) fix typos and output alignment
2) include _LARGEFILE64_SOURCE in the makefile to make 32b linux happy

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

Use of const-correct osrfObjectGetKeyConst() from Scott McKellar:

1. It replaces jsonObjectGetKey() with jsonObjectGetKeyConst().

2. It uses const pointers, instead of non-const pointers, to capture
the return values of jsonObjectGetIndex() and jsonObjectGetString().

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

Continued const-correctness improvement from Scott McKellar:

1. I changed the signature of jsonObjectDecodeClass so that it accepts
a non-const pointer parameter.  Since it doesn't change the contents
of the jsonObject, there's no need to confine it to non-const
jsonObjects.

2. I replaced jsonObjectGetKey() with jsonObjectGetKeyConst().

3. In one spot I captured the return value of jsonObjectGetIndex()
with a const pointer instead of a non-const pointer.

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

Patch from Scott McKellar to improve const-correctness in srfsh:

This patch sprinkles a few const qualifiers over srfsh.c, and also
fixes a couple of things I stumbled across along the way.

The main purpose is to treat jsonObjects in a more const-correct
manner.  I introduced a new jsonObjectGetKeyConst function, and I
prepared for the day when jsonObjectGetString() returns a const
pointer instead of a non-const pointer.

This patch relies on a previous patch that defines the new
jsonObjectGetKeyConst function.

1. In handle_login() I changed hash to a const pointer, so that we
can assign the return value of jsonObjectGetString() to it.

2. Later in the same function, I rearranged the code a bit to ensure
that we always free a prior value of login_session before giving it
a new value.  The original code potentially leaks memory.

3. In the same area I changed authtoken to a const pointer.

4. In send_request() I eliminated an unnecessary test for the
non-nullness of omsg->_result_content, because we had just tested
it a few lines ago.

5. Also in send_request(): I introduced a layer of strdup() when
we get content from jsonObjectGetString().  This change concerns
more than just const-correctness.

The problem with the original code is that we get content sometimes
from jsonFormatString() and sometimes from jsonObjectGetString().
The former returns a malloc'ed buffer, which we need to free.  The
latter returns a pointer to a buffer owned by a jsonObject, and we
should not free it.  As it happens, we do free it.  If we got the
string from jsonObjectGetString(), then we leave the jsonObject with
an invalid pointer inside it.  If we free the jsonObject later, we'll
try to free that same buffer a second time.  Oops.

Another issue is that jsonObjectGetString() can return NULL, which
we should handle more carefully.

These problems occur in two different places in the same function.
I fixed them both the same way, with some differences in the details.

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

Patch from Scott McKellar which introduces a const-accepting and -returning
version of jsonObjectGetKey.  This is the first step in a plan to push
const-correctness on the OpenSRF stack when dealing with complex objects.  The
hope is that this will increase the utility of compile-time checks in new
client code.

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

added some more rigorous checking of session connected-ness and some logging

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

repaired logic bug which left the parser state in the JSON_STATE_IN_KEY
state after exiting from an empty JSON object.

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