OpenSRF.git
13 years agominor layout change. calling notifyAll to ensure all threads are notified
erickson [Fri, 17 Aug 2007 21:50:01 +0000 (21:50 +0000)]
minor layout change.  calling notifyAll to ensure all threads are notified

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

13 years agofor now, we reference the wstx stax implementation directly to resolve class loader...
erickson [Tue, 14 Aug 2007 19:08:38 +0000 (19:08 +0000)]
for now, we reference the wstx stax implementation directly to resolve class loader issues

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

13 years agoPatch from Scott McKellar to protect SHA-1 against an off-by-one bug
miker [Tue, 7 Aug 2007 15:28:44 +0000 (15:28 +0000)]
Patch from Scott McKellar to protect SHA-1 against an off-by-one bug

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

13 years agoPatch from Dan Scott to remove mentions of open-ils from the example config file
miker [Tue, 7 Aug 2007 15:15:08 +0000 (15:15 +0000)]
Patch from Dan Scott to remove mentions of open-ils from the example config file

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

13 years agoadding new JSON parsing JS code, for wrapped class objects
erickson [Fri, 3 Aug 2007 19:31:12 +0000 (19:31 +0000)]
adding new JSON parsing JS code, for wrapped class objects

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

13 years agoadding basic logging api with a filelogger version and a test module
erickson [Fri, 3 Aug 2007 15:49:27 +0000 (15:49 +0000)]
adding basic logging api with a filelogger version and a test module

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

13 years agomore locale cleanup
miker [Fri, 3 Aug 2007 02:57:57 +0000 (02:57 +0000)]
more locale cleanup

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

13 years agoun-crazy-fy current_locale
miker [Fri, 3 Aug 2007 02:30:45 +0000 (02:30 +0000)]
un-crazy-fy current_locale

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

13 years agoformatting ... they are all tabs! ;)
miker [Fri, 3 Aug 2007 02:20:56 +0000 (02:20 +0000)]
formatting ... they are all tabs! ;)

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

13 years agoarg ... left out one minor fix
miker [Fri, 3 Aug 2007 02:19:32 +0000 (02:19 +0000)]
arg ... left out one minor fix

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

13 years agoformatting fixes; protect against NULL apache setting
miker [Fri, 3 Aug 2007 02:19:02 +0000 (02:19 +0000)]
formatting fixes; protect against NULL apache setting

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

13 years agounbreaking the perl session locale stuff
miker [Thu, 2 Aug 2007 03:03:35 +0000 (03:03 +0000)]
unbreaking the perl session locale stuff

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

13 years agoinitial sender_locale support ... probably going to break stuff; also, patch from...
miker [Thu, 2 Aug 2007 02:49:53 +0000 (02:49 +0000)]
initial sender_locale support ... probably going to break stuff; also, patch from Scott McKellar to define some undefined behavior in an snprintf call

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

13 years agoPatch from Scott McKellar to provide defined behavior when passing NULL to an sprintf
miker [Wed, 1 Aug 2007 14:40:39 +0000 (14:40 +0000)]
Patch from Scott McKellar to provide defined behavior when passing NULL to an sprintf

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

13 years agosynchronizing the initial xmpp login to prevent thread race condition on authentication
erickson [Sat, 28 Jul 2007 17:41:32 +0000 (17:41 +0000)]
synchronizing the initial xmpp login to prevent thread race condition on authentication

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

13 years agoadded shutdown call to cleanly disconnect from the opensrf network
erickson [Sat, 28 Jul 2007 17:38:26 +0000 (17:38 +0000)]
added shutdown call to cleanly disconnect from the opensrf network

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

13 years agore-factored and cleaned up the registered class base-class object
erickson [Sat, 28 Jul 2007 15:32:18 +0000 (15:32 +0000)]
re-factored and cleaned up the registered class base-class object

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

13 years agomaking sure error logs are logged to stderr even if syslogging is not an option ...
erickson [Thu, 26 Jul 2007 20:31:41 +0000 (20:31 +0000)]
making sure error logs are logged to stderr even if syslogging is not an option -- eventually, it should fall back to stderr logging like the C code does

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

13 years agoadded a try/catch around the XML parsing and an error log
erickson [Thu, 26 Jul 2007 20:30:52 +0000 (20:30 +0000)]
added a try/catch around the XML parsing and an error log

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

13 years agopython only evaluates default function param values once, so mutable types will alway...
erickson [Thu, 26 Jul 2007 20:30:16 +0000 (20:30 +0000)]
python only evaluates default function param values once, so mutable types will always be pointers to the original default val.  change the default osrfNetworkObject init function to None and setting default manually

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

13 years agoPatch from Scott McKellar to address potential buffer overflow in router query inside...
miker [Tue, 24 Jul 2007 01:48:09 +0000 (01:48 +0000)]
Patch from Scott McKellar to address potential buffer overflow in router query inside srfsh.

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

13 years agoPatch from Scott McKellar providing:
miker [Mon, 23 Jul 2007 17:44:46 +0000 (17:44 +0000)]
Patch from Scott McKellar providing:

1. In handle_print(), I added the ability to display the value of
raw_print.  If we can display pretty_print, we should be able to
display raw_print.

2. In a couple of places we display the value of login_session.
However if we aren't logged in, we end up passing NULL to printf(),
thereby invoking undefined behavior.  Apparently glibc responds by
printing "(null)".  I contrived explicitly to print "(none)" rather
than rely on glibc.

3. Since login_session is assigned from strdup() if not NULL, I added
a free() for it in a couple of places: one at the end of main(), and
one just before the assignment from strdup().

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

13 years agoadded JSON gateway support
erickson [Sun, 22 Jul 2007 22:06:49 +0000 (22:06 +0000)]
added JSON gateway support

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

13 years agoadded a bunch of comments to the test client to help explain the api
erickson [Sat, 21 Jul 2007 20:50:19 +0000 (20:50 +0000)]
added a bunch of comments to the test client to help explain the api
made the makefile 'run' command more verbose

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

13 years agoressurrected the srfsh time command
erickson [Sat, 21 Jul 2007 20:46:08 +0000 (20:46 +0000)]
ressurrected the srfsh time command

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

13 years agoPatch from Scott McKellar to fill buffer overflow holes:
miker [Sat, 21 Jul 2007 20:31:30 +0000 (20:31 +0000)]
Patch from Scott McKellar to fill buffer overflow holes:

The first overflow can happen with an excessively long username.

The second overflow is more doubtful, because the inputs come from
two other functions.  It's not obvious whether an overflow is possible
or not.  It may be that those functions will never return strings long
enough to overflow.  However it is easier to assume that they might,
and avoid the overflow for sure, than to determine whether an overflow
is possible in the first place.

In each case I declared a variable-length character array with a
calculated length.

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

13 years agoPatch from Ed Summers:
erickson [Sat, 21 Jul 2007 18:59:31 +0000 (18:59 +0000)]
Patch from Ed Summers:

Attached is a minor patch to OpenSRF/src/java/Makefile for creating a
opensrf.jar. I was thinking an opensrf.jar would make it easy to
deploy OpenSRF client in a Woodchip/OpenTaps instance. It's not yet
clear to me what the build environment is (or will be) for
Woodchip--but .jar files are used to across the board to manage
external dependencies in Javaland.

- - -

I've also given the depencies their own vars for easier handling and
made the dependency fetching smarter to prevent re-fetching existing deps

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

13 years agoMerging changes from branches/new-json2
erickson [Fri, 20 Jul 2007 12:58:59 +0000 (12:58 +0000)]
Merging changes from branches/new-json2

svn merge -r1006:1012 svn://svn.open-ils.org/OpenSRF/branches/new-json2
svn merge -r1018:1019 svn://svn.open-ils.org/OpenSRF/branches/new-json2
svn merge -r1022:1025 svn://svn.open-ils.org/OpenSRF/branches/new-json2
svn merge -r1026:1028 svn://svn.open-ils.org/OpenSRF/branches/new-json2
svn merge -r1036:1037 svn://svn.open-ils.org/OpenSRF/branches/new-json2
svn merge -r1039:1040 svn://svn.open-ils.org/OpenSRF/branches/new-json2
svn merge -r1041:1043 svn://svn.open-ils.org/OpenSRF/branches/new-json2
svn merge -r1045:1048 svn://svn.open-ils.org/OpenSRF/branches/new-json2

This moves OpenSRF to the new JSON wire protocol,
including:
    a new C parser
    a new osrfList based string_array module
    a compatibility layer for mimicking libobjson and legacy JSON I/O in the HTTP gateway
    a small JSON test program for profiling, etc.

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

13 years agofixed bug in json encoding of hinted objects and some display bugs
erickson [Wed, 18 Jul 2007 22:27:04 +0000 (22:27 +0000)]
fixed bug in json encoding of hinted objects and some display bugs

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

13 years agoadded config contexts for config files. fixed some osrfNetworkObject handling bugs...
erickson [Tue, 17 Jul 2007 14:07:43 +0000 (14:07 +0000)]
added config contexts for config files.  fixed some osrfNetworkObject handling bugs.  did some re-tabbing to change to space-based tabs

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

13 years agoPatch from Scott McKellar to repair srfsh buffer overflow:
erickson [Mon, 16 Jul 2007 13:03:48 +0000 (13:03 +0000)]
Patch from Scott McKellar to repair srfsh buffer overflow:

The potential overflow occurs in handle_introspection(), which in the
existing code uses sprintf() to insert some user-supplied input into a
fixed length buffer, without checking the length of the user-supplied
input.

There's always more than one way to fix this sort of thing.  For
example I could have formatted into a growing_buffer.  Instead, I
chose to calculate the necessary buffer size and then declare a
variable-length buffer.  That way I avoid two mallocs and two frees.

I tested this change by inserting some temporary printfs to capture
the commands being sent to parse_request().  The outputs before and
after the change were identical.

There's at least one more such potential overflow that I haven't
addressed yet.  It's near the top of handle_login(), where we
use sprintf() to insert a username into a fixed length buffer.
There may be others that I haven't noticed yet.

Scott McKellar
http://home.swbell.net/mck9/ct/

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

13 years agoPatch from Scott McKellar to clean up srfsh's user interface:
miker [Sun, 15 Jul 2007 02:18:41 +0000 (02:18 +0000)]
Patch from Scott McKellar to clean up srfsh's user interface:

1. srfsh exits the main loop in two circumstances.  The first is when
it sees the "quit" or "exit" command.  The second is when it sees
end-of-file, either because it reads the end of a script or because
the user enters the associated keystrokes (typically Control-D).

In the case of end-of-file, srfsh exits without issuing another
newline.  As a result, the next shell prompt shows up concatenated
to the srfsh prompt.  This result is harmless but annoying.

I tweaked the code to detect this situation and issue an extra
newline, so that the next shell prompt show up on a line by itself.

2. I strip off leading whitespace before processing the input
command.

As a result, srfsh will now recognize "quit", "exit", or a leading
exclamation point, even when preceded by whitespace.  Other commands
are not affected because strtok() already strips off leading
blanks.  Tab characters don't matter because readline() removes them.

3. I also strip off trailing whitespace.  This measure affects only
the "quit" and "exit" commands.

4. I ignore input lines whose first non-whitespace character is an
octothorpe ('#', also known as hash, pound, or tic-tac-toe).  In
other words, scripts can now have comments.

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

13 years agochanging back to an array, from calloc. more straight-forward, and avoids memset...
miker [Fri, 13 Jul 2007 14:52:20 +0000 (14:52 +0000)]
changing back to an array, from calloc.  more straight-forward, and avoids memset overhead

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

13 years agoPatch from Scott McKellar, with modifications, to remove unused code and provide...
miker [Fri, 13 Jul 2007 03:15:51 +0000 (03:15 +0000)]
Patch from Scott McKellar, with modifications, to remove unused code and provide some memory protection:

This patch contains one minor change and one less minor change.

--------

The less minor change is the elimination of some old code that once
opened a pipe to send commands to bash.  According to Bill Erickson
in a private email, that code is a leftover remnant from an
experiment and may be removed.

Before a patch was applied several days ago, things worked like this:
IF you compiled srfsh with a certain macro #defined, AND you entered
a command that was invalid to srfsh, THEN srfsh would pipe the
command to bash for execution.

It is doubtful that anyone but Bill ever used this variation, but if
anyone wants to keep it around, then this is his or her chance to
protest.  Before protesting, however, please note that the shell
escape mechanism now provides shell-like command processing.  I.e.
if you start the srfsh command line with an exclamation point, srfsh
passes the rest of the command to the default shell (normally
/bin/sh) for execution.  The command so passed can use environmental
variables, pipes, IO redirection, wild card expansion, and most of
the sorts of things you're used to from the shell command line.

Note that not all shells behave the same.  If you're used to using
tcsh as your shell, for example, you may find that sh won't work
quite the same way.  For now, at least, that's just too bad, but
that's the way shell escapes typically work.

It is probably possible to invoke the user's default shell as
identified by the environmental variable $SHELL.  However that
nicety would not be trivial to code.

----------

The minor change is that, after building an array of pointers
pointing to tokens from the input command, I set the next
pointer to NULL so that it mark the end of the token list.

(miker: I changed to calloc/free instead of an array, but left the final
NULL in place.)

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

13 years agoInitialized merge tracking via "svnmerge" with revisions "1003" from
erickson [Thu, 12 Jul 2007 18:21:22 +0000 (18:21 +0000)]
Initialized merge tracking via "svnmerge" with revisions "1003" from
svn://svn.open-ils.org/OpenSRF/branches/new-json2

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

13 years agoPatch from Scott McKellar to fix potential local buffer
miker [Wed, 11 Jul 2007 03:17:13 +0000 (03:17 +0000)]
Patch from Scott McKellar to fix potential local buffer
overflow attack against srfsh:

This patch fixes a potential buffer overflow in the parse_error
function.  The existing code concatenates the strtoked tokens into a
fixed-length buffer, with no check for overflow.  It isn't hard to
build an srfsh command that overflows the buffer, with baleful
results.

While it's not likely that anyone would do so by accident from the
command line, an srfsh script might well do so, especially if the
script were generated from another program.

More important, someone sufficiently clever might be able to use
such an overflow to work mischief.

My version of parse_error() uses a growing_buffer to accumulate
the tokens.

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

13 years agoPatch from Scott McKellar to remove srfsh.h and push all relevant
miker [Mon, 9 Jul 2007 13:08:34 +0000 (13:08 +0000)]
Patch from Scott McKellar to remove srfsh.h and push all relevant
delarations into the implementation file, with some additional cleanup
to remove "declared by not used" warnings:

1. We have no further use for the header file.  Accordingly I
include a patch to the Makefile, removing a dependency on the header.

2. I declared as static everything that was formerly in the header,
plus a few variables that were already defined in srfsh.c.

3. I commented out the shell_reader variable, since it is no longer
used.

4. I commented out two of the lines printed by the help command.
These lines refer to the "time" command, which srfsh no longer
supports.

5. I commented out the prototype for the handle_time function, since
the function itself is already commented out.

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

13 years agorepaired conflicting declarations
erickson [Sun, 8 Jul 2007 17:51:11 +0000 (17:51 +0000)]
repaired conflicting declarations

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

13 years agoPatch from Scott McKellar for declaration cleanup and improved error handling:
miker [Sun, 8 Jul 2007 17:40:23 +0000 (17:40 +0000)]
Patch from Scott McKellar for declaration cleanup and improved error handling:

1. I added the const qualifier to several function parameters.

2. I replaced inet_addr() with inet_aton().

According to the man page, inet_addr() is obsolete.  It reports an
error by returning -1, which however is a valid IP address
(255.255.255.255).  inet_aton reports errors in a more robust way.

3. I check the return value from inet_aton().  If the input address is
invalid, I log a message and return -1.

Without such a check, an invalid listener address leads to a server
that is completely unresponsive.  The user has no way to know what's
wrong unless he spots the mistake himself in the relevant
configuration file.

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

13 years agoadding COPYING and LICENSE.txt
miker [Fri, 6 Jul 2007 16:56:32 +0000 (16:56 +0000)]
adding COPYING and LICENSE.txt

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

13 years agoa little bit more cleanup from previous DOM/DomianObject removal
miker [Fri, 6 Jul 2007 12:29:52 +0000 (12:29 +0000)]
a little bit more cleanup from previous DOM/DomianObject removal

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

13 years agomoved from custom is_number() to standared ctype.h isdigit()
miker [Fri, 6 Jul 2007 00:20:44 +0000 (00:20 +0000)]
moved from custom is_number() to standared ctype.h isdigit()

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

13 years agoreformated error for easier reading
erickson [Thu, 5 Jul 2007 20:45:36 +0000 (20:45 +0000)]
reformated error for easier reading

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

13 years agoremoving signedness warnings coming from libxml2
miker [Thu, 5 Jul 2007 17:58:49 +0000 (17:58 +0000)]
removing signedness warnings coming from libxml2

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

13 years agofixed bug where no-text-data resulted in incorrect array index when parsing hinted...
erickson [Thu, 5 Jul 2007 16:00:42 +0000 (16:00 +0000)]
fixed bug where no-text-data resulted in incorrect array index when parsing hinted objects

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

13 years agofixed bug in to-json logic -- was still using older api
erickson [Thu, 5 Jul 2007 14:53:24 +0000 (14:53 +0000)]
fixed bug in to-json logic -- was still using older api

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

13 years agoremoved some initial (non-error) stderr logging.
erickson [Thu, 5 Jul 2007 12:22:45 +0000 (12:22 +0000)]
removed some initial (non-error) stderr logging.

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

13 years agoneed to wait for perl to catch up ...
miker [Thu, 5 Jul 2007 00:14:18 +0000 (00:14 +0000)]
need to wait for perl to catch up ...

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

13 years agocapture the perl pid using ps instead of $! (forking for daemonization)
miker [Thu, 5 Jul 2007 00:02:42 +0000 (00:02 +0000)]
capture the perl pid using ps instead of $! (forking for daemonization)

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

13 years agodo the process separation dance on server bootstrapping, just like with the C code
miker [Wed, 4 Jul 2007 23:48:21 +0000 (23:48 +0000)]
do the process separation dance on server bootstrapping, just like with the C code

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

13 years agoremoving vestigial DOM code from back when opensrf talked XML
miker [Wed, 4 Jul 2007 20:08:15 +0000 (20:08 +0000)]
removing vestigial DOM code from back when opensrf talked XML

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

13 years agoupdated the sylog and activity log file handling to accomodate differences from boots...
erickson [Mon, 2 Jul 2007 22:07:02 +0000 (22:07 +0000)]
updated the sylog and activity log file handling to accomodate differences from bootstrap.conf and opensrf_core.xml

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

13 years agoCompleting the JSON perl module move started by patch from Dan Scott.
miker [Mon, 2 Jul 2007 15:14:02 +0000 (15:14 +0000)]
Completing the JSON perl module move started by patch from Dan Scott.

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

13 years agoPatch from Dan Scott to move JSON to OpenSRF::Utils::JSON:
miker [Mon, 2 Jul 2007 15:11:15 +0000 (15:11 +0000)]
Patch from Dan Scott to move JSON to OpenSRF::Utils::JSON:

I noticed back when I was first installing OpenSRF that it includes a
module, JSON.pm, that exists at the root level of the package
directories. This would be fine, except it conflicts with a CPAN
module that is also named JSON, which confuses the CPAN installer when
you check for upgrades and conceivably could lead to a broken system.

I suggested to Mike that it would probably make sense to move the
OpenSRF version of the module into the OpenSRF/Utils/ package
namespace, and he agreed. Of course, there are ramifications
throughout the code, so I've tried to be extra-careful in catching and
correcting all of the places where the use of this module surfaces in
both OpenSRF and Evergreen.

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

13 years agoPatch from Dan Scott to finish up the removal of bootstrap.conf:
miker [Mon, 2 Jul 2007 14:24:04 +0000 (14:24 +0000)]
Patch from Dan Scott to finish up the removal of bootstrap.conf:

Attached are patches that complete the work that I should have done in
my first patch. I also modify the Makefile in OpenSRF to prevent
bootstrap.conf.example from being copied from the OpenSRF/examples
directory into /openils/conf/.

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

13 years agoPatch from Scott McKellar implementing cleaner daemonization; moved daemonizing code...
miker [Sat, 30 Jun 2007 03:14:36 +0000 (03:14 +0000)]
Patch from Scott McKellar implementing cleaner daemonization; moved daemonizing code above worker forking in system boostrapping:

1. As long as I was in the neighborhood, I replaced the error messages
going to stderr with messages going to the logging machinery.

2. I altered the comment just above daemonize(), because the original
was inaccurate.  This function does not change the process title.

3. Pedantic point: I captured the return value of fork() with a pid_t
instead of an int.

4. After the fork, the parent process terminates with _exit()
instead of exit().  That way it doesn't flush any buffers or close
any files.  It also doesn't call any functions registered with
atexit().  In fact it doesn't do much of anything except get out of
the way and let the child process take its place in every way.

5. The child process switches to the root directory, calls setsid()
to create a new session, and freopens the three standard streams
to /dev/null.

I could have changed directories before the fork, or at any of
several other places.  I don't think it makes any difference.

I could also have done the freopens before the fork, but if the
fork failed, I might not have a way to report it (if the logging is
going to stderr for some reason).

Note that I'm not flushing any buffers, apart from the three standard
streams.  There's no need to.  The child process inherits the file
descriptors intact and can do with them whatever it likes.  The
parent may have died, but the estate doesn't have to go through
probate.

I didn't add any error checking for chdir(), setsid(), or freopen().
The first two are unlikely to fail, as is the freopen of stdin.  The
freopens of stderr and stdout could fail, if for example they are
redirected to a full disk.  I don't know how paranoid we need to be.

For more details, see archives starting at

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

and also

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

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

13 years agocleaning up XML indentation via xmllint; removing bootstrap.conf, which is no longer...
miker [Fri, 29 Jun 2007 15:36:58 +0000 (15:36 +0000)]
cleaning up XML indentation via xmllint; removing bootstrap.conf, which is no longer needed (yay Dan!)

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

13 years agotest for failed forker creation before touching struct members
miker [Fri, 29 Jun 2007 14:07:01 +0000 (14:07 +0000)]
test for failed forker creation before touching struct members

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

13 years agoPatch from Dan Scott to move perl OpenSRF core bootstrapping settings into
miker [Fri, 29 Jun 2007 03:55:37 +0000 (03:55 +0000)]
Patch from Dan Scott to move perl OpenSRF core bootstrapping settings into
opensrf_core.xml.  This removes the dependency on the INI style bootstrap.conf
file:

Building on Nathan Eady's suggestion / intention from December
(http://list.georgialibraries.org/pipermail/open-ils-dev/2006-December/000177.html),
here is a patch that enables OpenSRF to avoid duplicating settings in
opensrf_core.xml and bootstrap.conf by having both Perl and C apps
read from opensrf_core.xml

The major limitation is that I've hardcoded /config/opensrf to appear as
the 'bootstrap' section of config for compatibility with the expectations of
the rest of OpenSRF. A broader patch would convert all of those other calls
from config->bootstrap->blah to config->opensrf->blah and make the
siblings of /config/opensrf visible in the config as well, as a more
generally useful approach.

Applied with minor changes to avoid API regressions and remove unneeded code.

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

13 years agoarg ... Patch from Scott McKellar that:
miker [Fri, 29 Jun 2007 02:24:47 +0000 (02:24 +0000)]
arg ... Patch from Scott McKellar that:

1. When we log messages to a log file, we open and close the file
for each message.  If the open fails, we write a message to stderr
about the failure to open, but we discard the original message that
we were asked to issue.

With this patch, after issuing the message about the failure to
open, we write the original message to stderr.

I believe that this change plugs the last message leak.  Now if we're
asked to issue a message, by golly we're going to issue it, one way
or another.

Of course the user may still not see the message, either through
inattention or, for example, because stderr is redirected to
/dev/null.  In that case the user is just a poo-poo head, and I have
no sympathy for him.

2. In an earlier post I proposed to change osrfLogSetType() so that
it would return the previous log type.  That way one could
temporarily change the log type and then restore it later, without
knowing in advance what to restore it to.

Upon further reflection I decided that the only plausible use for
this trick would be to reroute messages to standard error, and I
might as well provide a mechanism limited to this purpose.

Accordingly I created two new functions and prototyped them in log.h:

osrfLogToStderr() reroutes messages to stderr, and saves the previous
log type internally.

osrfRestoreLogType() restores the log type previously saved, if any.

This interface provides fewer ways for the calling code to mess up
than what I had originally contemplated.  First, the calling code
doesn't have to keep track of the previous log type.  Second, if the
messages are already rerouted, osrfLogToStderr() will do nothing.
One needn't worry about nested reroutings, or maintaining a stack of
log types, or anything of the sort.

If we ever need anything fancier we can build it, but I don't think
we will.

3. Wherever a function takes no parameters, I coded "void" as the
formal parameter list.  Otherwise the compiler doesn't know whether
the function takes parameters or not.

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

13 years agoPatch from Scott McKellar:
miker [Thu, 28 Jun 2007 03:14:06 +0000 (03:14 +0000)]
Patch from Scott McKellar:

At one time. if we had trouble reading or parsing the configuration
file, we would issue log messages to complain.  However those log
messages didn't have anywhere to go because, not having loaded the
configuration file, we didn't know where to write the messages.

The earlier patch wrote the same messages to standard error as well
as to the logging machinery, so that they would be visible.

These extra writes are no longer necessary,  With recent changes to
the logging machinery, all log messages will go to standard error
if no other destination has been defined for them.

Accordingly this patch removes two messages, now redundant, that
were being written to standard error.

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

13 years agoI missed patching the header somehow ... (Part of a patch from Scott McKellar.
miker [Thu, 28 Jun 2007 01:07:55 +0000 (01:07 +0000)]
I missed patching the header somehow ... (Part of a patch from Scott McKellar.
See http://svn.open-ils.org/trac/OpenSRF/changeset/977 for details.)

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

13 years agoPatch from Scott McKellar, finishing off the "log to stderr when all esle fails"...
miker [Tue, 26 Jun 2007 03:04:37 +0000 (03:04 +0000)]
Patch from Scott McKellar, finishing off the "log to stderr when all esle fails" work:

Currently if we have asked to log to a file, but we haven't yet
specified the name of that log file, then any messages issued are
dropped.

With this patch, any messages issued under these conditions will be
rerouted to standard error.  The rerouting is only temporary,
applying only to the current message.

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

13 years agoadding yet more router commands to gather some stats about request counts
miker [Mon, 25 Jun 2007 20:48:18 +0000 (20:48 +0000)]
adding yet more router commands to gather some stats about request counts

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

13 years agoadding router commands to gather some stats about request counts
miker [Mon, 25 Jun 2007 20:22:34 +0000 (20:22 +0000)]
adding router commands to gather some stats about request counts

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

13 years agoPatch from Scott McKellar to improve default logging:
miker [Mon, 25 Jun 2007 03:06:56 +0000 (03:06 +0000)]
Patch from Scott McKellar to improve default logging:

1. In log.h I added another macro OSRF_LOG_TYPE_STDERR by which the
application can specifically request logging to standard error.

2. In log.c I initialze _osrfLogType to OSRF_LOG_TYPE_STDERR so that
the application can issue a log message even if it hasn't called
osrfLogInit() yet.

3. I added the const qualifier to the parameters of the functions
_osrfLogToFile() and _osrfLogSetXid().

4. In osrfLogCleanup() I set _osrfLogAppname and _osrfLogFile to
NULL after freeing them.  Otherwise the application could cause
needless mischief by calling this function twice.

5. Also in osrfLogCleanup() I set _osrfLogType back to the default
OSRF_LOG_TYPE_STDERR so that the application can still issue a
message after calling osrfLogCleanup().

6. I rewrote osrfLogSetType() to use switch/case instead of an "if".
Now that we have three valid values to check instead of two, a
switch/case is a little tidier.  Also the default branch applies
the default log type OSRF_LOG_TYPE_STDERR.

7. _osrfLogDetail() had a local variable named "l" (lower case L).
I renamed it to "label" because "l" looks too much like a digit.

8.Also in _osrfLogDetail(): I added a branch to write a message to
stderr when _osrfLogType is OSRF_LOG_TYPE_STDERR.

9. The existing _osrfLogToFile function sizes and declares a buffer
named "buf", fills it with zeros, and then doesn't do anything with
it.  This buffer is probably a relic of some earlier version. I
eliminated it.

10. A few lines thereafter, I eliminated a useless call to bzero().

11. The call to strftime() contained a hard-coded buffer size.  I
changed it to use the sizeof operator, so that the buffer size is
defined in only one place.

12. If we can't open the log file, we write a message to stderr.
In this message I changed "file" to "log file" to make the meaning
more clear to a benighted and panicky user.

13. If after writing the message we can't close the file, the
existing code logs an error message by calling osrfLogWarning().
However in this circumstance this latter message will probably meet
the same fate, and no one will ever see it.  I changed the code so
as to write this message to stderr.

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

13 years agoallow the OSRF_HOSTNAME env variable to specify the local hostname -- for single...
miker [Fri, 22 Jun 2007 01:54:39 +0000 (01:54 +0000)]
allow the OSRF_HOSTNAME env variable to specify the local hostname -- for single-box installs

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

13 years agotesting email2
erickson [Thu, 21 Jun 2007 20:45:35 +0000 (20:45 +0000)]
testing email2

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

13 years agotesting email
erickson [Thu, 21 Jun 2007 20:42:38 +0000 (20:42 +0000)]
testing email

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

13 years agoforget to add libopensrf Makefile with latest re-org commit
erickson [Thu, 21 Jun 2007 16:22:02 +0000 (16:22 +0000)]
forget to add libopensrf Makefile with latest re-org commit

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

13 years agoMerged libopensrf source directories (libtransport, libstack, and utils) into a singl...
erickson [Thu, 21 Jun 2007 16:19:20 +0000 (16:19 +0000)]
Merged libopensrf source directories (libtransport, libstack, and utils) into a single libopensrf directory
Moved opensrf headers to trunk/include/opensrf
Moved objson headers to trunk/include/objson
Updated #include's throughout to be fully qualified.  e.g. <opensrf/utils.h>, <objson/object.h>
Removed old, unused trunk/src/xinclude code
Cleaned up Makefiles to support new directory layout

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

13 years agomade the logging module fail gracefully when syslog is not present
erickson [Thu, 21 Jun 2007 14:35:27 +0000 (14:35 +0000)]
made the logging module fail gracefully when syslog is not present

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

13 years agothinko, need to return an int from osrfLogGetLevel()
miker [Thu, 21 Jun 2007 02:20:10 +0000 (02:20 +0000)]
thinko, need to return an int from osrfLogGetLevel()

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

13 years agoAdded osrfLogGetLevel method, then applied
miker [Wed, 20 Jun 2007 03:49:48 +0000 (03:49 +0000)]
Added osrfLogGetLevel method, then applied
partial patch from Scott McKellar:

Whenever a declared identifier had two leading underscores, I
removed the first one, except for a few cases where I removed the
second one and kept the first one.

I moved the prototypes for the following functions to log.c, and
declared them static:

 osrfLogSetType
 _osrfLogDetail
 _osrfLogToFile
 _osrfLogSetXid

I declared the following variables static:

 _osrfLogType
 _osrfLogFacility
 _osrfLogActFacility
 _osrfLogFile
 _osrfLogAppname
 _osrfLogLevel
 _osrfLogActivityEnabled
 _osrfLogIsClient
 _osrfLogXid
 _osrfLogXidPfx

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

13 years agoPatch from Scott McKellar to add stderr logging when config (and thus configured...
miker [Sat, 16 Jun 2007 23:58:35 +0000 (23:58 +0000)]
Patch from Scott McKellar to add stderr logging when config (and thus configured logging) is unavailable:

If osrfConfigInit() is unable to load the configuration file,
it issues error messages to the log.

Unfortunately, if we can't load the configuration file, we don't know
where the log file is.  The usual result is that the messages don't
go anywhere.  At best, depending on some convoluted details that
aren't worth going into here, the messages will go to an obscure
system-wide syslog file that only a hardened system administrator
could love.

In this patch I send similar messages to standard error, so that
someone can see them.  It might make sense to delete the calls
to osrfLogWarning(), but I left them alone.

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

13 years agomore tuning of hinted object handling. added support for hinted object parsing in...
erickson [Fri, 15 Jun 2007 14:31:30 +0000 (14:31 +0000)]
more tuning of hinted object handling.  added support for hinted object parsing in gateway parser

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

13 years agotesting email 2
erickson [Fri, 15 Jun 2007 03:36:39 +0000 (03:36 +0000)]
testing email 2

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

13 years agotesting email
erickson [Fri, 15 Jun 2007 03:33:46 +0000 (03:33 +0000)]
testing email

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

13 years agoCleaned up the network hint / object registration code in net_obj
erickson [Fri, 15 Jun 2007 03:18:51 +0000 (03:18 +0000)]
Cleaned up the network hint / object registration code in net_obj
Added object2XML function for posting XML to the opensrf gateway
updated the gateway to deal with the new object API

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

13 years agoAdded a new XML-2-JSON utility for converting XMLized OpenSRF objects to their intern...
erickson [Thu, 14 Jun 2007 16:03:57 +0000 (16:03 +0000)]
Added a new XML-2-JSON utility for converting XMLized OpenSRF objects to their internal JSON representations
Added a new gateway param "input_format" to define the param format.
Options are "json" and "xml".
Unless overridden, input_format will match format

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

13 years agorolling back changes to objson includes. opensrf cannot find "opensrf/<blah>" since...
erickson [Wed, 13 Jun 2007 15:10:29 +0000 (15:10 +0000)]
rolling back changes to objson includes. opensrf cannot find "opensrf/<blah>" since there is no external include directory yet.

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

13 years agoPatch from Dan Scott to fix up the OpenSRF build process in the new SVN repo:
miker [Tue, 12 Jun 2007 02:52:42 +0000 (02:52 +0000)]
Patch from Dan Scott to fix up the OpenSRF build process in the new SVN repo:

OpenSRF-svn-build.patch: this modifies the objson headers to specify
that "utils.h" is actually found in "opensrf/utils.h". I beleive the
ILS build was failing with the unqualified include because it didn't
have access to the utils.h file that used to be kept around in a
common TMP directory when OpenSRF and ILS were built simultaneously.

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

13 years agoPatch from Scott McKellar; rearranges some logic to avoid a potential memory leak
miker [Tue, 12 Jun 2007 01:30:49 +0000 (01:30 +0000)]
Patch from Scott McKellar;  rearranges some logic to avoid a potential memory leak

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

13 years agoPatch from Scott McKellar:
miker [Tue, 12 Jun 2007 01:23:04 +0000 (01:23 +0000)]
Patch from Scott McKellar:

This small and inoffensive patch concerns two functions in utils.c:
init_proc_title() and set_proc_title().

1. I added some comments to explain what these functions do for a
living.

2. I declared the __global_argv and __global_argv_size variables
static.  No other source file refers to them.

3. I removed the leading underscores from their names.  Identifiers
beginning with two underscores are reserved for the implementation.

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

13 years agoCapturing the return value of the bootstrap call and emitting a message when encounte...
miker [Tue, 12 Jun 2007 01:15:55 +0000 (01:15 +0000)]
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

13 years agoPatch from Scott McKellar to clarify errors in bootstrapping of the server process
miker [Tue, 12 Jun 2007 01:11:52 +0000 (01:11 +0000)]
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

13 years agoPatch from Scott McKellar with minor adjustments:
miker [Tue, 12 Jun 2007 01:10:05 +0000 (01:10 +0000)]
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

13 years agoremoved unused TEMPLATEDIR directory creation. thanks, Dan S.
erickson [Tue, 12 Jun 2007 01:09:39 +0000 (01:09 +0000)]
removed unused TEMPLATEDIR directory creation.  thanks, Dan S.

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

13 years agoPatch from Scott McKellar:
miker [Tue, 12 Jun 2007 01:08:25 +0000 (01:08 +0000)]
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

13 years agoPatch from Scott McKellar:
miker [Tue, 12 Jun 2007 01:05:53 +0000 (01:05 +0000)]
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

13 years agoadding gateway parsing code. so far, we have XML gateway handling using python minidom.
erickson [Mon, 11 Jun 2007 22:21:08 +0000 (22:21 +0000)]
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

13 years agoadding sample makefile and install.conf file for opensrf installation
erickson [Mon, 11 Jun 2007 20:13:58 +0000 (20:13 +0000)]
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

13 years agoported XML parsing from libxml2 to Python's builtin xml.dom.minidom to reduce dependenies
erickson [Mon, 11 Jun 2007 20:06:56 +0000 (20:06 +0000)]
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

13 years agoPatch from Scott McKellar; cleaning up several error conditions
miker [Sun, 10 Jun 2007 23:41:14 +0000 (23:41 +0000)]
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

13 years agoPatch from Scott McKellar; Adds calls to strerror() where appropriate in
miker [Thu, 31 May 2007 12:12:02 +0000 (12:12 +0000)]
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

13 years agoremoved some error lines printed to stderr. not used and they also have the potentia...
erickson [Thu, 31 May 2007 03:33:06 +0000 (03:33 +0000)]
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

13 years agoslimmed some logging. capturing errno locally after the recv() call so it is not...
erickson [Thu, 31 May 2007 03:32:27 +0000 (03:32 +0000)]
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

13 years agono longer freeing return value from strerror since it should not be modified
erickson [Tue, 29 May 2007 13:26:33 +0000 (13:26 +0000)]
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

13 years agodueling patches of pedantism
miker [Tue, 29 May 2007 13:08:53 +0000 (13:08 +0000)]
dueling patches of pedantism

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

13 years agoadded some more descriptive logging. removed old signal handling code. no longer...
erickson [Tue, 29 May 2007 12:54:49 +0000 (12:54 +0000)]
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