Browse Source

configurable query timeout, s3 options for minIO, safety on cloud deploy job

pull/4023/head
Martin McKeaveney 5 years ago
parent
commit
18ad8ac1e5
  1. 6
      .github/workflows/deploy-cloud.yaml
  2. 5
      packages/server/src/api/controllers/query.js
  3. 1
      packages/server/src/environment.js
  4. 2
      packages/server/src/integrations/dynamodb.ts
  5. 2
      packages/server/src/integrations/oracle.ts
  6. 12
      packages/server/src/integrations/s3.ts

6
.github/workflows/deploy-cloud.yaml

@ -12,6 +12,12 @@ jobs:
runs-on: ubuntu-latest
steps:
- name: Fail if branch is not master
if: github.ref != 'refs/heads/master'
run: |
echo "Ref is not master, you must run this job from master."
exit 1
- uses: actions/checkout@v2
- name: Pull values.yaml from budibase-infra

5
packages/server/src/api/controllers/query.js

@ -7,8 +7,11 @@ const {
} = require("../../db/utils")
const { BaseQueryVerbs } = require("../../constants")
const { Thread, ThreadType } = require("../../threads")
const env = require("../../environment")
const Runner = new Thread(ThreadType.QUERY, { timeoutMs: 10000 })
const Runner = new Thread(ThreadType.QUERY, {
timeoutMs: env.QUERY_THREAD_TIMEOUT || 10000,
})
// simple function to append "readable" to all read queries
function enrichQueries(input) {

1
packages/server/src/environment.js

@ -65,6 +65,7 @@ module.exports = {
DEPLOYMENT_CREDENTIALS_URL: process.env.DEPLOYMENT_CREDENTIALS_URL,
ALLOW_DEV_AUTOMATIONS: process.env.ALLOW_DEV_AUTOMATIONS,
DISABLE_THREADING: process.env.DISABLE_THREADING,
QUERY_THREAD_TIMEOUT: process.env.QUERY_THREAD_TIMEOUT,
_set(key, value) {
process.env[key] = value
module.exports[key] = value

2
packages/server/src/integrations/dynamodb.ts

@ -191,7 +191,7 @@ module DynamoModule {
return response
}
async describe(query: { table: string; }) {
async describe(query: { table: string }) {
const params = {
TableName: query.table,
}

2
packages/server/src/integrations/oracle.ts

@ -450,7 +450,7 @@ module OracleModule {
})
return lastRow.rows
} else {
return [{ [ operation.toLowerCase() ]: true }]
return [{ [operation.toLowerCase()]: true }]
}
}
}

12
packages/server/src/integrations/s3.ts

@ -29,6 +29,18 @@ module S3Module {
type: "password",
required: true,
},
endpoint: {
type: "string",
required: false,
},
s3ForcePathStyle: {
type: "boolean",
required: false,
},
signatureVersion: {
type: "string",
required: false,
},
},
query: {
read: {

Loading…
Cancel
Save