Merge branch 'master' of github.com:brianc/node-postgres

Author: cecemel

.eslintrc

@@ -2,7 +2,7 @@
   "plugins": ["prettier"],
   "parser": "@typescript-eslint/parser",
   "extends": ["plugin:prettier/recommended", "prettier/@typescript-eslint"],
-  "ignorePatterns": ["node_modules", "coverage", "packages/pg-protocol/dist/**/*"],
+  "ignorePatterns": ["node_modules", "coverage", "packages/pg-protocol/dist/**/*", "packages/pg-query-stream/dist/**/*"],
   "parserOptions": {
     "ecmaVersion": 2017,
     "sourceType": "module"

.travis.yml

@@ -43,6 +43,38 @@ matrix:
         postgresql: '9.5'
       dist: precise
 
+    # Run tests/paths with client certificate authentication
+    - node_js: lts/*
+      env:
+        - CC=clang CXX=clang++ npm_config_clang=1 PGUSER=postgres PGDATABASE=postgres
+          PGSSLMODE=verify-full
+          PGSSLROOTCERT=$TRAVIS_BUILD_DIR/packages/pg/test/tls/test-server-ca.crt
+          PGSSLCERT=$TRAVIS_BUILD_DIR/packages/pg/test/tls/test-client.crt
+          PGSSLKEY=$TRAVIS_BUILD_DIR/packages/pg/test/tls/test-client.key
+          PG_CLIENT_CERT_TEST=1
+      before_script:
+        - chmod go= packages/pg/test/tls/test-client.key
+        - |
+          sudo sed -i \
+            -e '/^ssl_cert_file =/d' \
+            -e '/^ssl_key_file =/d' \
+            /etc/postgresql/10/main/postgresql.conf
+
+          cat <<'travis ci breaks heredoc' | sudo tee -a /etc/postgresql/10/main/postgresql.conf > /dev/null
+            ssl_cert_file = 'test-server.crt'
+            ssl_key_file = 'test-server.key'
+            ssl_ca_file = 'test-client-ca.crt'
+
+        - printf 'hostssl all all %s cert\n' 127.0.0.1/32 ::1/128 | sudo tee /etc/postgresql/10/main/pg_hba.conf > /dev/null
+        - sudo make -C packages/pg/test/tls install DESTDIR=/var/ramfs/postgresql/10/main
+        - sudo systemctl restart postgresql@10-main
+        - yarn build
+      script:
+        - cd packages/pg
+        - node test/integration/connection-pool/tls-tests.js
+        - npm install --no-save pg-native
+        - node test/integration/connection-pool/tls-tests.js native
+
     # different PostgreSQL versions on Node LTS
     - node_js: lts/erbium
       addons:
@@ -57,6 +89,10 @@ matrix:
       addons:
         postgresql: '9.6'
 
+    # only run lint on latest Node LTS
+    - node_js: lts/*
+      script: yarn lint
+
     # PostgreSQL 9.2 only works on precise
     - node_js: lts/carbon
       addons:

.yarnrc

@@ -0,0 +1 @@
+--install.ignore-engines true

CHANGELOG.md

@@ -4,6 +4,12 @@ For richer information consult the commit log on github with referenced pull req
 
 We do not include break-fix version release in this file.
 
+### pg@8.5.0
+
+- Fix bug forwarding [ssl key](https://github.com/brianc/node-postgres/pull/2394).
+- Convert pg-query-stream internals to [typescript](https://github.com/brianc/node-postgres/pull/2376).
+- Performance [improvements](https://github.com/brianc/node-postgres/pull/2286).
+
 ### pg@8.4.0
 
 - Switch to optional peer dependencies & remove [semver](https://github.com/brianc/node-postgres/commit/a02dfac5ad2e2abf0dc3a9817f953938acdc19b1) package which has been a small thorn in the side of a few users.

SPONSORS.md

@@ -8,6 +8,7 @@ node-postgres is made possible by the helpful contributors from the community as
 - [Nafundi](https://nafundi.com)
 - [CrateDB](https://crate.io/)
 - [BitMEX](https://www.bitmex.com/app/trade/XBTUSD)
+- [Dataform](https://dataform.co/)
 
 # Supporters
 
@@ -32,3 +33,4 @@ node-postgres is made possible by the helpful contributors from the community as
 - Simple Analytics
 - Trevor Linton
 - Ian Walter
+- @Guido4000

package.json

@@ -10,22 +10,23 @@
     "packages/*"
   ],
   "scripts": {
-    "test": "yarn lint && yarn lerna exec yarn test",
-    "build": "yarn lerna exec --scope pg-protocol yarn build",
+    "test": "yarn lerna exec yarn test",
+    "build": "tsc --build",
+    "build:watch": "tsc --build --watch",
     "pretest": "yarn build",
-    "lint": "if [ -x ./node_modules/.bin/prettier ]; then eslint '*/**/*.{js,ts,tsx}'; fi;"
+    "prepublish": "yarn build",
+    "lint": "eslint '*/**/*.{js,ts,tsx}'"
   },
   "devDependencies": {
-    "@typescript-eslint/eslint-plugin": "^2.27.0",
-    "@typescript-eslint/parser": "^2.27.0",
-    "eslint": "^6.8.0",
-    "eslint-config-prettier": "^6.10.1",
+    "@typescript-eslint/eslint-plugin": "^4.4.0",
+    "@typescript-eslint/parser": "^4.4.0",
+    "eslint": "^7.11.0",
+    "eslint-config-prettier": "^6.12.0",
     "eslint-plugin-node": "^11.1.0",
-    "eslint-plugin-prettier": "^3.1.2",
-    "lerna": "^3.19.0"
-  },
-  "optionalDependencies": {
-    "prettier": "2.0.4"
+    "eslint-plugin-prettier": "^3.1.4",
+    "lerna": "^3.19.0",
+    "prettier": "2.1.2",
+    "typescript": "^4.0.3"
   },
   "prettier": {
     "semi": false,

packages/pg-cursor/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pg-cursor",
-  "version": "2.4.1",
+  "version": "2.5.1",
   "description": "Query cursor extension for node-postgres",
   "main": "index.js",
   "directories": {
@@ -17,6 +17,6 @@
   "license": "MIT",
   "devDependencies": {
     "mocha": "^7.1.2",
-    "pg": "^8.4.1"
+    "pg": "^8.5.1"
   }
 }

packages/pg-pool/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pg-pool",
-  "version": "3.2.1",
+  "version": "3.2.2",
   "description": "Connection pool for node-postgres",
   "main": "index.js",
   "directories": {

packages/pg-protocol/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pg-protocol",
-  "version": "1.3.0",
+  "version": "1.4.0",
   "description": "The postgres client/server binary protocol, implemented in TypeScript",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
@@ -13,13 +13,17 @@
     "chunky": "^0.0.0",
     "mocha": "^7.1.2",
     "ts-node": "^8.5.4",
-    "typescript": "^3.7.3"
+    "typescript": "^4.0.3"
   },
   "scripts": {
     "test": "mocha dist/**/*.test.js",
     "build": "tsc",
     "build:watch": "tsc --watch",
     "prepublish": "yarn build",
     "pretest": "yarn build"
-  }
+  },
+  "files": [
+    "/dist/*{js,ts,map}",
+    "/src"
+  ]
 }

packages/pg-protocol/src/buffer-writer.ts

@@ -5,7 +5,7 @@ export class Writer {
   private offset: number = 5
   private headerPosition: number = 0
   constructor(private size = 256) {
-    this.buffer = Buffer.alloc(size)
+    this.buffer = Buffer.allocUnsafe(size)
   }
 
   private ensure(size: number): void {
@@ -15,7 +15,7 @@ export class Writer {
       // exponential growth factor of around ~ 1.5
       // https://stackoverflow.com/questions/2269063/buffer-growth-strategy
       var newSize = oldBuffer.length + (oldBuffer.length >> 1) + size
-      this.buffer = Buffer.alloc(newSize)
+      this.buffer = Buffer.allocUnsafe(newSize)
       oldBuffer.copy(this.buffer)
     }
   }