Skip to content

Revert v3.4.1 — the broker credentials do not belong in the repo - #300

Merged
engmung merged 1 commit into
mainfrom
dev
Aug 12, 2026
Merged

Revert v3.4.1 — the broker credentials do not belong in the repo#300
engmung merged 1 commit into
mainfrom
dev

Conversation

@engmung

@engmung engmung commented Aug 12, 2026

Copy link
Copy Markdown
Owner

Reverts the v3.4.1 release commit. @SimonePDA proposed the better shape and it is obviously right:

let's add in the MQTT page the input fields for the Broker, user and the password, so that is not available in the Github files

Runtime configuration, the same way Wi-Fi already works — no credential in the source, none in the image, and each owner points at whatever broker they use. It also removes the question this commit was built to answer.

Exposure, stated plainly. The credentials were on main for about ten minutes (pushed 07:39 UTC, reverted 07:49) and the site auto-deployed, so patternflow.work/flash/bin/v3.4.1/patternflow.ino.bin served them for roughly six of those. This revert stops new downloads and restores the manifest to v3.4.0. It does not undo what it cannot: anyone who cloned or downloaded inside that window has them, and the values remain in git history — a revert removes them from the tree, not from the objects.

So the credential should be treated as exposed and rotated, regardless of how small the window was. That is Simone's to do and he is already reworking the topic permissions.

Not force-rewriting history for this: it breaks every existing clone, and rotation makes the old value worthless anyway, which is the stronger fix. Say the word if you would rather scrub it.

🤖 Generated with Claude Code

@vercel

vercel Bot commented Aug 12, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
pattern-flow-open Ready Ready Preview Aug 12, 2026 7:50am
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
pattern-flow_origin Ignored Ignored Aug 12, 2026 7:50am

@engmung
engmung merged commit 875d889 into main Aug 12, 2026
3 checks 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