http://www.couchbase.com/issues This file is an XML representation of an issue en-us 5.2.4 845 26-12-2012 http://www.couchbase.com/issues/browse/JCBC-140 Couchbase Java Client The code breaks abruptly when connection URI is not given in a proper format in the SDK client for connecting the to required Couchbase server. <br/> For instance, when I am specifying the URI as &quot;http:/10.3.2.57:8091/index.html&quot; instead of &quot;http:/10.3.2.57:8091/pools&quot;, exception appears, <br/> whereas a proper error message should be returned saying that &#39;Connection could not be established to the requested server location&#39;. <br/> Please find attached the log file for the exception trace. 1939 - &lt;manifest&gt;&lt;remote name=&quot;couchbase&quot; fetch=&quot;<a href="git://github.com/couchbase/">git://github.com/couchbase/</a>&quot;/&gt;&lt;remote name=&quot;membase&quot; fetch=&quot;<a href="git://github.com/membase/">git://github.com/membase/</a>&quot;/&gt;&lt;remote name=&quot;apache&quot; fetch=&quot;<a href="git://github.com/apache/">git://github.com/apache/</a>&quot;/&gt;&lt;remote name=&quot;erlang&quot; fetch=&quot;<a href="git://github.com/erlang/">git://github.com/erlang/</a>&quot;/&gt;&lt;default remote=&quot;couchbase&quot; revision=&quot;master&quot;/&gt;&lt;project name=&quot;tlm&quot; path=&quot;tlm&quot; revision=&quot;12abea946eafd7411273d18a10ae1f84390db3d4&quot;&gt;&lt;copyfile src=&quot;Makefile.top&quot; dest=&quot;Makefile&quot;/&gt;&lt;/project&gt;&lt;project name=&quot;bucket_engine&quot; path=&quot;bucket_engine&quot; revision=&quot;70b3624abc697b7d18bf3d57f331b7674544e1e7&quot;/&gt;&lt;project name=&quot;ep-engine&quot; path=&quot;ep-engine&quot; revision=&quot;68e3f7c57ae40c42dcb94e4d282bbd1648f9adc0&quot;/&gt;&lt;project name=&quot;libconflate&quot; path=&quot;libconflate&quot; revision=&quot;2cc8eff8e77d497d9f03a30fafaecb85280535d6&quot;/&gt;&lt;project name=&quot;libmemcached&quot; path=&quot;libmemcached&quot; revision=&quot;ca739a890349ac36dc79447e37da7caa9ae819f5&quot; remote=&quot;membase&quot;/&gt;&lt;project name=&quot;libvbucket&quot; path=&quot;libvbucket&quot; revision=&quot;00d3763593c116e8e5d97aa0b646c42885727398&quot;/&gt;&lt;project name=&quot;membase-cli&quot; path=&quot;membase-cli&quot; revision=&quot;7fe4121e7e83952a4cb032e25a2cb9fca1709354&quot; remote=&quot;membase&quot;/&gt;&lt;project name=&quot;memcached&quot; path=&quot;memcached&quot; revision=&quot;7ea975a93a0231393502af4ca98976eee8a83386&quot; remote=&quot;membase&quot;/&gt;&lt;project name=&quot;moxi&quot; path=&quot;moxi&quot; revision=&quot;52a5fa887bfff0bf719c4ee5f29634dd8707500e&quot;/&gt;&lt;project name=&quot;ns_server&quot; path=&quot;ns_server&quot; revision=&quot;315aa6af297d7139c505e2fa9aa31dd429615a7d&quot;/&gt;&lt;project name=&quot;portsigar&quot; path=&quot;portsigar&quot; revision=&quot;1bc865e1622fb93a3fe0d1a4cdf18eb97ed9d600&quot;/&gt;&lt;project name=&quot;sigar&quot; path=&quot;sigar&quot; revision=&quot;63a3cd1b316d2d4aa6dd31ce8fc66101b983e0b0&quot;/&gt;&lt;project name=&quot;couchbase-examples&quot; path=&quot;couchbase-examples&quot; revision=&quot;544688dc56420faaf6f25946dd4b3ef3f7c15286&quot;/&gt;&lt;project name=&quot;couchbase-python-client&quot; path=&quot;couchbase-python-client&quot; revision=&quot;006c1aa8b76f6bce11109af8a309133b57079c4c&quot;/&gt;&lt;project name=&quot;couchdb&quot; path=&quot;couchdb&quot; revision=&quot;b68b5f40e0911db7de651b457e6a79a5937ff810&quot;/&gt;&lt;project name=&quot;couchdbx-app&quot; path=&quot;couchdbx-app&quot; revision=&quot;1130fa3c1f117527b761497cfa0c15a1e9968808&quot;/&gt;&lt;project name=&quot;couchstore&quot; path=&quot;couchstore&quot; revision=&quot;b5937c4479bf05dcc67264efe19abaf52870a127&quot;/&gt;&lt;project name=&quot;geocouch&quot; path=&quot;geocouch&quot; revision=&quot;849d5443689b1924f097548af864c539bffcc929&quot;/&gt;&lt;project name=&quot;mccouch&quot; path=&quot;mccouch&quot; revision=&quot;88701cc326bc3dde4ed072bb8441be83adcfb2a5&quot;/&gt;&lt;project name=&quot;testrunner&quot; path=&quot;testrunner&quot; revision=&quot;c4e82c929b3c7f2328f234fc9d79b38868268455&quot;/&gt;&lt;project name=&quot;otp&quot; path=&quot;otp&quot; revision=&quot;b6dc1a844eab061d0a7153d46e7e68296f15a504&quot; remote=&quot;erlang&quot;/&gt;&lt;project name=&quot;icu4c&quot; path=&quot;icu4c&quot; revision=&quot;26359393672c378f41f2103a8699c4357c894be7&quot; remote=&quot;couchbase&quot;/&gt;&lt;project name=&quot;snappy&quot; path=&quot;snappy&quot; revision=&quot;5681dde156e9d07adbeeab79666c9a9d7a10ec95&quot; remote=&quot;couchbase&quot;/&gt;&lt;project name=&quot;v8&quot; path=&quot;v8&quot; revision=&quot;447decb75060a106131ab4de934bcc374648e7f2&quot; remote=&quot;couchbase&quot;/&gt;&lt;project name=&quot;gperftools&quot; path=&quot;gperftools&quot; revision=&quot;8f60ba949fb8576c530ef4be148bff97106ddc59&quot; remote=&quot;couchbase&quot;/&gt;&lt;project name=&quot;pysqlite&quot; path=&quot;pysqlite&quot; revision=&quot;0ff6e32ea05037fddef1eb41a648f2a2141009ea&quot; remote=&quot;couchbase&quot;/&gt;&lt;/manifest&gt; JCBC-140

Code breaks when Connection URI is improper

Improvement Major Closed Fixed Public Deepti Dawar Deepti Dawar Wed, 7 Nov 2012 03:18:54 -0600 Thu, 24 Jan 2013 23:46:37 -0600 Tue, 15 Jan 2013 11:45:02 -0600 1.1-dp3 1.1.1 library 0 0 Deepti,<br/> <br/> I think the logs already provide a helpful error message with the stack trace that shows what it tried to parse.<br/> <br/> In the case you provided, the logs would say 2012-11-08 10:29:09.254 WARN com.couchbase.client.vbucket.ConfigurationProviderHTTP: Provided URI <a href="http://192.168.1.105:8091/index.html">http://192.168.1.105:8091/index.html</a> has an unparsable response...skipping<br/> <br/> and then print a stack trace of what was tried to parse (in this case hughe chunks of HTML).<br/> <br/> Also, if you pass in more URIs it would try to get the next one to establish a successful connection.<br/> <br/> I think we could catch it and add a different error message, but then it wont be clear to the user what happened.<br/> <br/> If you think we really should change something here, please reopen the issue and then we can discuss it together at a broader audience!<br/> <br/> Thanks,<br/> Michael Dear Michael,<br/> <br/> As discussed with you, I think the exception here should be caught in a very generic manner that gives the user some information like this - &quot;Connection could not be established - Either the URI provided is incorrect or the host is unavailable&quot;. As the user is not concerned about what is happening beyond the scene and not in the least about what is getting parsed in order for the connection to be established. He is just concerned about the system URL he wants to connect to and the network. Either of these two if unavailable with cause an issue in connection, only that needs to be highlighted back to the user in form of a warning message. It should not abruptly break the code. Hence, I am reopening this defect for further investigation.<br/> <br/> Regards,<br/> Deepti File changed.<br/> Please approve for check in. Hi Michael, <br/> <br/> Please review at <br/> <br/> <a href="http://review.couchbase.org/#/c/23648/2">http://review.couchbase.org/#/c/23648/2</a> <br/> <br/> Regards, <br/> Deepti I think this should be converted to some kind of connection or execution exception, providing more detailed information. Mark, can we have any exceptions like DisplayableException which could be shown to the end user ? <br/> <br/> Can you please give details about the connection or execution exceptions. What is displayed to the user is a sub-component of logging and not of the exception itself. An exception is not an exception unless it represents an exceptional condition; it is up to the code catching the exception to convert this to something the user can understand. <br/> <br/> I am not sure if the Couchbase Java client has classes for &quot;connection&quot; exceptions which indicate a difficulty in communication between client and server, but if there isn&#39;t such a class then it should be created. <br/> <br/> Theroetically <br/> <br/> class CouchbaseConnectionException extends Exception { <br/> &nbsp;&nbsp;&nbsp;&nbsp;.... <br/> } <br/> <br/> Typically it seems most of the Couchbase exceptions in the Java client are RuntimeExceptions or ExecutionExceptions (i.e. generic &quot;unchecked&quot; exceptions). <br/> <br/> Matt and Michale might have more knowledge of the various classes contained therein and their suitability of use. I agree. There should be in-house exception classes meant to facilitate this purpose. <br/> A user working with couchbase sdk, should have the feasibility of using those exception classes for understanding various scenarios and then acting accordingly. <br/> <br/> Please let us know if there are any exception classes for communicating the connection exception to the user in which we can embed a custom message stating what happened. Please review at <br/> <br/> <a href="http://review.couchbase.org/#/c/23648/">http://review.couchbase.org/#/c/23648/</a> Changes checked in for review in gerrit. Rank 9494 Time In Status