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));
>
>
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.