Skip to content

Pass an anonymization level to the debug bundle SDK call - #234

Merged
lixmal merged 1 commit into
mainfrom
anonymize-level
Aug 10, 2026
Merged

Pass an anonymization level to the debug bundle SDK call#234
lixmal merged 1 commit into
mainfrom
anonymize-level

Conversation

@lixmal

@lixmal lixmal commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

Adapts the app to the debug bundle anonymization level introduced in netbirdio/netbird#7102 and bumps the netbird submodule to include it.

A troubleshoot-screen option for the strict level can follow separately.

@coderabbitai

coderabbitai Bot commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

The NetBird submodule reference advances to a newer commit. EngineRunner.debugBundle passes Android.AnonymizeLevelDefault to the Go client. The public Java method signature remains unchanged.

Changes

Debug Bundle Client Update

Layer / File(s) Summary
Client reference and debug-bundle call
netbird, tool/src/main/java/io/netbird/client/tool/EngineRunner.java
The submodule advances to commit 5584f8ef0a8dbd7d1db49655914d7c2c39b431cf. EngineRunner.debugBundle passes Android.AnonymizeLevelDefault and documents that strict anonymization is unused.

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related PRs

Suggested reviewers: pappz

Poem

A rabbit checks the bundle call,
The default level passes all.
The client pointer hops ahead,
Strict mode waits, unused instead.
Small changes land with carrots bright.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly describes the main change: passing an anonymization level to the debug bundle SDK call.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch anonymize-level

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@lixmal
lixmal merged commit f5eb122 into main Aug 10, 2026
7 checks passed
@lixmal
lixmal deleted the anonymize-level branch August 10, 2026 10:54
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.

2 participants