Re: endpoint case
Michael B Allen <[email protected]>
| Newsgroups | gmane.network.samba.java |
|---|---|
| Message-ID | <CAGMFw4gnLnPGkz0_peHbeYj_XVSDAZzR_+0zTW4tyySdZJwxxA@mail.gmail.com> |
Hi vianney, I have made this change in my local copy. As long as I don't experience any problems with it, it should make it out in the next release (which should be in a few months or so ...). Thanks for the feedback. Mike -- Michael B Allen Java Active Directory Integration http://www.ioplex.com/ On Wed, Nov 9, 2011 at 6:36 PM, vianney <[email protected]> wrote: > In windows the case is not a big deal but in linux systems, sometimes they > expect uppercase. So i changed the following in DcerpcBinding.java to allow that > uppercase string to reach them: > > From: > > void setOption(String key, Object val) throws DcerpcException { > if (key.equals("endpoint")) { > endpoint = val.toString().toLowerCase(); > if (endpoint.startsWith("\\pipe\\")) { > String iface = (String)INTERFACES.get(endpoint.substring(6)); > > > To: > > void setOption(String key, Object val) throws DcerpcException { > if (key.equals("endpoint")) { > // VNA: forcefully transforming the endpoint into lowercase limits the > choice of the library's user. > // Changing it here so that the developer gets to make that choice. > endpoint = val.toString(); > if (endpoint.toLowerCase().startsWith("\\pipe\\")) { > String iface = (String)INTERFACES.get(endpoint.toLowerCase().substring(6)); > >