Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add overrideConfig for export server #5280

Merged
merged 3 commits into from
Mar 9, 2022

Conversation

maxgrossman
Copy link
Contributor

No description provided.

@@ -320,7 +320,8 @@ function zipOutput(hash,output,outFile,outDir,outZip,isFile,format,cb) {
/**
* Builds the hootenanny command(s) from parts of provided request.
*/
function buildCommand(paramschema, queryOverrideTags, querybbox, querypoly, isFile, input, outDir, outFile, doCrop, ignoreSourceIds, ignoreConf) {
function buildCommand(params, queryOverrideTags, querybbox, querypoly, isFile, input, outDir, outFile, doCrop, ignoreSourceIds, ignoreConf) {
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

change paramschema to the params object from the request so we can pass along the data source also

@@ -399,6 +403,8 @@ function buildCommand(paramschema, queryOverrideTags, querybbox, querypoly, isFi
return command;
};

exports.buildCommand = buildCommand
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

add export so we can run a test

@maxgrossman
Copy link
Contributor Author

adding some test steps.

  1. export data using a bbox that has a lat or lon distance greater than 0.01 degrees
  2. add a config called conf/core/NodeExportAPIOverrides.conf and add the json {"reader.http.bbox.max.download.size": "0.01"}
  3. Add the entry "overrideConfig":"NodeExportAPIOverrides" to the "API" datasource in the node-export-server/config.json file
  4. restart your service and try exporting again. see that hoot returns an error that it cannot export greater than 0.01

Copy link
Contributor

@brianhatchl brianhatchl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems legit 🎤 🔨

@brianhatchl brianhatchl changed the title add ovverideConfig for export server Add overrideConfig for export server Mar 8, 2022
@maxgrossman maxgrossman merged commit 8613a93 into master Mar 9, 2022
@maxgrossman maxgrossman deleted the add_overrideConfig_to_export_server branch March 9, 2022 20:13
@maxgrossman maxgrossman restored the add_overrideConfig_to_export_server branch March 9, 2022 20:13
@maxgrossman maxgrossman deleted the add_overrideConfig_to_export_server branch March 9, 2022 20:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants