Opened 11 years ago

Closed 8 years ago

#368 closed defect (fixed)

expand/format parameters should not be allowed in CCRS

Reported by: Dimitar Misev Owned by: Bang Pham Huu
Priority: minor Milestone: 9.2
Component: secore Version: 8.4
Keywords: Cc: Piero Campalani
Complexity: Easy

Description

Using expand or format parameters in the inner CRS URLs of a compound CRS URL should throw an exception.

Change History (12)

comment:1 by Dimitar Misev, 11 years ago

Milestone: 8.59.0-alpha1

comment:2 by abeccati, 10 years ago

Milestone: 9.0-beta19.0

Milestone 9.0-beta1 deleted

comment:3 by Dimitar Misev, 8 years ago

Owner: changed from Dimitar Misev to Bang Pham Huu
Status: newassigned

comment:4 by Dimitar Misev, 8 years ago

Milestone: 9.0.x9.2

comment:5 by Bang Pham Huu, 8 years ago

I think there is a exclusive problem if "expand" parameter is not allowed in compound CRS as you also want to add "expand=full" to get the full resolved CRS in here http://rasdaman.org/ticket/1106.

comment:6 by Dimitar Misev, 8 years ago

But these are not expanded by default anyway, check http://localhost:8080/def/crs-compound?1=http://localhost:8080/def/crs/EPSG/0/4326&2=http://localhost:8080/def/crs/EPSG/0/4327 for example.

If I add an expand parameter to some single CRS, e.g. http://localhost:8080/def/crs-compound?1=http://localhost:8080/def/crs/EPSG/0/4326?expand=full&2=http://localhost:8080/def/crs/EPSG/0/4327 it has no effect and is simply put in the componentReferenceSystem:

<?xml version="1.0" encoding="ISO-8859-1"?>
<CompoundCRS xmlns:gml='http://www.opengis.net/gml/3.2'
   xmlns:epsg='urn:x-ogp:spec:schema-xsd:EPSG:0.1:dataset'
   xmlns:xlink='http://www.w3.org/1999/xlink'
   gml:id='crs' xmlns:gco="http://www.isotc211.org/2005/gco" xmlns:gmd="http://www.isotc211.org/2005/gmd">
   <metaDataProperty>
      <epsg:CommonMetaData>
         <epsg:type>compound</epsg:type>
      </epsg:CommonMetaData>
   </metaDataProperty>
   <scope>not known</scope>
   <identifier codeSpace='OGC'>http://localhost:8080/def/crs-compound?1=http://localhost:8080/def/crs/EPSG/0/4326?expand=full%262=http://localhost:8080/def/crs/EPSG/0/4327</identifier>
   <name>geographic3D to geographic2D / WGS 84 (geographic 3D)</name>
   <componentReferenceSystem xlink:href='http://localhost:8080/def/crs/EPSG/0/4326?expand=full'/>
   <componentReferenceSystem xlink:href='http://localhost:8080/def/crs/EPSG/0/4327/'/>
</CompoundCRS>

Please check if this is a valid definition btw.

in reply to:  6 ; comment:7 by Dimitar Misev, 8 years ago

Replying to dmisev:

Please check if this is a valid definition btw.

This is a low priority ticket and actually I'm not sure if it's necessary at all. But the validity of the compound CRS definition should be checked.

in reply to:  7 comment:8 by Bang Pham Huu, 8 years ago

Replying to dmisev:

Replying to dmisev:

Please check if this is a valid definition btw.

This is a low priority ticket and actually I'm not sure if it's necessary at all. But the validity of the compound CRS definition should be checked.

:) high to me and I think one can not use Rasdaman but when SECORE was deployed publicly then it is another problem and I'm interested in to check valid.

comment:9 by Dimitar Misev, 8 years ago

I mean it's low prio regarding the original intention of the ticket, it's not so critical if no exception is thrown. The priority is suitably minor, there are more important bugs.

Using expand or format parameters in the inner CRS URLs of a compound CRS URL should throw an exception.

comment:10 by Bang Pham Huu, 8 years ago

Resolution: fixed
Status: assignedclosed

Patch was accepted, I will close this ticket.

comment:11 by Bang Pham Huu, 8 years ago

Resolution: fixed
Status: closedreopened

comment:12 by Bang Pham Huu, 8 years ago

Resolution: fixed
Status: reopenedclosed
Note: See TracTickets for help on using tickets.