4208422b83c2840b27487dd5a6020197cfe028a8
[OpenSRF.git] / ChangeLog
1 commit ae7d4165ffbcfb35f98700c061e02714b9f3ebee
2 Author: Galen Charlton <gmc@equinoxinitiative.org>
3 Date:   Mon May 7 19:30:24 2018 -0400
4
5     update version numbers for OpenSRF 3.0.1
6     
7     Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>
8
9 7       7       README
10 2       2       src/perl/lib/OpenSRF.pm
11 1       1       src/python/setup.py
12 1       1       version.m4
13
14 commit b5c03e9036a58acfe3099730d98666af7578472e
15 Author: Galen Charlton <gmc@equinoxinitiative.org>
16 Date:   Mon May 7 19:25:45 2018 -0400
17
18     update release notes for OpenSRF 3.0.1
19     
20     Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>
21
22 37      1       doc/RELEASE_NOTES.txt
23
24 commit b704d6ad373ed581b4f8caa2bd156161a0aa1bbd
25 Author: Galen Charlton <gmc@equinoxinitiative.org>
26 Date:   Mon May 7 16:03:16 2018 -0400
27
28     LP#1243841: quiet a misleading indentation warning
29     
30     Adjusts a spaces-vs-tabs issue to fix the following compilation
31     warning:
32     
33     osrf_prefork.c: In function ‘check_children’:
34     osrf_prefork.c:1067:5: warning: this ‘if’ clause does not
35     guard... [-Wmisleading-indentation]
36          if( select_ret <= 0 ) // we're done here
37          ^~
38     osrf_prefork.c:1072:2: note: ...this statement, but the latter is
39     misleadingly indented as if it is guarded by the ‘if’
40       cur_child = forker->first_child;
41       ^~~~~~~~~
42     
43     Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>
44
45 1       1       src/libopensrf/osrf_prefork.c
46
47 commit 62a29dafa4b978d3bc6f4ef8d2dddebe1b319ebb
48 Author: Jason Stephenson <jason@sigio.com>
49 Date:   Tue Nov 7 16:39:39 2017 -0500
50
51     LP#1243841 - Quiet remaining Make install warnings.
52     
53     We silence the following warnings:
54     
55     apachetools.c:179:15: warning: initialization discards 'const' qualifier
56     from pointer target type [enabled by default]
57     
58     apachetools.c:181:8: warning: assignment discards 'const' qualifier from
59     pointer target type [enabled by default]
60     
61     apachetools.c:183:8: warning: assignment discards 'const' qualifier from
62     pointer target type [enabled by default]
63     
64     ./osrf_websocket_translator.c:541:9: warning: return makes integer from
65     pointer without a cast [enabled by default]
66     
67     ./osrf_http_translator.c:300:25: warning: passing argument 1 of 'free'
68     discards 'const' qualifier from pointer target type [enabled by default]
69     
70     Signed-off-by: Jason Stephenson <jason@sigio.com>
71     Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>
72
73 1       1       src/gateway/apachetools.c
74 1       1       src/gateway/osrf_http_translator.c
75 1       1       src/gateway/osrf_websocket_translator.c
76
77 commit 35c7bb1b81bd41d99ad0a0756bf669a13e6ef63b
78 Author: Jason Stephenson <jason@sigio.com>
79 Date:   Tue Nov 7 15:58:05 2017 -0500
80
81     LP#1243841 - Quiet additional Make warnings and some code cleanup.
82     
83     We make the following warnings go away:
84     
85     osrf_legacy_json.c:607:6: warning: variable ‘fourth_dash’ set but
86     not used [-Wunused-but-set-variable]
87     
88     osrf_legacy_json.c:836:5: warning: passing argument 3 of ‘makeNode’
89     discards ‘const’ qualifier from pointer target type [enabled by
90     default]
91     
92     utils.c:133:2: warning: format not a string literal and no format
93     arguments [-Wformat-security]
94     
95     We also cleanup the while block nested in a do while block around line
96     63 of osrf_cache.c to be more readable by adding braces and breaking
97     it across 3 lines.
98     
99     Signed-off-by: Jason Stephenson <jason@sigio.com>
100     Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>
101
102 3       1       src/libopensrf/osrf_cache.c
103 4       6       src/libopensrf/osrf_legacy_json.c
104 6       3       src/libopensrf/utils.c
105
106 commit 4dadb76c5b7e73615de4a1ff923aa46b77b95330
107 Author: Chris Sharp <csharp@georgialibraries.org>
108 Date:   Tue Sep 19 21:25:12 2017 -0400
109
110     LP#1243841 - Quiet wrong format warnings during make install.
111     
112     During make install, the compiler warns that %d expects an int when the
113     actual value is a long int.  Changing %d to %ld fixes the issue.
114     
115     Signed-off-by: Chris Sharp <csharp@georgialibraries.org>
116     Signed-off-by: Jason Stephenson <jason@sigio.com>
117     Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>
118
119 3       3       src/gateway/osrf_websocket_translator.c
120
121 commit c298ab7017627c3502a1b185e2d5871dd0b671ee
122 Author: Chris Sharp <csharp@georgialibraries.org>
123 Date:   Tue Sep 19 20:46:28 2017 -0400
124
125     LP#1243841 - Quiet unused return value warning in srfsh.c
126     
127     Using the technique described here: https://stackoverflow.com/a/13999461.
128     
129     Signed-off-by: Chris Sharp <csharp@georgialibraries.org>
130     Signed-off-by: Jason Stephenson <jason@sigio.com>
131     Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>
132
133 1       1       src/srfsh/srfsh.c
134
135 commit 3994a17e604f32dc184e9f01afac1b3c573b9927
136 Author: Chris Sharp <csharp@georgialibraries.org>
137 Date:   Tue Sep 19 20:37:13 2017 -0400
138
139     LP#1243841 - Quiet unused return value warnings.
140     
141     Using advice given here: https://stackoverflow.com/a/13999461, "The
142     only good (if ugly) way to suppress these is to convert the return
143     value into something that the compiler agrees that you can ignore."
144     
145     Signed-off-by: Chris Sharp <csharp@georgialibraries.org>
146     Signed-off-by: Jason Stephenson <jason@sigio.com>
147     Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>
148
149 4       4       src/libopensrf/utils.c
150
151 commit f50772c5726b1655d67bfefaaa5cd7d4d8892b47
152 Author: Chris Sharp <csharp@georgialibraries.org>
153 Date:   Tue Sep 19 18:10:00 2017 -0400
154
155     LP#1243841 - Wrap truth test using "=" in an extra set of parens.
156     
157     Referring to https://stackoverflow.com/questions/5476759/compiler-warning-suggest-parentheses-around-assignment-used-as-truth-value,
158     it is recommended to wrap variable assignments that are used as a truth
159     test within an extra set of parentheses.
160     
161     Signed-off-by: Chris Sharp <csharp@georgialibraries.org>
162     Signed-off-by: Jason Stephenson <jason@sigio.com>
163     Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>
164
165 1       1       src/libopensrf/osrf_cache.c
166
167 commit 2f7688068af65cbe44c460daf0835b25344e6cca
168 Author: Jason Stephenson <jason@sigio.com>
169 Date:   Sun Feb 18 12:12:13 2018 -0500
170
171     LP1340982: Ignore both 'comment' and '#comment' nodes in Settings Parse.
172     
173     Modify the XML2perl helper function in OpenSRF::Utils::SettingsParser
174     to ignore comment nodes that have names of comment and #comment.
175     
176     To reproduce the bug:
177     
178     1. Stop OpenSRF services.
179     
180     2. Add a XML comment, any text betwen <!-- and -->, in the list of
181     MARC templates in the opensrf.xml file.
182     
183     3. Restart OpenSRF services.
184     
185     4. In the staff client, go to Cataloging->Create New MARC Record.
186     
187     5. In the list of templates, you will #comment where you added the
188     comment. (NOTE: I only tested with 1 comment.  Things may get worse
189     with more than 1 comment.)
190     
191     6. There will be an error like the following in the
192     open-ils.cat_stderr.log if you attempt to choose the #comment entry:
193     
194        Caught error from 'run' method: Exception: OpenSRF::EX::ERROR
195        2018-02-18T11:24:30 OpenSRF::Application
196        /usr/local/share/perl/5.22.1/OpenSRF/Application.pm:240 System
197        ERROR: Unable to open MARC template file: HASH(0x2249928) :
198     
199     To test the fix:
200     
201     1. Leave the comments in the opensrf.xml file from before.
202     
203     2. Apply this commit, do make and make install for OpenSRF.
204     
205     3. Just to make sure, do make and make install for Evergreen.
206     
207     4. Repeat steps 3 and 4 from above.
208     
209     5. You should NOT see any templates named #comment.
210     
211     Signed-off-by: Jason Stephenson <jason@sigio.com>
212     Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>
213
214 1       1       src/perl/lib/OpenSRF/Utils/SettingsParser.pm
215
216 commit 24a14a104c2a216664ab52311eb9f90955b75e31
217 Author: Bill Erickson <berickxx@gmail.com>
218 Date:   Tue Apr 10 15:06:56 2018 -0400
219
220     LP#1762815 Empty client TZ defaults to server TZ (Perl)
221     
222     Fixes an issue in the Perl client time zone handling that resulted in
223     the server defaulting to UTC time instead of the server time zone when
224     no time zone value was received from the client.
225     
226     Signed-off-by: Bill Erickson <berickxx@gmail.com>
227     Signed-off-by: Jason Stephenson <jason@sigio.com>
228
229 1       0       src/perl/lib/OpenSRF/DomainObject/oilsMessage.pm
230
231 commit 0c7f94abf5911cdf042c1162705e721a4fd2899f
232 Author: Bill Erickson <berickxx@gmail.com>
233 Date:   Wed Jan 24 15:16:14 2018 -0500
234
235     LP#1744158 Websocket proc exits on ejabberd disconnect
236     
237     Any errors relaying websocket messages to OpenSRF now result in the WS
238     client being disconnected, allowing the WS process to exit.  This
239     prevents the WS gateway from accepting requests it cannot process and
240     allows the client to connect to a new WS process.
241     
242     Signed-off-by: Bill Erickson <berickxx@gmail.com>
243     Signed-off-by: Jason Stephenson <jason@sigio.com>
244
245 17      4       src/gateway/osrf_websocket_translator.c
246
247 commit 0cb4209fa03091b62df2793a7b273e0c981798d3
248 Author: Bill Erickson <berickxx@gmail.com>
249 Date:   Wed Jan 31 13:07:41 2018 -0500
250
251     LP#1746577 Websocket responder exits on jabber disconnect
252     
253     Regularly check for Jabber socket disconnects in the websocket gateway
254     "responder" thread (that relays messages from opensrf to the WS client)
255     and force a client disconnect when a broken jabber socket is detected.
256     
257     Signed-off-by: Bill Erickson <berickxx@gmail.com>
258     Signed-off-by: Chris Sharp <csharp@georgialibraries.org>
259     Signed-off-by: Jason Stephenson <jason@sigio.com>
260
261 18      2       src/gateway/osrf_websocket_translator.c