Skip to content

Import Cert functionality for Android App - #121

Open
EyeOfMidas wants to merge 2 commits into
Fancy-Mumble:mainfrom
EyeOfMidas:import-cert
Open

EyeOfMidas wants to merge 2 commits into
Fancy-Mumble:mainfrom
EyeOfMidas:import-cert

Conversation

@EyeOfMidas

@EyeOfMidas EyeOfMidas commented Oct 3, 2026 •

Copy link
Copy Markdown

Description

This adds an option in the cert selection dropdown to import a cert (.fmid) from a file on the system. Otherwise, you're forced to create a new account per device and the SuperUser or an admin has to register each one.

Changes

  • Added Import Certificate to the initial screen of the android app
  • This required adding the ability to read files off the system, so I added the tauri filesystem plugin
  • Used translation tools to localize the text as best I could

I can alter or remove the following changes, I added them in because I had trouble building without them:

  • added android CMAKE_TOOLCHAIN values so building can work immediately
  • added the protobuf dependency since it wouldn't build without it
  • Installed latest rust 1.98.1 and fixed a few warnings that didn't happen for older versions

Checklist

  • [X ] Code compiles without warnings (cargo clippy --workspace -- -D warnings)
  • [X ] TypeScript type-checks (npx tsc --noEmit)
  • [X ] Frontend tests pass (npm test)
  • [X ] Rust tests pass (cargo test --package mumble-protocol --features opus-codec --lib)
  • [X ] New functionality has tests
  • [X ] Boy Scout Rule applied - files left cleaner than found

…t exported fmid cert file; used translation tools to build out localized text for import; adding toolchain wrappers in the config.toml
@EyeOfMidas
EyeOfMidas marked this pull request as ready for review October 3, 2026 14:26
@EyeOfMidas

Copy link
Copy Markdown
Author

Tangentially related: the mumble-server repo you've forked has a build issue, but the Issues section is not enabled for it, so I'm not sure how to report.

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