guettli/sharedinbox
1
0
Fork 0
Code Issues 21 Pull Requests 6 Actions Packages Projects Releases Wiki Activity

fix: remove hashed_ip from bugreport service, store email in mail.eml #442

Merged
guettlibot merged 1 commits from refs/pull/442/head into main 2026-06-05 14:23:06 +00:00
Conversation 0 Commits 1 Files Changed 1
+940 -6381
guettlibot commented 2026-06-05 12:44:38 +00:00 (Migrated from codeberg.org)
Copy Link
Copy Source

Summary

  • Remove hashed_ip entirely: dropped HashedIP field, hashIP function, and all IP extraction logic from the server. No IP address is collected or stored in any form.
  • Move contact email out of report.json: if the user opts to include their email for follow-up, it is now written to mail.eml in the report directory instead of being embedded in report.json. This keeps PII separate from the structured report data.
  • Remove now-unused imports (crypto/sha256, encoding/hex, strings).
  • Flutter client (bug_report_screen.dart) was already not sending a hashed_ip field — no client changes needed.

Test plan

  • go build ./... in server/bugreport/ passes with no errors
  • go vet ./... passes with no warnings
  • Reports without a contact email produce only report.json (no mail.eml)
  • Reports with a contact email produce report.json (no email key) and mail.eml containing the address

Closes #441

🤖 Generated with Claude Code

## Summary - **Remove hashed_ip entirely**: dropped `HashedIP` field, `hashIP` function, and all IP extraction logic from the server. No IP address is collected or stored in any form. - **Move contact email out of report.json**: if the user opts to include their email for follow-up, it is now written to `mail.eml` in the report directory instead of being embedded in `report.json`. This keeps PII separate from the structured report data. - Remove now-unused imports (`crypto/sha256`, `encoding/hex`, `strings`). - Flutter client (`bug_report_screen.dart`) was already not sending a `hashed_ip` field — no client changes needed. ## Test plan - [x] `go build ./...` in `server/bugreport/` passes with no errors - [x] `go vet ./...` passes with no warnings - Reports without a contact email produce only `report.json` (no `mail.eml`) - Reports with a contact email produce `report.json` (no `email` key) and `mail.eml` containing the address Closes #441 🤖 Generated with [Claude Code](https://claude.com/claude-code)
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
NeedSupervisor
Issue escalated to a human supervisor; agentloop will skip it until cleared.
 State/InProgress
State/Later
State/Planned
automerge
Eligible for automatic merge by CI
 ci-failure
Issue opened by agentloop to track a failing CI workflow; used for deduplication.
 do-not-merge
Plan PR — review only, do not merge.
 loop/code
Add to run the built-in "code" prompt; override at prompts/code.md.
 loop/code-ci-pending
Prompt "code" finished; waiting for the PR's CI to pass before advancing.
 loop/code-done
Prompt "code" finished successfully.
 loop/code-in-process
Agent for the "code" prompt is currently running on this issue.
 loop/merge
Managed by agentloop
 loop/merge-done
Managed by agentloop
 loop/merge-in-process
Managed by agentloop
 loop/plan
Add to run the built-in "plan" prompt; override at prompts/plan.md.
 loop/plan-done
Prompt "plan" finished successfully.
 loop/plan-in-process
Agent for the "plan" prompt is currently running on this issue.
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
guettlibot guettli
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: guettli/sharedinbox#442