Regression in 2.7.x, valid coverage names are rejected because not valid xml names

classic Classic list List threaded Threaded
42 messages Options
Regression in 2.7.x, valid coverage names are rejected because not valid xml names – Hi, I've got a report of a regression in 2.7.x that has been caused by this commit by David Winslow (for which I cannot find a discussion, nor ...
Hi, I made a short test with 2.7-beta and found that validation does not actually work just as planned from the UI. - I changed cov...
I think that adding "soft" validation for different services is going to be out of the scope of what I can do for this iteration. The g...
On Thu, Jan 22, 2015 at 7:51 PM, David Winslow <dwinslow@...> wrote: > I think that adding "soft" validation for different ...
Here is a lira covering a subset of this topic (references to workspace styles): http://jira.codehaus.org/browse/GEOS-6811 -- Jody Garnett ...
This appears to be describing an unrelated bug. However, the URLEncoder test that I proposed would also prohibit colons so, back to the drawing ...
Slightly related - or at least the fix is related. If we change how names are handled it is good to preserve backwards compatibility... Here w...
I'm going to revive this thread a bit as I'm now attempting to start work on [GEOS-6811] <http://jira.codehaus.org/browse/GEOS-6811> and it...
On Mon, Jan 26, 2015 at 11:36 PM, Travis Brundage < tbrundage@...> wrote: > I'm going to revive this thread a bit as I'm now attempti...
I've confirmed that correcting the character handling in REST config addresses the problem that I was trying to solve. It's a pretty small patc...
On Tue, Jan 27, 2015 at 5:46 PM, David Winslow <dwinslow@...> wrote: > I've confirmed that correcting the character handling in REST ...
I am going to write up a proposal for taking our internal names and making them "safe" at the protocol boundaries. The idea may not com...
Here is the proposal: https://github.com/geoserver/geoserver/wiki/GSIP-127 ---------------------------------------------------------------------...
The proposal to automatically rewrite names seems pretty complicated. I have a number of concerns but I want to focus on just one: The goal i...
So how can I kick this along David? Would you be content if we use this class to: * Introduce warnings during configuration load * Lock dow...
Updated the proposal to be more predictable / less ambitious ... https://github.com/geoserver/geoserver/wiki/GSIP-127 -- Jody Garnett On 2...
On Fri, Jan 30, 2015 at 1:43 AM, Jody Garnett <jody.garnett@...> wrote: > Updated the proposal to be more predictable / less ambitiou...
Establishing a new level of control here is a bit of a reach when we are trying to close a validation gap? How about I take migration out of t...
On Fri, Jan 30, 2015 at 4:20 PM, Jody Garnett <jody.garnett@...> wrote: > Establishing a new level of control here is a bit of a reac...
So do we do a warning along the lines of "13A is not valid for all services, will be published as the "_13A" for WFS" -- ...
On Fri, Jan 30, 2015 at 4:50 PM, Jody Garnett <jody.garnett@...> wrote: > So do we do a warning along the lines of "13A is not v...
I guess I see where the friction is coming from, you are focused on the external name used by WMS and I am thinking of keeping our internal catal...
On Fri, Jan 30, 2015 at 7:29 PM, Jody Garnett <jody.garnett@...> wrote: > I guess I see where the friction is coming from, you are fo...
So we are left with the question of how to notify admins for layers that are disabled for a specific service. As for the styles name change (d...
I am uncomfortable with the idea of silently disabling feature type layers - if the user has turned on both WFS and WMS publication. This prop...
Based on the 29 open tickets which deal with non-simple resource names we have real issues with them. Could we at least try to emphasize in the do...
Hi, Wanted to throw a little experience we had with names and how we solved them; We have an upload capability with our application for user...
I am focused on a slightly different issue than WCS - styles that no longer have a valid name after 2.7 changes. The style names have been baked ...
On Fri, Jan 30, 2015 at 4:56 PM, Jody Garnett <jody.garnett@...> wrote: > I am focused on a slightly different issue than WCS - style...
Okay I will update the proposal accordingly... -- Jody Garnett On 30 January 2015 at 07:58, Andrea Aime <andrea.aime@...> wrote: ...
I can see that being maddening for administrators who were not quite paying attention. Easier to publish something, and they can double back if t...
On Fri, Jan 30, 2015 at 4:33 PM, Andrea Aime <andrea.aime@...> wrote: > On Fri, Jan 30, 2015 at 4:20 PM, Jody Garnett <jody.garnet...
The hope is to prevent the creation of more "unsafe" names on disk (so I do not expect to see foo__bar and foo::bar). The goal was to ...
Hi, I was browsing old tickets from Jira and noticed that there are loads of tickets telling that this or that does not work with REST if lay...
Maybe link the ticket with a depends on. It will help gather up for test cases. -- Jody Garnett On 29 January 2015 at 11:52, Rahkonen Jukk...
Hi, I found 29 such open tickets and added the list into GSIP-127. I found much dust from the Jira as well. -Jukka- Jody Garnett wrote: ...
On Thu, Jan 22, 2015 at 12:26 PM, Andrea Aime <andrea.aime@...> wrote: > Hi, > I've got a report of a regression in 2.7.x that ha...
On Thu, Jan 22, 2015 at 12:30 PM, Andrea Aime <andrea.aime@...> wrote: > For the record, found the pull, not the discussion: > ht...
On Thu, Jan 22, 2015 at 12:37 PM, Andrea Aime <andrea.aime@...> wrote: > On Thu, Jan 22, 2015 at 12:30 PM, Andrea Aime < > and...
Ah hem, errata corrige, the commit in question was also silently backported to 2.6.x, so we have the same regression on the stable series... ...
It seems a bit drastic to me too. If we end up deciding to adjust this validation I am available to put together a patch. Workspace names need...
On Thu, Jan 22, 2015 at 3:24 PM, David Winslow <dwinslow@...> wrote: > It seems a bit drastic to me too. If we end up deciding to adj...