[Geoserver-devel] GetMapIntegration tests issue

2012-03-22 Thread Niels Charlier
Hi Justin, I was wondering if you were ever able to figure this out or created a jira issue for it: http://sourceforge.net/mailarchive/forum.php?thread_name=CAEwWEk3JZYA9jQovKYUbi9uj6gGLCp9TeZZVTD7M_6xw-CvTPQ%40mail.gmail.comforum_name=geoserver-devel I've been having exactly the same problem

[Geoserver-devel] apologies : major commit screw-up

2012-05-08 Thread Niels Charlier
it was supposed to be in one single commit with meaningful comments and meaningfull diff information. I am really sorry this happened. With Regards, Niels Charlier -- Live Security Virtual Conference Exclusive live event will cover

Re: [Geoserver-devel] apologies : major commit screw-up

2012-05-08 Thread Niels Charlier
commit with meaningful comments and meaningfull diff information. I am really sorry this happened. With Regards, Niels Charlier -- Live Security Virtual Conference Exclusive live event will cover all the ways

Re: [Geoserver-devel] apologies : major commit screw-up

2012-05-09 Thread Niels Charlier
Okay, it's been done. Everything was reverted back and then one single readable commit All these revisions can now just be ignored That has the advantage of clarity On 9 mei 2012 07:12 Niels Charlier ni...@scitus.be wrote: Okay, I can do that today no problem, but then you will have to wait

[Geoserver-devel] starting 3D work

2012-05-21 Thread Niels Charlier
Hi everyone, I am in the process of initiating work on the extension of the 3D features in geotools and geoserver. As I understand there have been some recent developments in this area, including the support of W3DS requests and fixing the postgis 3d back-end. At this point, as I understand, the

Re: [Geoserver-devel] starting 3D work

2012-05-21 Thread Niels Charlier
Hi, Thanks for your comments. 1) support for a 3D WFS Bounding Box, Do you want to filter data by a 3D BB on GetFeature requests for example? or do you want to have a 3D BB computed and returned by the server, for example, in the GetCapabilities? Exactly this: Filter data by a 3D

[Geoserver-devel] 3D geometries property files

2012-05-28 Thread Niels Charlier
Hi, I am trying to write some unit tests in geoserver that use 3d geometries, and was looking for examples but cannot actually find much 3d geometry testing on the geoserver side. I tried using property files with WKT, however the WKT format for 3d geometries doesn't seem to be supported, for

Re: [Geoserver-devel] 3D geometries property files

2012-05-30 Thread Niels Charlier
As far as I know the JTS WKT reader will respect a third coordinate. For instance if i parse POINT(1 2 3) I do get back a Point that has an underlying coordinate that maintains a z value of 3. Not sure about the other geometries but the impression I got from Martin is that its

Re: [Geoserver-devel] 3D geometries property files

2012-06-04 Thread Niels Charlier
might be a different story. Long story short through I believe you can use property datastore as it is today to test 2.5d as long as you are not doing transactions. -Justin On Mon, May 28, 2012 at 1:15 AM, Niels Charlier ni...@scitus.be mailto:ni...@scitus.be wrote

Re: [Geoserver-devel] [Geotools-devel] 3D geometries property files

2012-06-04 Thread Niels Charlier
The problem is actually not with wtkwriter but with the GML encoder. It outputs 2 dimensions and I need to get all three. That's a problem for WFS supporting 3 dimensions. For example, in PointTypeBinding I see the code: if (pos.equals(name.getLocalPart())) { Point point = (Point)

[Geoserver-devel] question of feasibility - 1d geometry

2012-08-20 Thread Niels Charlier
think this would be? Kind Regards Niels Charlier -- Live Security Virtual Conference Exclusive live event will cover all the ways today's security and threat landscape has changed and how IT managers can respond

[Geoserver-devel] CSW Module Structure

2012-12-05 Thread Niels Charlier
question posed was whether 'internal store' should be renamed to 'default store'. Kind Regards Niels Charlier -- LogMeIn Rescue: Anywhere, Anytime Remote support for IT. Free Trial Remotely access PCs and mobile devices

Re: [Geoserver-devel] CSW Module Structure

2012-12-07 Thread Niels Charlier
Right... i guess it would help to know what the classpath overhead is in terms of additional dependencies, etc... of the internal store is. As far as i can see it looks to be pretty minimal, no additional libs and just a handful of classes. @Niels: do I have that right? Yes, i only depends

Re: [Geoserver-devel] CSW Module Structure

2012-12-07 Thread Niels Charlier
Could the current work be committed, so that we can have a look at it? https://github.com/NielsCharlier/geoserver.git branch: cswpublic relies on https://github.com/NielsCharlier/geotools.git branch: csw -- LogMeIn

Re: [Geoserver-devel] CSW Module Structure

2012-12-10 Thread Niels Charlier
Okay so I am going to conclude that we go with Justin's proposal to put the internal store in to core. The other thing is, should I keep its name or rename it to DefaultCatalogStore? Regards Niels On 12/07/2012 05:50 PM, Andrea Aime wrote: On Fri, Dec 7, 2012 at 2:30 PM, Niels Charlier ni

Re: [Geoserver-devel] app-schema WMS sld filtering

2013-01-15 Thread Niels Charlier
Hi Victor, Unfortunately, functions throw this kind of exception whenever something goes wrong in the evaluation of an argument, this is kind of misleading but it has nothing to do with the data type. If the field HASRELATEDACTIVITY does not even exist as

Re: [Geoserver-devel] review of csw pull request

2013-01-23 Thread Niels Charlier
, if there are any more questions, please ask. Kind Regards Niels Charlier -- Master Visual Studio, SharePoint, SQL, ASP.NET, C# 2012, HTML5, CSS, MVC, Windows 8 Apps, JavaScript and much more. Keep your skills current

Re: [Geoserver-devel] review of csw pull request

2013-01-24 Thread Niels Charlier
So if I understand this correctly this means two additional changes. 1. Remove the ElementSet argument from CatalogStore.getRecords() method. 2. Remove CatalogStore.translateProperty() (1) is trivial, more or less all calls to it passed in null. Made that change and tests are happy.

Re: [Geoserver-devel] review of csw pull request

2013-01-28 Thread Niels Charlier
Hi Justin, In response to your questions. On 01/25/2013 05:24 PM, Justin Deoliveira wrote: 1. This method wold never be called by client code, it is more an internal method used to the mapping? I think it is right that it's probably not really useful to the client code, however it is used

Re: [Geoserver-devel] review of csw pull request

2013-02-07 Thread Niels Charlier
On 02/06/2013 06:22 PM, Andrea Aime wrote: The idea that there are two separate ways to translate between the internal data model and the exposed data model bugs me, seems redundant and needlessly complicated. How do you mean there are two separate ways to translate between internal and

Re: [Geoserver-devel] review of csw pull request

2013-02-07 Thread Niels Charlier
On 02/07/2013 02:02 PM, Andrea Aime wrote: On Thu, Feb 7, 2013 at 1:57 PM, Niels Charlier ni...@scitus.be mailto:ni...@scitus.be wrote: On 02/06/2013 06:22 PM, Andrea Aime wrote: The idea that there are two separate ways to translate between the internal data model

Re: [Geoserver-devel] review of csw pull request

2013-02-12 Thread Niels Charlier
Hi Justin, Sorry for the late response, lost track of this a bit. Rather than being needlessly complicated I believe I have caused confusion by giving these two methods the same name - I should not have done that. What they have in common is that they both convert from Name to

Re: [Geoserver-devel] SLD filters complex feature types

2013-03-11 Thread Niels Charlier
Hello Akos, Do I get this right that your are implementing this AixmDataStore which returns complex features, in other words you are not using app-schema but building your own complex features? If so, you shouldn't be using the interface DataStore because it is derived from

[Geoserver-devel] Security improvements

2013-06-11 Thread Niels Charlier
Hello everyone, I am currently having a look at what needs to happen to implement some additional features and improvements in the security system. The first thing is making rules that combine layers and services, which is now impossible. This seems like a pretty straight-forward improvement

Re: [Geoserver-devel] Security improvements

2013-06-14 Thread Niels Charlier
Hello Andrea, That actually seems to be what they are looking for, filter based security was another thing they are interested in. How would this plugin integrate with GeoWebCache? Cheers Niels On 14/06/13 10:44, Andrea Aime wrote: On Wed, Jun 12, 2013 at 11:49 PM, Niels Charlier ni

[Geoserver-devel] GSIP 98 - Promote CSW Module to Extensions

2013-07-24 Thread Niels Charlier
Hi, We would like to move the CSW module from community to extensions. Here is the proposal: http://geoserver.org/display/GEOS/GSIP+98+-+Promote+CSW+Module+to+Extensions Feedback more than welcome. Kind Regards Niels Charlier

Re: [Geoserver-devel] GSIP 98 - Promote CSW Module to Extensions

2013-08-07 Thread Niels Charlier
Andrea, Solved most of the issues in this pull request: https://github.com/geoserver/geoserver/pull/289 Have a look. Cheers Niels On 27/07/13 18:16, Andrea Aime wrote: On Thu, Jul 25, 2013 at 12:25 AM, Niels Charlier ni...@scitus.be mailto:ni...@scitus.be wrote: Hi, We would like

Re: [Geoserver-devel] GSIP 98 - Promote CSW Module to Extensions

2013-08-12 Thread Niels Charlier
Are we ready to vote on this now? Cheers Niels On 07/08/13 23:40, Niels Charlier wrote: Andrea, Solved most of the issues in this pull request: https://github.com/geoserver/geoserver/pull/289 Have a look. Cheers Niels On 27/07/13 18:16, Andrea Aime wrote: On Thu, Jul 25, 2013 at 12:25 AM

Re: [Geoserver-devel] CSW did not make it into extension by RC1

2013-08-19 Thread Niels Charlier
Yes, a pull request has just been made for this: https://github.com/geoserver/geoserver/pull/299 Kind Regards Niels On 18/08/13 14:52, Andrea Aime wrote: Hi, I was looking at the release notes and noticed that CSW is not yet in extension despite the positive vote for the associated GSIP Do

Re: [Geoserver-devel] CSW did not make it into extension by RC1

2013-08-19 Thread Niels Charlier
Done https://github.com/geoserver/geoserver/pull/300 On 19/08/13 16:15, Andrea Aime wrote: On Mon, Aug 19, 2013 at 4:08 PM, Niels Charlier ni...@scitus.be mailto:ni...@scitus.be wrote: Yes, a pull request has just been made for this: https://github.com/geoserver/geoserver/pull/299

Re: [Geoserver-devel] [Hudson] Build failed in Hudson: geoserver-master-deploy #3649

2013-08-19 Thread Niels Charlier
I don't actually understasnd what is going wrong here... Can anyone interpret the output? Cheers Niels On 19/08/13 17:55, Hudson wrote: See http://hudson.opengeo.org/hudson/job/geoserver-master-deploy/3649/ -- [...truncated 27383 lines...] [INFO]

Re: [Geoserver-devel] CSW did not make it into extension by RC1

2013-08-22 Thread Niels Charlier
Thank you, Andrea On 22/08/13 18:47, Andrea Aime wrote: On Thu, Aug 22, 2013 at 6:42 PM, Niels Charlier ni...@scitus.be mailto:ni...@scitus.be wrote: On 22/08/13 18:16, Andrea Aime wrote: On Tue, Aug 20, 2013 at 10:48 PM, Niels Charlier ni...@scitus.be mailto:ni...@scitus.be

Re: [Geoserver-devel] CSW did not make it into extension by RC1

2013-08-22 Thread Niels Charlier
On 22/08/13 18:16, Andrea Aime wrote: On Tue, Aug 20, 2013 at 10:48 PM, Niels Charlier ni...@scitus.be mailto:ni...@scitus.be wrote: Yes you were right, sorry about that, a fix has been pull requested. 2.4.x request updated. Hi, had a quick look at the extension, now it contains

Re: [Geoserver-devel] master failing on CSW

2013-09-02 Thread Niels Charlier
Hi Jody, I believe this is because the build was started before the dependant geotools patch was pulled. I think because nothing changed since then on the GS repo, hudson didn't start a new build. Can you force it to try and build again? I am quite certain it would be solved then. Cheers

Re: [Geoserver-devel] FOSS4G 2013 Code Sprint this Sunday

2013-09-23 Thread Niels Charlier
On 23/09/13 12:25, Andrea Aime wrote: On Fri, Sep 20, 2013 at 10:57 PM, Sen, Marcus A. m...@bgs.ac.uk mailto:m...@bgs.ac.uk wrote: I'm not sure how many people are thinking of attending the FOSS4G Code sprint this Sunday here in Nottingham but I gather there are no particular plans

Re: [Geoserver-devel] FOSS4G 2013 Code Sprint this Sunday

2013-09-24 Thread Niels Charlier
On 24/09/13 17:46, Sen, Marcus A. wrote: -Original Message- From: Andrea Aime [mailto:andrea.a...@geo-solutions.it] Sent: 23 September 2013 20:49 To: Niels Charlier Cc: Geoserver-devel Subject: Re: [Geoserver-devel] FOSS4G 2013 Code Sprint this Sunday Better than nothing

Re: [Geoserver-devel] Local and remote references in app-schema

2013-11-28 Thread Niels Charlier
Hi Andrea, On 28/11/13 09:38, Andrea Aime wrote: Very nice. I searched around but could not locate where the valueOf() mandatory xpath function is managed. Do you have pointers? Ah, I don't think this is implemented. I'm also wondering about remote references, does app-schema

Re: [Geoserver-devel] How about prefixing our jar artifacts with gs- ?

2013-11-28 Thread Niels Charlier
+1 definitely good idea On 28/11/13 15:03, Andrea Aime wrote: Hi, this thing has bothered me for some time now... when I look at the WEB-INF/lib contents it's easy to locate GeoTools jars, it's easy to locate imageio-ext ones, but the GeoServer ones are sprinkled here and there. What people

Re: [Geoserver-devel] geopackage community module

2013-12-03 Thread Niels Charlier
Indeed On 03/12/13 16:43, Justin Deoliveira wrote: Hey Jukka, Niels has been working on updating the geotools module to incorporate the latest changes according to the spec. In addition he has been splitting up the GeoServer code into multiple output formats as previously discussed in this

[Geoserver-devel] mbtiles mosaic

2013-12-05 Thread Niels Charlier
Hello list, I am starting to work on build a mosaic coverage reader for an mbtiles database. I am suspecting that the 'mosaicing' logic is already present in geotools. I was wondering if anyone familiar with image-mosaic and image-mosaic-jdbc modules can give me any tips on what to use / point me

[Geoserver-devel] new community module: mbtiles

2013-12-09 Thread Niels Charlier
Hi everyone, I would like to add a community module for the mbtiles format in geoserver. We also recently just started an unsupported mbtiles module in geotools, which lies the the foundations of the format. The geoserver package provides the outputformats for geoserver (this is similar to the

Re: [Geoserver-devel] new community module: mbtiles

2013-12-09 Thread Niels Charlier
On 09/12/13 14:44, Andrea Aime wrote: On Mon, Dec 9, 2013 at 2:35 PM, Niels Charlier ni...@scitus.be mailto:ni...@scitus.be wrote: Hi everyone, I would like to add a community module for the mbtiles format in geoserver. We also recently just started an unsupported mbtiles

Re: [Geoserver-devel] Build failed in Jenkins: geoserver-master #136

2013-12-09 Thread Niels Charlier
Anyone an idea what this means? I don't see how this is related to my changes. I also don't seem to have this problem if I build locally. Regards Niels On 09/12/13 14:43, moni...@boundlessgeo.com wrote: See http://ares.opengeo.org/jenkins/job/geoserver-master/136/changes Changes: [niels]

Re: [Geoserver-devel] Build failed in Jenkins: geoserver-master #136

2013-12-09 Thread Niels Charlier
... Regards Niels On 09/12/13 18:22, Andrea Aime wrote: On Mon, Dec 9, 2013 at 4:09 PM, Niels Charlier ni...@scitus.be mailto:ni...@scitus.be wrote: Anyone an idea what this means? I don't see how this is related to my changes. I also don't seem to have this problem if I build locally

Re: [Geoserver-devel] app-schema WMS test changes

2014-01-13 Thread Niels Charlier
On 13/01/14 09:16, Ben Caradoc-Davies wrote: Rini and Niels, current app-schema WMS tests use JPEG output, so perceptualdiff is heavily affected by changes in JPEG compression artifacts. To fix a build failure in app-schema online tests (rendering the stratchart), I have manually reviewed

[Geoserver-devel] multiple coverages in a grid coverage reader

2014-01-14 Thread Niels Charlier
Hello Mailing List, I am writing a geopkg coverage reader. A geopkg can have many coverages, each of which can have their own envelope, CRS, resolutions, ... Currently the API works as follows. - The GridFormatFactorySpi Service returns a AbstractGridFormat (not the GridFormat interface but an

Re: [Geoserver-devel] multiple coverages in a grid coverage reader

2014-01-27 Thread Niels Charlier
On 27/01/14 17:51, Andrea Aime wrote: What's stopping you from overriding getOriginalEnvelope(String coverageName)? The reason why getOriginalEnvelope() (without the coverage name) was kept was to avoid breaking all of the existing client code just for the sake of NetCDF. If your

[Geoserver-devel] mapinfo tiff files

2014-02-10 Thread Niels Charlier
Hi, I made a patch a few months ago to support tiff files made by mapinfo: these are plain tiff files accompanied by a .tab file with metadata, written in a mapinfo specific format. I integrated this option in the geotiff reader. How it works: if there is no metadata in the tiff (so it is not

Re: [Geoserver-devel] mapinfo tiff files

2014-02-10 Thread Niels Charlier
--- On Mon, Feb 10, 2014 at 2:30 PM, Niels Charlier ni...@scitus.be mailto:ni...@scitus.be wrote: Hi, I made a patch a few months ago to support tiff files made by mapinfo: these are plain tiff files accompanied by a .tab file with metadata, written in a mapinfo specific format

Re: [Geoserver-devel] Regression in CSW, core classes not namespace aware anymore?

2014-02-25 Thread Niels Charlier
Hello Andrea, All the changes that were made to those base classes were done to make those classes generic for different record types / schemes; either because the old version of those classes just assumed DC records and thus needed to be changed to support other types; or to avoid

Re: [Geoserver-devel] Regression in CSW, core classes not namespace aware anymore?

2014-02-28 Thread Niels Charlier
On 28/02/14 11:38, Andrea Aime wrote: On Tue, Feb 25, 2014 at 3:41 PM, Andrea Aime andrea.a...@geo-solutions.it mailto:andrea.a...@geo-solutions.it wrote: On a first glance, I can't see why those two maps in the two classes below use a String as key (localpart) rather than the

[Geoserver-devel] Encode z-dimension in GML2

2014-03-19 Thread Niels Charlier
Hello everyone, I propose the following change: https://github.com/geotools/geotools/pull/402 It allows the z-dimension to be encoded in GML2 whenever it is present in the data (it does now only if the CRS is explicitly 3D). This is already how it works for most of the other output formats

Re: [Geoserver-devel] [Geotools-devel] Encode z-dimension in GML2

2014-03-21 Thread Niels Charlier
On 20/03/14 09:38, Andrea Aime wrote: On Wed, Mar 19, 2014 at 11:17 PM, Niels Charlier ni...@scitus.be mailto:ni...@scitus.be wrote: Hello everyone, I propose the following change: https://github.com/geotools/geotools/pull/402 It allows the z-dimension to be encoded in GML2

Re: [Geoserver-devel] WFS 2.0 Stored Query data store

2014-03-28 Thread Niels Charlier
Hello Gabriel, I have been working on enabling transactions in wfs-ng. On 19/03/14 20:38, Gabriel Roldan wrote: As for transaction support, all the bits are in mostly in there. The one difficulty I found I think was a difference on how JDBC stores manage transactions (as ContentDataStore

Re: [Geoserver-devel] Kicked out of the 2.5.x community build mbtiles and geopackage

2014-04-01 Thread Niels Charlier
Andrea, I didn't realise. The reason is that the gt-mbtiles module wasn't added to the build in the geotools 11.x branch. I have updated this, it should be okay now. Cheers Niels On 01/04/14 11:23, Andrea Aime wrote: Hi, our nightly build has been failing since March 20 because these two

Re: [Geoserver-devel] Kicked out of the 2.5.x community build mbtiles and geopackage

2014-04-01 Thread Niels Charlier
On 01/04/14 12:44, Andrea Aime wrote: On Tue, Apr 1, 2014 at 12:39 PM, Niels Charlier ni...@scitus.be mailto:ni...@scitus.be wrote: Andrea, I didn't realise. The reason is that the gt-mbtiles module wasn't added to the build in the geotools 11.x branch. I have updated

Re: [Geoserver-devel] WFS-NG development

2014-05-05 Thread Niels Charlier
Hi Sampo, Have a look here: https://github.com/geotools/geotools/tree/wfs-ng-improvements Kind Regards Niels On 05/05/14 19:10, Justin Deoliveira wrote: Hey Sampo, Yes work is progressing nicely. Niels is currently working on porting tests from the old wfs module to the wfs-ng one in a

Re: [Geoserver-devel] Status update + merge

2014-05-19 Thread Niels Charlier
Thanks for the merging Sampo, looks good. Cheers Niels On 15/05/14 11:09, Sampo Savolainen wrote: Hi, I've successfully merged my and Niels's work. It was surprisingly easy with only one minor conflict. I noticed that you have discussed having a shared branch for me, Niels and Reni. This

[Geoserver-devel] imagemosaic blank response issues

2014-11-25 Thread Niels Charlier
Hello, A client needed to configure an imagemosaic datastore with a custom made mosaic configuration .properties file. They were having issues: with native JAI the tiled image was showing black boxes and if transparentcolor was set to #000 an java.lang.IllegalStateException: Unsupported

Re: [Geoserver-devel] proposal: Layer with Service Security

2015-01-23 Thread Niels Charlier
On 23-01-15 14:21, Andrea Aime wrote: On Fri, Jan 23, 2015 at 2:11 PM, Niels Charlier ni...@scitus.be mailto:ni...@scitus.be wrote: The proposal has now been changed. Please re-read it and place your comments! And oh, also, I'd say it's important to clarify that a potentially common

[Geoserver-devel] proposal: Layer with Service Security

2015-01-20 Thread Niels Charlier
Hello Group, There has been a request to allow a basic combination of layer and service security in the integrated geoserver security subsystem. I have made a proposal to that end: https://github.com/geoserver/geoserver/wiki/GSIP-125---Layer-with-Service-Security In summary, the proposal is to

[Geoserver-devel] ldap authentication provider role service

2015-02-17 Thread Niels Charlier
, 2015-02-12 19:11 GMT+01:00 Niels Charlier ni...@scitus.be mailto:ni...@scitus.be: Hi, Trying to wrap my head around the LDAP security configuration and I'm looking for someone who knows about this. I've read: http://docs.geoserver.org/latest/en/user/security/tutorials/ldap

Re: [Geoserver-devel] ldap authentication provider role service

2015-02-17 Thread Niels Charlier
Bartolomeoli wrote: Hi Niels, 2015-02-17 15:43 GMT+01:00 Niels Charlier ni...@scitus.be mailto:ni...@scitus.be: Hello Mauro and Mailing List, It appears to me that the LDAP authentication provider and role service currently are unable to be used simultaneously. This is because

Re: [Geoserver-devel] proposal: Layer with Service Security

2015-01-28 Thread Niels Charlier
the reasoning is 1/ Redesigning security from start is an exercise that has already been done. Geofence is a more advanced and flexible security system. It would indeed make no sense to redesign the default security to bring it to the same level. 2/ The design is therefore not completely

Re: [Geoserver-devel] proposal: Layer with Service Security

2015-01-23 Thread Niels Charlier
. Thanks. Kind Regards Niels On 22-01-15 12:08, Andrea Aime wrote: On Tue, Jan 20, 2015 at 3:53 PM, Niels Charlier ni...@scitus.be mailto:ni...@scitus.be wrote: Hello Group, There has been a request to allow a basic combination of layer and service security in the integrated geoserver

Re: [Geoserver-devel] proposal: Layer with Service Security

2015-01-24 Thread Niels Charlier
On 23-01-15 19:15, Andrea Aime wrote: Either that, or you'll end up having to expand all possibilities and maintain that expansion over time as layers get added removed, with catalogs that have hundreds of thousands of items it will simply become un-manageable, meaning the security

Re: [Geoserver-devel] proposal: Layer with Service Security

2015-01-23 Thread Niels Charlier
On 23-01-15 14:19, Andrea Aime wrote: Hi Niels, still does not say anything about UI security changes? Sorry, it sounds as if you are referring to something that was mentioned before but I cannot find this. If you mean the geoserver admin UI, indeed it would have to be updated accordingly.

Re: [Geoserver-devel] proposal: Layer with Service Security

2015-01-24 Thread Niels Charlier
On 24-01-15 15:17, Andrea Aime wrote: On Sat, Jan 24, 2015 at 1:51 PM, Niels Charlier ni...@scitus.be mailto:ni...@scitus.be wrote: On 23-01-15 19:15, Andrea Aime wrote: Either that, or you'll end up having to expand all possibilities and maintain that expansion over time

Re: [Geoserver-devel] proposal: Layer with Service Security

2015-02-10 Thread Niels Charlier
is, a concern both me and Jody apparently share). We're basically open to anticipate part of our work right now, even if we have no pressing need for it, to avoid duplication of work. Cheers Andrea On Tue, Feb 3, 2015 at 9:47 AM, Niels Charlier ni...@scitus.be mailto:ni...@scitus.be wrote

Re: [Geoserver-devel] security regression blocking release

2015-03-18 Thread Niels Charlier
Hi, I have found the issue. The regression seems to be introduced in https://github.com/geoserver/geoserver/commit/c1750a1499fc059ecce153322eef8ff119684881 by n-lagomarsini In DefaultResourceAccessManager method getSecurityFilter, where the layer security tree is converted to a filter, it

Re: [Geoserver-devel] security regression blocking release

2015-03-18 Thread Niels Charlier
On 18-03-15 14:01, Nicola Lagomarsini wrote: sorry for the late reply. We decided to replace the layer security tree with a filter in order to speed up performances when accessing the Catalog with the JDBCConfig plugin. We have not managed to introduce any intentional security regression,

Re: [Geoserver-devel] security regression blocking release

2015-03-18 Thread Niels Charlier
Hello Simone, On 18-03-15 15:23, Simone Giannecchini wrote: I will ask Nicola to clarify further. This work was performed in order to make GeoServer work nicely when JDBCConfig is used and there is a ton of layers around (order of 100K). This means, if you make a fix we need to make sure

Re: [Geoserver-devel] security regression blocking release

2015-03-18 Thread Niels Charlier
the filter is needed and why, and it is not created otherwise. That is much more logical and readable. Regards Niels On 18-03-15 16:41, Andrea Aime wrote: On Wed, Mar 18, 2015 at 12:10 PM, Niels Charlier ni...@scitus.be mailto:ni...@scitus.be wrote: Hi, I have found the issue

Re: [Geoserver-devel] security regression blocking release

2015-03-18 Thread Niels Charlier
On 18-03-15 16:57, Andrea Aime wrote: On Wed, Mar 18, 2015 at 4:51 PM, Niels Charlier ni...@scitus.be mailto:ni...@scitus.be wrote: In my change you can clearly see in the if block when the filter is needed and why, and it is not created otherwise. That is much more logical

Re: [Geoserver-devel] security regression blocking release

2015-03-18 Thread Niels Charlier
. But perhaps I am mistaken. Regards Niels On 18-03-15 16:55, Andrea Aime wrote: org.geoserver.security.impl.On Wed, Mar 18, 2015 at 3:35 PM, Niels Charlier ni...@scitus.be mailto:ni...@scitus.be wrote: That's cool, but I think in the first place there should be a simple test (it does not need

Re: [Geoserver-devel] security regression blocking release

2015-03-18 Thread Niels Charlier
On 18-03-15 17:10, Andrea Aime wrote: Thank Niels, I don't think we have anyone reviewing pull requests from this angle... I invite you to join the team that checks the incoming pull requests and thus help increase the GeoServer code quality Andrea, I didn't mean to criticize this code so

Re: [Geoserver-devel] security regression blocking release

2015-03-18 Thread Niels Charlier
On 18-03-15 17:29, Andrea Aime wrote: Also, I stress that you seem to have a different point of view to things that would be useful when reviewing pull requests, so I renew my invite to help checking the incoming pull requests, be them from external contributors, or core developers. Ben was

Re: [Geoserver-devel] Build failed in Jenkins: geoserver-master #1181

2015-03-13 Thread Niels Charlier
Doubt this is my commit... On 13-03-15 09:42, moni...@boundlessgeo.com wrote: See http://ares.opengeo.org/jenkins/job/geoserver-master/1181/changes Changes: [niels] LDAP Security : upgrade to 1.3.2, fix busy port issue -- [...truncated 3487

Re: [Geoserver-devel] spring security ldap

2015-02-22 Thread Niels Charlier
On 21-02-15 19:36, Andrea Aime wrote: You mean upgrading the version of the ldap test lib? Or both? It seems that for the non test jar we are getting it via a transitive dependency towards org.springframework.security:spring-security-ldap: [INFO] +-

Re: [Geoserver-devel] spring security ldap

2015-02-22 Thread Niels Charlier
On 22-02-15 09:18, Mauro Bartolomeoli wrote: Hi Niels, 2015-02-21 17:46 GMT+01:00 Niels Charlier ni...@scitus.be mailto:ni...@scitus.be: I have found a message on a forum about this online http://forum.spring.io/forum/spring-projects/data/ldap/101924-occasional-error-binding

Re: [Geoserver-devel] ldap authentication provider role service

2015-02-21 Thread Niels Charlier
from its password, which is not such a safe method. Patch: https://github.com/NielsCharlier/geoserver/commit/3b8c4aab7a2ae32b55fd546cdce279267c757938 Thanks! Kind Regards, Niels On 17-02-15 17:47, Mauro Bartolomeoli wrote: Hi Niels, 2015-02-17 15:43 GMT+01:00 Niels Charlier ni...@scitus.be

[Geoserver-devel] spring security ldap

2015-02-21 Thread Niels Charlier
Hi, I believe there is an issue with the LDAP tests in geoserver, although it may depend on platform. Each test an LDAP server is set up locally on port 10389 and automatically shut down at the end. After a few tests it randomly occurs that it takes a while (perhaps about half a minute) to

Re: [Geoserver-devel] proposal: Layer with Service Security

2015-01-23 Thread Niels Charlier
done On 23-01-15 14:21, Andrea Aime wrote: On Fri, Jan 23, 2015 at 2:11 PM, Niels Charlier ni...@scitus.be mailto:ni...@scitus.be wrote: The proposal has now been changed. Please re-read it and place your comments! And oh, also, I'd say it's important to clarify that a potentially

Re: [Geoserver-devel] proposal: Layer with Service Security

2015-01-23 Thread Niels Charlier
I added it. On 23-01-15 15:50, Andrea Aime wrote: On Fri, Jan 23, 2015 at 3:47 PM, Niels Charlier ni...@scitus.be mailto:ni...@scitus.be wrote: On 23-01-15 14:19, Andrea Aime wrote: Hi Niels, still does not say anything about UI security changes? Sorry, it sounds

Re: [Geoserver-devel] proposal: Layer with Service Security

2015-01-23 Thread Niels Charlier
/geoserver/geoserver/wiki/GSIP-120 I am still working on it. Christian On Fri, Jan 23, 2015 at 3:50 PM, Andrea Aime andrea.a...@geo-solutions.it mailto:andrea.a...@geo-solutions.it wrote: On Fri, Jan 23, 2015 at 3:47 PM, Niels Charlier ni...@scitus.be mailto:ni...@scitus.be wrote

Re: [Geoserver-devel] proposal: Layer with Service Security

2015-01-23 Thread Niels Charlier
The proposal has now been changed. Please re-read it and place your comments! On 20-01-15 15:53, Niels Charlier wrote: Hello Group, There has been a request to allow a basic combination of layer and service security in the integrated geoserver security subsystem. I have made a proposal

Re: [Geoserver-devel] App-Schema online tests

2015-05-12 Thread Niels Charlier
Hello Torben, Sorry for my late response. Picking up after a holiday. The oracle online tests are not so straight-forward to set up. Can I be of any assistance or have you sorted it out in the meantime? Regards Niels On 24-04-15 01:50, Torben Barsballe wrote: Excellent, good to know we are

Re: [Geoserver-devel] GeoFence REST API -- Re: geofence geoserver integration

2015-04-09 Thread Niels Charlier
Andrea, Thanks for this suggestion. I can't really find a straight-forward solution to call CXF without the use of a servlet that must be defined in web.xml. And there is only one of those, in web-app where I can't place it because geofence-server is only an extension and you can't force cxf

Re: [Geoserver-devel] GeoFence REST API -- Re: geofence geoserver integration

2015-04-12 Thread Niels Charlier
On 12-04-15 22:03, Andrea Aime wrote: Did you check the possibility of making your wrapper a application context aware, grab the app context, which should be a WebApplicationContext, and call getServletContext on it ? How does that help you add a web servlet programmatically? That helps

Re: [Geoserver-devel] GeoFence REST API -- Re: geofence geoserver integration

2015-04-12 Thread Niels Charlier
need to have the whole of geoserver dependant on CXF. Regards Niels On 10-04-15 17:32, Emanuele Tajariol wrote: Alle 14:15:02 di Friday 10 April 2015, Andrea Aime ha scritto: On Thu, Apr 9, 2015 at 10:26 PM, Niels Charlier ni...@scitus.be wrote: Andrea, Thanks for this suggestion. I can't

Re: [Geoserver-devel] GSIP 120, REST API for access control

2015-04-12 Thread Niels Charlier
Is a user/role rest API also planned? Regards Niels On 17-10-14 17:04, Christian Mueller wrote: https://github.com/geoserver/geoserver/wiki/GSIP-120-REST-API-for-access-control Please comment and vote. Cheers -- DI Christian Mueller MSc (GIS), MSc (IT-Security) OSS Open Source Solutions

Re: [Geoserver-devel] GeoFence REST API -- Re: geofence geoserver integration

2015-04-14 Thread Niels Charlier
servlet programmatically starting next week. Is the pull req ready for building the whole stuff, apart from the REST interface? Thanks, Emanuele Alle 22:05:05 di Sunday 12 April 2015, Niels Charlier ha scritto: On 12-04-15 22:03, Andrea Aime wrote: Did you check the possibility

Re: [Geoserver-devel] Branching GeoFence 2.2.x -- Re: geofence geoserver integration

2015-04-08 Thread Niels Charlier
. For the GeoFence community module in GeoServer 2.7 we'll have to use the GeoFence 2.2.x branch, in order to have the DTOs matching, while GeoServer master (2.8) will use GeoFence master (3.0). Cheers, Emanuele Alle 16:33:31 di Monday 6 April 2015, Niels Charlier ha scritto: I made a pull

Re: [Geoserver-devel] Branching GeoFence 2.2.x -- Re: geofence geoserver integration

2015-04-08 Thread Niels Charlier
: https://github.com/geoserver/geofence/blob/master/src/pom.xml#L15 The refactor branch has already been merged into master. Cheers, Emanuele Alle 12:55:36 di Wednesday 8 April 2015, Niels Charlier ha scritto: Hello Emanuele, That sounds good. When the geofence 3.0 branch is there I

Re: [Geoserver-devel] user/role spring rest api

2015-05-20 Thread Niels Charlier
On 20-05-15 12:22, Andrea Aime wrote: Hum... it would be the fourth object - xml technology we have in the code base (old xml parsers/encoders, xsd parsers encoders, xstream). From what it looks this one is in the same basket as xstream. I don't want to sound against innovation, but we cannot

[Geoserver-devel] user/role spring rest api

2015-05-20 Thread Niels Charlier
Hello, While doing a pull request on the geofence-server rest api, I had a question. This rest api is made in the spring rest api, unlike the rest of geoserver which uses restful. Jody suggested we might wish to (gradually) move towards that system, and I prefer it as well, It is very easy to

[Geoserver-devel] geofence rest API PR

2015-06-04 Thread Niels Charlier
Emanuele, I just wanted to verify that the geofence rest API PR okay to merge now: https://github.com/geoserver/geoserver/pull/1044 Regards Niels -- ___ Geoserver-devel

Re: [Geoserver-devel] geofence rest API PR

2015-06-08 Thread Niels Charlier
Hello Emanuele, thank for your thorough review! On 06/05/2015 11:50 AM, Emanuele Tajariol wrote: - I see that for the geofence REST API we only have XML output, while the other geoserver REST calls also allows json and html. Is that ok? Should it be made expicitly clear in the doc? Good

Re: [Geoserver-devel] geofence rest API PR

2015-06-10 Thread Niels Charlier
with invalid service/username/groupname/rolename, fixed priority bug, added test for it, updated docs. Thank you in advance for your review. Kind Regards Niels On 09-06-15 00:15, Niels Charlier wrote: Hello Emanuele, thank for your thorough review! On 06/05/2015 11:50 AM, Emanuele Tajariol wrote: - I

Re: [Geoserver-devel] Asking for new vector tiles community module

2015-06-26 Thread Niels Charlier
Hi Tore, Would you consider this PR please: https://github.com/ElectronicChartCentre/java-vector-tile/pull/4 It allows to provide coordinates in the 0..extent-1 range immediately, rather than expecting a 0..255 range and scaling. However, this is just an optional setting and the default

  1   2   3   4   5   6   >