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 a ticket?):

https://github.com/geoserver/geoserver/commit/b00ed3c5e86006a009ba44dd4c4f4f3767b68999

The issue is that the validator is rejecting xml invalid names for all
catalog components, in our
case we have a store and coverage with a xml invalid name, but I don't see
a good reason
why they should be rejected:

015-01-21 16:59:09,950 ERROR [geoserver.rest] - :Validation failed with 1
errors.  First error message is: Name '137105_CSKS3_DGM_B_HR_01_VV_
RA_FF_20141017035343_20141017035413' is not legal XML'
2015-01-21 16:59:09,950 ERROR [geoserver.rest] -
org.geoserver.rest.RestletException: java.lang.RuntimeException: Validation
failed with 1 errors.  First error message is: Name
'137105_CSKS3_DGM_B_HR_01_VV_RA_FF_20141017035343_20141017035413' is not
legal XML'
    at org.geoserver.rest.ReflectiveResource.handleException(
ReflectiveResource.java:326)

In particular, store and coverage names are not going to be used a xml
elements,
unlike feature type ones, where in fact the issue is real.

Is it just a case of overzelous checking? This change is going to prevent
many
in the metoc world from upgrading, as the above name structure is pretty
common (coverage names are normally following some structured convention
where each element has a specific meaning (btw, the above is not valid as
a xml element name is because it starts with a number).

I'm proposing to restrict it only to FeatureType elements.

Cheers
Andrea


-- 
==
GeoServer Professional Services from the experts! Visit
http://goo.gl/NWWaa2 for more information.
==

Ing. Andrea Aime
@geowolf
Technical Lead

GeoSolutions S.A.S.
Via Poggio alle Viti 1187
55054  Massarosa (LU)
Italy
phone: +39 0584 962313
fax: +39 0584 1660272
mob: +39  339 8844549

http://www.geo-solutions.it
http://twitter.com/geosolutions_it

*AVVERTENZE AI SENSI DEL D.Lgs. 196/2003*

Le informazioni contenute in questo messaggio di posta elettronica e/o
nel/i file/s allegato/i sono da considerarsi strettamente riservate. Il
loro utilizzo è consentito esclusivamente al destinatario del messaggio,
per le finalità indicate nel messaggio stesso. Qualora riceviate questo
messaggio senza esserne il destinatario, Vi preghiamo cortesemente di
darcene notizia via e-mail e di procedere alla distruzione del messaggio
stesso, cancellandolo dal Vostro sistema. Conservare il messaggio stesso,
divulgarlo anche in parte, distribuirlo ad altri soggetti, copiarlo, od
utilizzarlo per finalità diverse, costituisce comportamento contrario ai
principi dettati dal D.Lgs. 196/2003.



The information in this message and/or attachments, is intended solely for
the attention and use of the named addressee(s) and may be confidential or
proprietary in nature or covered by the provisions of privacy act
(Legislative Decree June, 30 2003, no.196 - Italy's New Data Protection
Code).Any use not in accord with its purpose, any disclosure, reproduction,
copying, distribution, or either dissemination, either whole or partial, is
strictly forbidden except previous formal approval of the named
addressee(s). If you are not the intended recipient, please contact
immediately the sender by telephone, fax or e-mail and delete the
information in this message that has been received in error. The sender
does not give any warranty or accept liability as the content, accuracy or
completeness of sent messages and accepts no responsibility  for changes
made after they were sent or for other risks which arise as a result of
e-mail transmission, viruses, etc.

-------------------------------------------------------
------------------------------------------------------------------------------
New Year. New Location. New Benefits. New Data Center in Ashburn, VA.
GigeNET is offering a free month of service with a new server in Ashburn.
Choose from 2 high performing configs, both with 100TB of bandwidth.
Higher redundancy.Lower latency.Increased capacity.Completely compliant.
http://p.sf.net/sfu/gigenet
_______________________________________________
Geoserver-devel mailing list
Geoserver-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geoserver-devel

Reply via email to