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

bugreport service - update #441

New Issue
Closed
opened 2026-06-05 12:36:45 +00:00 by guettli · 1 comment
guettli commented 2026-06-05 12:36:45 +00:00 (Migrated from codeberg.org)
Copy Link
Copy Source

bugreport sending part:

do not send hashed_ip.

bugreport service:

do not store hashed_ip. No IP is needed at all.

Store email in an mail.eml in the directory, not in report.json

## bugreport sending part: do not send hashed_ip. --- ## bugreport service: do not store hashed_ip. No IP is needed at all. Store email in an mail.eml in the directory, not in report.json
guettlibot commented 2026-06-05 12:45:03 +00:00 (Migrated from codeberg.org)
Copy Link
Copy Source

PR created at https://codeberg.org/guettli/sharedinbox/pulls/442.

Here's a summary of what was done for issue #441:

Changes made to server/bugreport/main.go:

  1. Removed hashed_ip entirely — dropped the HashedIP field from BugReport, deleted the hashIP function, and removed all IP extraction logic (including the X-Forwarded-For handling). No IP address is collected or stored in any form.

  2. Moved contact email out of report.json — removed the Email field from the BugReport struct. When a user opts to include their contact email for follow-up, it is now written to a separate mail.eml file in the report directory instead of being embedded in report.json. This keeps PII isolated from the structured report data.

  3. Removed now-unused imports — crypto/sha256, encoding/hex, and strings were all only used for IP hashing and are now gone.

The Flutter client (bug_report_screen.dart) was already not sending any hashed_ip field, so no client-side changes were needed.

Verified: go build ./... and go vet ./... both pass cleanly.

Pull request: https://codeberg.org/guettli/sharedinbox/pulls/442

PR created at https://codeberg.org/guettli/sharedinbox/pulls/442. --- Here's a summary of what was done for issue #441: **Changes made to `server/bugreport/main.go`:** 1. **Removed `hashed_ip` entirely** — dropped the `HashedIP` field from `BugReport`, deleted the `hashIP` function, and removed all IP extraction logic (including the `X-Forwarded-For` handling). No IP address is collected or stored in any form. 2. **Moved contact email out of `report.json`** — removed the `Email` field from the `BugReport` struct. When a user opts to include their contact email for follow-up, it is now written to a separate `mail.eml` file in the report directory instead of being embedded in `report.json`. This keeps PII isolated from the structured report data. 3. **Removed now-unused imports** — `crypto/sha256`, `encoding/hex`, and `strings` were all only used for IP hashing and are now gone. The Flutter client (`bug_report_screen.dart`) was already not sending any `hashed_ip` field, so no client-side changes were needed. **Verified:** `go build ./...` and `go vet ./...` both pass cleanly. Pull request: https://codeberg.org/guettli/sharedinbox/pulls/442
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
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 loop/code-done
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#441