Skip to content

Conversation

@lipskis
Copy link
Contributor

@lipskis lipskis commented Sep 24, 2025

This app initiates an Elasticsearch client and uses it in express route handlers for the most common Elasticsearch operations. Right now the setup covers info, indices.create, indices.delete, and get, update, search, delete operations for documents.

See appsignal/appsignal-nodejs#1092

@lipskis lipskis self-assigned this Sep 24, 2025
@backlog-helper
Copy link

backlog-helper bot commented Sep 24, 2025

✔️ All good!

New issue guide | Backlog management | Rules | Feedback

@lipskis lipskis added the chore A small task that takes a day or two at the most. label Sep 24, 2025
@lipskis lipskis force-pushed the elasticsearch-express-app branch from b5a7ddb to a7fccc2 Compare September 30, 2025 17:10
@lipskis lipskis requested review from tombruijn and unflxw September 30, 2025 17:14
@lipskis lipskis marked this pull request as ready for review September 30, 2025 17:14
Copy link
Contributor

@unflxw unflxw left a comment

Choose a reason for hiding this comment

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

It's really nice to see such a thorough test setup, testing different aspects of the Elastic search integration. Good stuff!

@lipskis
Copy link
Contributor Author

lipskis commented Oct 2, 2025

Will wait for #337 and then rebase

lipskis and others added 3 commits October 2, 2025 19:04
This app initiates a Elasticsearch `client` and then calls it in
`Express` routes.

See appsignal/appsignal-nodejs#1092
This change adds endpoints for frequently used operations with
Elasticsearch. It adds endpoints for creating and deleting indices, as
well as separate endpoints for `index`, `get`, `update`, `search` and
`delete` document operations.
Use setup function to initialize index on app startup and handle missing
index.

Remove `winston`, all its dependencies, logger setup and any calls to
it.

Add `WebWorker` to `compilerOptions.lib` in `tsconfig.json` to fix
missing `StreamPipeOptions` in `apache/arrow-js`.
@lipskis lipskis force-pushed the elasticsearch-express-app branch from db33ad0 to 0b4c94d Compare October 2, 2025 16:05
@backlog-helper
Copy link

backlog-helper bot commented Oct 3, 2025


This is a message from the daily scheduled checks.

New issue guide | Backlog management | Rules | Feedback

App name was defined in docker-compose env and in the appsignal.cjs. Use
just one.
Install `less` pager and exit if a child process fails in the `run`
command.
All other anchor tags or form submission open in the same tab/window.
Open `Cluster info` document in the same tab.
@lipskis lipskis merged commit 415643e into main Oct 3, 2025
72 checks passed
@lipskis lipskis deleted the elasticsearch-express-app branch October 3, 2025 09:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore A small task that takes a day or two at the most.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants