We prefer pull requests (against the main branch) as described here (
https://docs.geotools.org/latest/developer/procedures/pull_requests.html)
and an accompanying issue on the tracker (
https://osgeo-org.atlassian.net/jira/software/c/projects/GEOT/issues/)
Thanks
Ian
On Thu, 29 Jul 2021 at 00:59, Kuntz, Karl A <karl.ku...@1898andco.com>
wrote:
> Hey all,
>
>
>
> I’m new here. We’re planning on running GeoServer on top of MongoDB
> Atlas. We’re successfully doing it with the addition of a patch, attached
> and shown below. It’s very basic, just adds support for the URI scheme
> that starts with mongodb+srv:// in addition to the mongodb:// prefix. I’m
> hoping someone with commit access would be willing to add it to main for
> others as well. I’m happy to write tests for it (probably in
> MongoSchemaDBStoreTest.java), but didn’t see any tests specifically
> focused on the prefix(s), so I haven’t up to this point.
>
>
>
> Thanks!
>
> -Karl
>
>
>
> diff --git
> a/modules/plugin/mongodb/src/main/java/org/geotools/data/mongodb/MongoDataStore.java
> b/modules/plugin/mongodb/src/main/java/org/geotools/data/mongodb/MongoDataStore.java
>
> index 5c710b4233..299194e5c5 100644
>
> ---
> a/modules/plugin/mongodb/src/main/java/org/geotools/data/mongodb/MongoDataStore.java
>
> +++
> b/modules/plugin/mongodb/src/main/java/org/geotools/data/mongodb/MongoDataStore.java
>
> @@ -185,9 +185,10 @@ public class MongoDataStore extends ContentDataStore {
>
> if (dataStoreURI == null) {
>
> throw new IllegalArgumentException("dataStoreURI may not be
> null");
>
> }
>
> - if (!dataStoreURI.startsWith("mongodb://")) {
>
> + if (!dataStoreURI.startsWith("mongodb://")
>
> + && !dataStoreURI.startsWith("mongodb+srv://")) {
>
> throw new IllegalArgumentException(
>
> - "incorrect scheme for URI, expected to begin with
> \"mongodb://\", found URI of \""
>
> + "incorrect scheme for URI, expected to begin with
> \"mongodb://\" or \"mongodb+srv://\", found URI of \""
>
> + dataStoreURI
>
> + "\"");
>
> }
>
> @@ -223,7 +224,8 @@ public class MongoDataStore extends ContentDataStore {
>
> + "\"",
>
> e);
>
> }
>
> - } else if (schemaStoreURI.startsWith("mongodb:")) {
>
> + } else if (schemaStoreURI.startsWith("mongodb:")
>
> + || schemaStoreURI.startsWith("mongodb+srv:")) {
>
> try {
>
> return new MongoSchemaDBStore(schemaStoreURI);
>
> } catch (IOException e) {
>
>
>
>
> _______________________________________________
> GeoTools-Devel mailing list
> GeoTools-Devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/geotools-devel
>
--
Ian Turton
_______________________________________________
GeoTools-Devel mailing list
GeoTools-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel
