felix-users mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Guillaume Nodet <gno...@apache.org>
Subject Re: Weird backslash/escape behavior in Felix using Karaf
Date Fri, 11 May 2018 07:07:53 GMT
Could you try the same kind of tests using the TypedProperties from felix
utils ?  It does have an interpolation layer which could be the culprit by
escaping twice at the end...

2018-05-10 22:10 GMT+02:00 Harrison Tarr <harrison.tarr@connexta.com>:

> Carsten Ziegeler commented on my ticket in JIRA and mentioned that he
> doesn't think it's a bug with the ConfigurationHandler. I have added an
> additional test to the ConfigurationHandler where I write a config, read it
> and write it again and have confirmed that it stays consistent. (I have
> updated my PR with the new code) I wonder where this inconsistency is
> happening? It's almost like it reads it twice before writing it. For
> example, the written config should have `\\\\` and then it is read in and
> it gets turned into `\\` but then it is again written out as `\\`. This
> makes me think it is reading `\\\\` turning it into `\\` (which makes
> sense) but then somewhere it is reading it again and turning it into `\`
> which then gets written out as `\\` and then eventually turned into
> nothing.  Any ideas where that could be happening?
>
> Harrison
>
> -----Original Message-----
> From: Harrison Tarr <harrison.tarr@connexta.com>
> Sent: Thursday, May 10, 2018 11:53 AM
> To: users@felix.apache.org
> Subject: RE: Weird backslash/escape behavior in Felix using Karaf
>
> Hi Guillaume,
>
> I think that's exactly the case. I put some breakpoints in that copy of
> the ConfigurationHandler and the read method was hit. It still looks like
> it doesn't necessarily handle the backslashes the way I would expect. Does
> anyone have any insight into that?
>
> Harrison
>
> -----Original Message-----
> From: Guillaume Nodet <gnodet@apache.org>
> Sent: Wednesday, May 9, 2018 6:57 PM
> To: users@felix.apache.org
> Subject: Re: Weird backslash/escape behavior in Felix using Karaf
>
> FileInstall now uses the TypedProperties class to read the configuration.
> It has its own copy of the ConfigurationHandler class:
>
> https://github.com/apache/felix/tree/trunk/utils/src/
> main/java/org/apache/felix/utils/properties
>
> Maybe that's the reason why you didn't hit your code ?
>
> Guillaume
>
> 2018-05-10 3:27 GMT+02:00 Harrison Tarr <harrison.tarr@connexta.com>:
>
> > Hi all,
> >
> > I'm trying to resolve a bug with a configuration. The issue is with
> > the handling of backslashes. My config is a Windows path, eg:
> > 'fileRoot="\\127.0.0.1\file\path\"'. I did some experimenting in the
> > Felix code and I seemed to have found a work around that didn't break
> > any unit tests; I realized after, once building the whole module, that
> > it did in fact break unit tests of a different class. I noticed that
> > the behavior that did not agree with what I expected was in the
> > `readQuoted` method of the `ConfigurationHandler` class. (Pull request
> and bug JIRA for reference:
> > https://github.com/apache/felix/pull/136 https://issues.apache.org/
> > jira/browse/FELIX-5849)
> >
> > Anyway, I loaded my customized module into my project and promptly
> > noticed that I was still having the issue - in fact, my new code
> > wasn't even being hit. It appears that Karaf doesn't use the `read`
> > methods in the `ConfigurationHandler` class, but does use the `write`
> > methods. Does anyone know where else the config could be getting read
> from?
> >
> > Best Regards,
> >
> > Harrison Tarr
> >
>
>
>
> --
> ------------------------
> Guillaume Nodet
>



-- 
------------------------
Guillaume Nodet

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message