.github/renovate.json

@@ -1,13 +0,0 @@
-{
-    "$schema": "https://docs.renovatebot.com/renovate-schema.json",
-    "extends": [
-        "config:recommended"
-    ],
-    "configMigration": true,
-    "lockFileMaintenance": {
-        "enabled": true,
-        "automerge": true,
-        "automergeType": "branch",
-        "ignoreTests": true
-    }
-}

.github/workflows/docker-image.yml

@@ -5,8 +5,18 @@ on:
       - 6
 jobs:
   lint-test:
-    name: lint, build & test
+    name: lint & test
-    uses: './.github/workflows/lint-test.yml'
+    runs-on: ubuntu-latest
+    steps:
+    - name: checkout
+      uses: actions/checkout@v2
+    - name: setup Node
+      uses: actions/setup-node@v1
+      with:
+        node-version: 18.x
+    - run: npm install
+    - run: npm run lint
+    - run: npm test
 
   build-and-publish:
     name: build & publish Docker image
@@ -14,32 +24,23 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: check out the repo
-        uses: actions/checkout@v4
+        uses: actions/checkout@v2
-      - name: set up QEMU
-        uses: docker/setup-qemu-action@v3
-      - name: configure Docker to use buildx
-        uses: docker/setup-buildx-action@v3
       - name: log in to Docker Hub
-        uses: docker/login-action@v3
+        uses: docker/login-action@v1
         with:
           username: ${{ secrets.DOCKER_USERNAME }}
           password: ${{ secrets.DOCKER_ACCESS_TOKEN }}
       - name: build Docker image & push to Docker Hub
-        uses: docker/build-push-action@v4
+        uses: docker/build-push-action@v2
         with:
-          context: .
           push: true
-          platforms: linux/amd64,linux/arm64
           tags: |
             derhuerst/db-rest:6
             derhuerst/db-rest:latest
-          # https://docs.docker.com/build/ci/github-actions/examples/#github-cache
-          cache-from: type=gha
-          cache-to: type=gha,mode=max,oci-mediatypes=true,compression=zstd
 
       # this is for the public-transport/infrastructure cluster
       - name: log in to GitHub Container Registry
-        uses: docker/login-action@v3
+        uses: docker/login-action@v1
         with:
           registry: ghcr.io
           username: ${{ github.repository_owner }}
@@ -51,14 +52,7 @@ jobs:
         id: datetime
         run: echo "::set-output name=datetime::$(date -u +'%Y-%m-%dT%H.%M.%SZ')"
       - name: push Docker image to GitHub Registry
-        uses: docker/build-push-action@v4
+        uses: docker/build-push-action@v2
         with:
-          context: .
           push: true
-          platforms: linux/amd64,linux/arm64
+          tags: ghcr.io/${{github.repository}}:v6_${{steps.hash.outputs.hash}}_${{steps.datetime.outputs.datetime}}
-          tags: |
-            ghcr.io/${{github.repository}}:v6
-            ghcr.io/${{github.repository}}:v6_${{steps.hash.outputs.hash}}_${{steps.datetime.outputs.datetime}}
-          # https://docs.docker.com/build/ci/github-actions/examples/#github-cache
-          cache-from: type=gha
-          cache-to: type=gha,mode=max,oci-mediatypes=true,compression=zstd

.github/workflows/lint-test.yml

@@ -1,35 +0,0 @@
-name: lint, build & test
-on:
-  push:
-    branches:
-      - '*'
-  pull_request:
-    branches:
-      - '*'
-  workflow_call:
-
-jobs:
-  lint-test:
-    name: lint, build & test
-    runs-on: ubuntu-latest
-    strategy:
-      matrix:
-        node-version:
-          - '16.x'
-          - '18.x'
-          - '20.x'
-          - '22.x'
-
-    steps:
-    - name: checkout
-      uses: actions/checkout@v4
-    - name: setup Node
-      uses: actions/setup-node@v4
-      with:
-        node-version: ${{ matrix.node-version }}
-
-    - run: npm ci
-
-    - run: npm run lint
-    - run: npm run build
-    - run: npm test

.gitignore

@@ -5,6 +5,7 @@ Thumbs.db
 node_modules
 npm-debug.log
 
+/package-lock.json
 /dump.rdb
 
 /docs/*.html

Dockerfile

@@ -3,8 +3,8 @@ WORKDIR /app
 
 # install dependencies
 RUN apk add --update git bash
-ADD package.json package-lock.json /app
+ADD package.json /app
-RUN npm ci
+RUN npm install
 
 # build documentation
 ADD . /app

api-docs.js

@@ -1,5 +1,5 @@
 import {generateApiDocs} from 'hafas-rest-api/tools/generate-docs.js'
-import {api} from '../api.js'
+import {api} from './api.js'
 
 const HEAD = `\
 # \`v6.db.transport.rest\` API documentation
@@ -61,7 +61,7 @@ Uses [\`hafasClient.journeys()\`](https://github.com/public-transport/hafas-clie
 \`from\` (A), \`to\` (B), and the optional \`via\` must each have one of these formats:
 
 - as stop/station ID (e.g. \`from=8010159\` for *Halle (Saale) Hbf*)
-- as a POI (e.g. \`from.id=991561765&from.latitude=51.48364&from.longitude=11.98084&from.name=Halle+(Saale),+Stadtpark+Halle+(Grünanlagen)\` for *Halle (Saale), Stadtpark Halle (Grünanlagen)*)
+- as a POI (e.g. \`from.id=991561765&from.latitude=51.48364&from.longitude=11.98084\` for *Halle+(Saale),+Stadtpark+Halle+(Grünanlagen)*)
 - as an address (e.g. \`from.latitude=51.25639&from.longitude=7.46685&from.address=Hansestadt+Breckerfeld,+Hansering+3\` for *Hansestadt Breckerfeld, Hansering 3*)
 
 ### Pagination
@@ -416,42 +416,40 @@ curl "https://v6.db.transport.rest/radar?$bbox&results=10" -s | jq
 `,
 }
 
-const generateMarkdownApiDocs = async function* () {
+const {
-	const {
 	listOfRoutes,
 	routes,
 	tail,
-	} = generateApiDocs(api.routes)
+} = generateApiDocs(api.routes)
 
-	const sortedRoutes = Object.entries(routes)
+const sortedRoutes = Object.entries(routes)
-	.sort(([routeA], [routeB]) => {
+.sort(([routeA], [routeB]) => {
 	const iA = order.indexOf(routeA)
 	const iB = order.indexOf(routeB)
 	if (iA >= 0 && iB >= 0) return iA - iB
 	if (iA < 0 && iB >= 0) return 1
 	if (iB < 0 && iA >= 0) return -1
 	return 0
-	})
+})
 
-	yield HEAD
+const write = process.stdout.write.bind(process.stdout)
-	yield `\n\n`
 
-	yield listOfRoutes
+write(HEAD)
-	yield `\n\n`
+write(`\n\n`)
 
-	for (const [route, params] of sortedRoutes) {
+write(listOfRoutes)
-		yield `## \`GET ${route}\`\n\n`
+write(`\n\n`)
-		yield descriptions[route] || ''
+
-		yield `\n### Query Parameters\n`
+for (const [route, params] of sortedRoutes) {
-		yield params
+	write(`## \`GET ${route}\`\n\n`)
+	write(descriptions[route] || '')
+	write(`
+### Query Parameters
+`)
+	write(params)
 	if (examples[route]) {
-			yield '\n' + examples[route]
+		write('\n' + examples[route])
 	}
-		yield `\n\n`
+	write(`\n\n`)
-	}
-	yield tail
-}
-
-export {
-	generateMarkdownApiDocs,
 }
+write(tail)

api.js

@@ -12,7 +12,7 @@ import Redis from 'ioredis'
 import {createCachedHafasClient} from 'cached-hafas-client'
 import {createRedisStore} from 'cached-hafas-client/stores/redis.js'
 import serveStatic from 'serve-static'
-import {parseBoolean, parseInteger} from 'hafas-rest-api/lib/parse.js'
+import {parseBoolean} from 'hafas-rest-api/lib/parse.js'
 import {loyaltyCardParser} from './lib/loyalty-cards.js'
 import {route as stations} from './routes/stations.js'
 import {route as station} from './routes/station.js'
@@ -24,7 +24,6 @@ const docsRoot = pathJoin(__dirname, 'docs')
 
 const berlinHbf = '8011160'
 
-// todo: use process.env.HAFAS_USER_AGENT if defined
 let hafas = createHafas(pkg.name)
 let healthCheck = createHealthCheck(hafas, berlinHbf)
 
@@ -61,12 +60,6 @@ const mapRouteParsers = (route, parsers) => {
 			default: 'false',
 			parse: parseBoolean,
 		},
-		age: {
-			description: 'Age of traveller',
-			type: 'integer',
-			defaultStr: '*adult*',
-			parse: parseInteger
-		},
 	}
 }
 

build/index.js

@@ -1,86 +0,0 @@
-#!/usr/bin/env node
-
-// todo: use import assertions once they're supported by Node.js & ESLint
-// https://github.com/tc39/proposal-import-assertions
-import {createRequire} from 'module'
-const require = createRequire(import.meta.url)
-
-import {dirname, join} from 'node:path'
-import {pipeline} from 'node:stream/promises'
-import {createReadStream, createWriteStream} from 'node:fs'
-import {copyFile} from 'node:fs/promises'
-import _technicalDocsCli from '@derhuerst/technical-docs-cli'
-import {config} from '../api.js'
-const {
-	createMarkdownRenderer,
-	determineSyntaxStylesheetPath,
-} = _technicalDocsCli
-import {generateMarkdownApiDocs} from './api-docs.js'
-
-const pkg = require('../package.json')
-
-const BASE_URL = new URL('..', import.meta.url).href
-const API_DOCS_DEST = 'docs/api.md'
-const DOCS_TO_RENDER = [
-	['docs/readme.md', 'docs/index.html'],
-	['docs/getting-started.md', 'docs/getting-started.html'],
-	['docs/api.md', 'docs/api.html'],
-]
-const SYNTAX_STYLESHEET_URL = '/syntax.css'
-const SYNTAX_STYLESHEET_SRC = determineSyntaxStylesheetPath('github')
-const SYNTAX_STYLESHEET_DEST = 'docs/syntax.css'
-
-{
-	console.info('writing Markdown API docs to ' + API_DOCS_DEST)
-
-	const destPath = new URL(API_DOCS_DEST, BASE_URL).pathname
-	await pipeline(
-		generateMarkdownApiDocs(),
-		createWriteStream(destPath),
-	)
-}
-
-const markdownRenderingCfg = {
-	syntaxStylesheetUrl: SYNTAX_STYLESHEET_URL,
-	additionalHeadChildren: (h) => {
-		if (!pkg.goatCounterUrl) return []
-		return [
-			// https://72afc0822cce0642af90.goatcounter.com/help/skip-dev#skip-loading-staging-beta-sites-312
-			h('script', `
-if (window.location.host !== ${JSON.stringify(config.hostname)}) {
-    window.goatcounter = {no_onload: true}
-}
-`),
-			// https://72afc0822cce0642af90.goatcounter.com/help/start
-			h('script', {
-				src: '//gc.zgo.at/count.js',
-				async: true,
-				'data-goatcounter': pkg.goatCounterUrl,
-			}),
-		]
-	},
-}
-for (const [src, dest] of DOCS_TO_RENDER) {
-	console.info(`rendering Markdown file ${src} to HTML file ${dest}`)
-
-	const srcPath = new URL(src, BASE_URL).pathname
-	const destPath = new URL(dest, BASE_URL).pathname
-	// unfortunately, we can't use stream.pipeline right now
-	// see https://github.com/unifiedjs/unified-stream/issues/1
-	await new Promise((resolve, reject) => {
-		createReadStream(srcPath)
-		.once('error', reject)
-		.pipe(createMarkdownRenderer(markdownRenderingCfg))
-		.once('error', reject)
-		.pipe(createWriteStream(destPath))
-		.once('error', reject)
-		.once('finish', resolve)
-	})
-}
-
-{
-	const srcPath = SYNTAX_STYLESHEET_SRC
-	const destPath = new URL(SYNTAX_STYLESHEET_DEST, BASE_URL).pathname
-	console.info(`copying syntax stylesheet from ${srcPath} to ${destPath}`)
-	await copyFile(srcPath, destPath)
-}

docs/readme.md

@@ -2,27 +2,23 @@
 
 [`v6.db.transport.rest`](https://v6.db.transport.rest/) is a [REST API](https://restfulapi.net) for the public transportation system in Germany.
 
-[![API status](https://badgen.net/uptime-robot/status/m793274556-25c5e9bbab0297d91cda7134)](https://stats.uptimerobot.com/57wNLs39M/793274556)
+[![API status](https://badgen.net/uptime-robot/status/m784879516-8a977fa91b975fc3884a9857)](https://stats.uptimerobot.com/57wNLs39M/784879516)
 
-Because it wraps [an API](https://github.com/public-transport/hafas-client/blob/6/readme.md#background) of [Deutsche Bahn](https://de.wikipedia.org/wiki/Deutsche_Bahn), it **includes most of the long-distance and regional traffic, as well as some international trains and local buses**. Essentially, it returns whatever data the [*DB Navigator* app](https://www.bahn.de/p/view/service/mobile/db-navigator.shtml) shows*, **including realtime delays and disruptions**.
+Because it wraps [an API](https://github.com/public-transport/hafas-client/blob/6/readme.md#background) of [Deutsche Bahn](https://de.wikipedia.org/wiki/Deutsche_Bahn), it **includes most of the long-distance and regional traffic, as well as some international trains and local buses**. Essentially, it returns whatever data the [*DB Navigator* app](https://www.bahn.de/p/view/service/mobile/db-navigator.shtml) shows, **including realtime delays and disruptions**.
-
-*When comparing results from this API to what the DB Navigator app shows there might be occasional differences due to them utilizing different, not 100% identical backends.*
 
 - [Getting Started](getting-started.md)
-- [API documentation](api.md) (run `npm run build` to generate)
+- [API documentation](api.md)
-- [OpenAPI playground with API documentation](https://petstore.swagger.io/?url=https%3A%2F%2Fv6.db.transport.rest%2F.well-known%2Fservice-desc%0A)
+- [OpenAPI playground](https://petstore.swagger.io/?url=https%3A%2F%2Fv6.db.transport.rest%2F.well-known%2Fservice-desc%0A)
 
 ## Why use this API?
 
 ### Realtime Data
 
-This API returns realtime data whenever it is upstream. The [API for DB's mobile app](https://github.com/public-transport/hafas-client/blob/6/p/db/readme.md) provides it.
+This API returns realtime data whenever its upstream, the [API for DB's mobile app](https://github.com/public-transport/hafas-client/blob/6/p/db/readme.md), provides it.
-
-*Note: Different endpoints might remove realtime data like delays and cancellations at different times, i.e. after a journey's arrival.*
 
 ### No API Key
 
-You can just use the API without authentication. There's a [rate limit](https://apisyouwonthate.com/blog/what-is-api-rate-limiting-all-about) of 100 requests/minute set up.
+You can just use the API without authentication. There's a [rate limit](https://apisyouwonthate.com/blog/what-is-api-rate-limiting-all-about) of 100 requests/minute (burst 150 requests/minute) set up.
 
 ### CORS
 

license.md

@@ -1,4 +1,4 @@
-Copyright (c) 2024, Jannis R
+Copyright (c) 2022, Jannis R
 
 Permission to use, copy, modify, and/or distribute this software for any purpose with or without fee is hereby granted, provided that the above copyright notice and this permission notice appear in all copies.
 

package.json

@@ -2,19 +2,9 @@
 	"private": true,
 	"name": "db-rest",
 	"description": "A clean REST API wrapping around the Deutsche Bahn API.",
-	"version": "6.0.5",
+	"version": "6.0.0",
 	"type": "module",
-	"bin": {
-		"db-rest": "./index.js"
-	},
 	"main": "index.js",
-	"files": [
-		"docs",
-		"lib",
-		"routes",
-		"api.js",
-		"index.js"
-	],
 	"author": "Jannis R <mail@jannisr.de>",
 	"homepage": "https://github.com/derhuerst/db-rest/tree/6",
 	"repository": "derhuerst/db-rest",
@@ -46,20 +36,20 @@
 		"serve-static": "^1.14.1"
 	},
 	"devDependencies": {
-		"@derhuerst/technical-docs-cli": "^1.5.0",
+		"@derhuerst/technical-docs-cli": "^1.1.0",
-		"axios": "^1.6.1",
+		"axios": "~1.1",
 		"eslint": "^8.12.0",
 		"get-port": "^6.1.2",
 		"ndjson": "^2.0.0",
 		"pino-pretty": "^9.1.1",
-		"tap-min": "^3.0.0",
+		"tap-min": "^2.0.0",
 		"tape": "^5.5.2"
 	},
 	"scripts": {
-		"build": "REDIS_URL='' ./build/index.js",
+		"docs": "node api-docs.js >docs/api.md && build-technical-doc --syntax-stylesheet-url /syntax.css <docs/readme.md >docs/index.html && build-technical-doc --syntax-stylesheet-url /syntax.css <docs/getting-started.md >docs/getting-started.html && build-technical-doc --syntax-stylesheet-url /syntax.css <docs/api.md >docs/api.html && build-technical-doc --syntax-stylesheet github >docs/syntax.css",
+		"build": "npm run docs",
 		"start": "node index.js",
 		"lint": "eslint .",
 		"test": "node test/index.js | tap-min"
-	},
+	}
-	"goatCounterUrl": "https://37462fdee48390778258.goatcounter.com/count"
 }

readme.md

@@ -15,16 +15,14 @@
 
 ## installing & running
 
-### access to Redis
+`db-rest` expects a [Redis](https://redis.io/) server running on `127.0.0.1:6379` (default port), but you can set the `REDIS_URL` environment variable to change this.
-
-It is recommended that you let `bvg-rest` cache HAFAS responses within a [Redis](https://redis.io/) cache. To use this feature, set `$REDIS_URL` (e.g. to `redis://localhost:6379/1` when running Redis locally).
 
 ### via Docker
 
-A Docker image [is available as `docker.io/derhuerst/db-rest:6`](https://hub.docker.com/r/docker.io/derhuerst/db-rest:6).
+A Docker image [is available as `derhuerst/db-rest:6`](https://hub.docker.com/r/derhuerst/db-rest:6).
 
 ```shell
-docker run -d -p 3000:3000 docker.io/derhuerst/db-rest:6
+docker run -d -p 3000:3000 derhuerst/db-rest:6
 ```
 
 *Note:* The Docker image does not contain the Redis server.
@@ -36,8 +34,6 @@ git clone https://github.com/derhuerst/db-rest.git
 cd db-rest
 git checkout 6
 npm install
-
-export HOSTNAME='my-vbb-rest-api.example.org'
 npm run build
 
 redis-server &
@@ -49,7 +45,6 @@ To keep the API running permanently, use tools like [`forever`](https://github.c
 
 ## Related Projects
 
-- [`DB-Adapter-v6`](https://github.com/olech2412/DB-Adapter-v6) – A Java API client for `db-rest`.
 - [`vbb-rest`](https://github.com/derhuerst/vbb-rest) – A clean REST API wrapping around the VBB API.
 - [`bvg-rest`](https://github.com/derhuerst/bvg-rest) – A clean REST API wrapping around the BVG API.
 - [`hvv-rest`](https://github.com/derhuerst/hvv-rest) – A clean REST API wrapping around the HVV API.

test/index.js

@@ -105,6 +105,6 @@ tape.test('/stations?query=frankfurt%20ha works', async (t) => {
 		for await (const station of parser) stations.push(station)
 
 		t.ok(stations.find(s => s.id === FRANKFURT_MAIN_HBF))
-		t.ok(stations.length > 0)
+		t.ok(Object.keys(stations).length > 0)
 	}
 })