25.11 HTTP stream errors handling (#478)

Author: slvrtrn

.docker/clickhouse/single_node_tls/Dockerfile

@@ -1,4 +1,4 @@
-FROM clickhouse/clickhouse-server:25.8-alpine
+FROM clickhouse/clickhouse-server:25.10-alpine
 COPY .docker/clickhouse/single_node_tls/certificates /etc/clickhouse-server/certs
 RUN chown clickhouse:clickhouse -R /etc/clickhouse-server/certs \
     && chmod 600 /etc/clickhouse-server/certs/* \

.github/ISSUE_TEMPLATE/bug_report.md

@@ -4,13 +4,14 @@ about: Create a report to help us improve
 title: ''
 labels: bug
 assignees: ''
-
 ---
 
 <!-- delete unnecessary items -->
+
 ### Describe the bug
 
 ### Steps to reproduce
+
 1.
 2.
 3.
@@ -22,13 +23,16 @@ assignees: ''
 ### Error log
 
 ### Configuration
+
 #### Environment
-* Client version:
-* Language version:
-* OS:
+
+- Client version:
+- Language version:
+- OS:
 
 #### ClickHouse server
-* ClickHouse Server version:
-* ClickHouse Server non-default settings, if any:
-* `CREATE TABLE` statements for tables involved:
-* Sample data for all these tables, use [clickhouse-obfuscator](https://github.com/ClickHouse/ClickHouse/blob/master/programs/obfuscator/Obfuscator.cpp#L42-L80) if necessary
+
+- ClickHouse Server version:
+- ClickHouse Server non-default settings, if any:
+- `CREATE TABLE` statements for tables involved:
+- Sample data for all these tables, use [clickhouse-obfuscator](https://github.com/ClickHouse/ClickHouse/blob/master/programs/obfuscator/Obfuscator.cpp#L42-L80) if necessary

.github/ISSUE_TEMPLATE/feature_request.md

@@ -4,10 +4,10 @@ about: Suggest an idea for the client
 title: ''
 labels: enhancement
 assignees: ''
-
 ---
 
 <!-- delete unnecessary items -->
+
 ### Use case
 
 ### Describe the solution you'd like

.github/pull_request_template.md

@@ -1,8 +1,11 @@
 ## Summary
+
 A short description of the changes with a link to an open issue.
 
 ## Checklist
+
 Delete items not relevant to your PR:
+
 - [ ] Unit and integration tests covering the common scenarios were added
 - [ ] A human-readable description of the changes was provided to include in CHANGELOG
 - [ ] For significant changes, documentation in https://github.com/ClickHouse/clickhouse-docs was updated with further explanations or tutorials

CHANGELOG.md

@@ -1,3 +1,11 @@
+# 1.13.0
+
+## New features
+
+- Server-side exceptions that occur in the middle of the HTTP stream are now handled correctly. This requires [ClickHouse 25.11+](https://github.com/ClickHouse/ClickHouse/pull/88818). Previous ClickHouse versions are unaffected by this change. ([#478])
+
+[#478]: https://github.com/ClickHouse/clickhouse-js/pull/478
+
 # 1.12.1
 
 ## Improvements

docker-compose.cluster.yml

@@ -2,7 +2,7 @@ version: '2.3'
 
 services:
   clickhouse1:
-    image: 'clickhouse/clickhouse-server:${CLICKHOUSE_VERSION-25.8-alpine}'
+    image: 'clickhouse/clickhouse-server:${CLICKHOUSE_VERSION-25.10-alpine}'
     ulimits:
       nofile:
         soft: 262144
@@ -21,7 +21,7 @@ services:
       - './.docker/clickhouse/users.xml:/etc/clickhouse-server/users.xml'
 
   clickhouse2:
-    image: 'clickhouse/clickhouse-server:${CLICKHOUSE_VERSION-25.8-alpine}'
+    image: 'clickhouse/clickhouse-server:${CLICKHOUSE_VERSION-25.10-alpine}'
     ulimits:
       nofile:
         soft: 262144

docker-compose.yml

@@ -1,7 +1,7 @@
 #version: '3.8'
 services:
   clickhouse:
-    image: 'clickhouse/clickhouse-server:${CLICKHOUSE_VERSION-25.8-alpine}'
+    image: 'clickhouse/clickhouse-server:${CLICKHOUSE_VERSION-25.10-alpine}'
     container_name: 'clickhouse-js-clickhouse-server'
     environment:
       CLICKHOUSE_SKIP_USER_SETUP: 1

eslint.config.mjs

@@ -79,6 +79,6 @@ export default defineConfig(
   },
   // Ignore build artifacts and externals
   {
-    ignores: ['out', 'dist', 'node_modules', 'webpack'],
+    ignores: ['coverage', 'out', 'dist', 'node_modules', 'webpack'],
   },
 )

karma.config.mjs

@@ -36,6 +36,10 @@ export default function (config) {
       ],
     },
     reporters: ['mocha'],
+    mochaReporter: {
+      output: 'minimal',
+      ignoreSkipped: true,
+    },
     port: 9876,
     colors: true,
     logLevel: config.LOG_INFO,

package.json

@@ -27,9 +27,9 @@
     "lint": "eslint .",
     "lint:fix": "eslint --fix .",
     "test": ".scripts/jasmine.sh jasmine.all.json",
-    "test:common:unit": ".scripts/jasmine.sh jasmine.common.unit.json",
+    "test:common:unit": "CLICKHOUSE_TEST_SKIP_INIT=1 .scripts/jasmine.sh jasmine.common.unit.json",
     "test:common:integration": ".scripts/jasmine.sh jasmine.common.integration.json",
-    "test:node:unit": ".scripts/jasmine.sh jasmine.node.unit.json",
+    "test:node:unit": "CLICKHOUSE_TEST_SKIP_INIT=1 .scripts/jasmine.sh jasmine.node.unit.json",
     "test:node:tls": ".scripts/jasmine.sh jasmine.node.tls.json",
     "test:node:integration": ".scripts/jasmine.sh jasmine.node.integration.json",
     "test:node:integration:local_cluster": "CLICKHOUSE_TEST_ENVIRONMENT=local_cluster npm run test:node:integration",
@@ -42,7 +42,7 @@
     "prepare": "husky"
   },
   "devDependencies": {
-    "@eslint/js": "^9.34.0",
+    "@eslint/js": "^9.39.1",
     "@faker-js/faker": "^10.0.0",
     "@istanbuljs/nyc-config-typescript": "^1.0.2",
     "@types/jasmine": "^5.1.8",
@@ -52,7 +52,7 @@
     "@types/split2": "^4.2.3",
     "@types/uuid": "^11.0.0",
     "apache-arrow": "^21.0.0",
-    "eslint": "^9.34.0",
+    "eslint": "^9.39.1",
     "eslint-config-prettier": "^10.1.8",
     "eslint-plugin-expect-type": "^0.6.2",
     "eslint-plugin-prettier": "^5.5.4",
@@ -81,8 +81,8 @@
     "ts-node": "^10.9.2",
     "tsconfig-paths": "^4.2.0",
     "tsconfig-paths-webpack-plugin": "^4.2.0",
-    "typescript": "^5.9.2",
-    "typescript-eslint": "^8.43.0",
+    "typescript": "^5.9.3",
+    "typescript-eslint": "^8.46.4",
     "uuid": "^13.0.0",
     "webpack": "^5.101.1",
     "webpack-cli": "^6.0.1",