Re: [Exist-open] valid characters for collection names

[Adam R. <ada...@de...>]

Yes, you will also need to call build.sh dist-war to generate a new .war =
file.

Let me know how it goes and if all is sucessful I will get it into the =
codebase.

Thanks Adam.

-----Original Message-----
From: Andrew Nagy [mailto:and...@vi...]
Sent: Mon 26/11/2007 15:18
To: Adam Retter
Cc: Wolfgang; exi...@li...
Subject: RE: [Exist-open] valid characters for collection names
=20
Adam - sorry for the delay, I was away for the Thanksgiving Holiday.

I have never patched exist before.  Can I apply this patch and run the =
build.sh script?  Will that create a new webapp war file?
The patch looks simple enough - should solve my problem.

Thanks!

Andrew

> -----Original Message-----
> From: Adam Retter [mailto:ada...@de...]
> Sent: Wednesday, November 21, 2007 1:00 PM
> To: Andrew Nagy
> Cc: Wolfgang; exi...@li...
> Subject: RE: [Exist-open] valid characters for collection names
>
> Okay then attached is a quick and dirty patch that is not well tested,
> but it does allow me to access collections through REST using the
> encoded URL form.
>
> I have applied it to GET, POST and HEAD - it probably will also need =
to
> be extended to DELETE and PUT eventually.
>
> Any improvements or better suggestions from anyone are welcome.
>
> Let me know how you get on with it...
>
>
> On Wed, 2007-11-21 at 12:02 -0500, Andrew Nagy wrote:
> > >
> > > I think there may be a bigger problem -
> > >
> > > Using 1.1.2 I just created two collections - "/db/1:1" and
> "/db/2,2"
> > > - I can access neither of these by REST, if I try the URL
> > > http://localhost:8080/exist/servlet/db/2,2 I get a HTTP 500 and if
> I
> > > try
> > > http://localhost:8080/exist/servlet/db/2%2C2 I get a HTTP 400
> error.
> > >
> > > If I browse the collections through rest -
> > > http://localhost:8080/exist/servlet/db/ It shows me the collection
> > > names in a URL encoded form, im not sure that this should be the
> > > case either.
> > >
> > > So there are obviously some issues around this. Andrew does this
> > > summarise your problems? If so we can now look at fixing them :-)
> > > Then another release candidate of 1.1.2 may be in order...
> >
> > Yep - you got it!  Let me know if I can help in anyway - especially
> with testing.  I haven't explored the exist code yet so I am not very
> familiar with it.
> >
> > Thanks!
> >
> > Andrew
> >
> >