Skip to content

Small fixes from the review - #25

Merged
snaketech-tu merged 1 commit into
fix/store-and-collectorfrom
fix/small-things
Nov 10, 2026
Merged

snaketech-tu merged 1 commit into
fix/store-and-collectorfrom
fix/small-things

Conversation

@snaketech-tu

Copy link
Copy Markdown
Owner

Merge after "Store and collector fixes from the review".

Changes

  • Broken config.json: a config file that cannot be read is moved to config.broken.json and Stackline starts with defaults. A banner says so, instead of failing to start.
  • --host 0.0.0.0: this now works. Requests addressed to any IP address with the right port are answered, while names are still refused, which keeps DNS rebinding out. Stackline warns that the link travels unencrypted.
  • CSV export: cells that start like a spreadsheet formula (= + - @) get a leading '.

Tests

  • New tests for the broken config, IP hosts on the network, and CSV cells.
  • cargo test, clippy, fmt and npm run check pass.

- --host beyond this computer answers any IP address of the machine (names
  stay refused) and warns that the link travels unencrypted
- A settings file that cannot be read is set aside as config.broken.json;
  Stackline starts with the defaults and says so in the UI
- CSV cells that start like a formula start with an apostrophe
@snaketech-tu
snaketech-tu merged commit ad664fc into fix/store-and-collector Nov 10, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant