Skip to content

[PM-1568] Support Mulch browser for autofill#2427

Closed
fd1f wants to merge 3578 commits intobitwarden:mainfrom
fd1f:master
Closed

[PM-1568] Support Mulch browser for autofill#2427
fd1f wants to merge 3578 commits intobitwarden:mainfrom
fd1f:master

Conversation

@fd1f
Copy link

@fd1f fd1f commented Mar 23, 2023

Type of change

  • Bug fix
  • New feature development
  • Tech debt (refactoring, code cleanup, dependency upgrades, etc)
  • Build/deploy pipeline (DevOps)
  • Other

Objective

This PR adds the Mulch to the autofill whitelist.

Code changes

  • src/Android/Accessibility/AccessibilityHelpers.cs
  • src/Android/Autofill/AutofillHelpers.cs
  • src/Android/Resources/xml/autofillservice.xml

All to add Mulch support.

github-actions bot and others added 30 commits June 29, 2022 09:11
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
…improved the code so there are fewer direct access from the VM to the View (bitwarden#1879)
… service name hardcoded to register/resolve a service (bitwarden#1865)
We only need master password hash because this is currently
only used for sso password setting after auto-provisioning. Key
Connector is not involved in these accounts
…1976)

* Added to AppResources.resx the message warning the user about setting the lockout option to "Never"

* Added the condition to check the newly selected option on the vault timeout settings

* Changed the wording on the warning as to reflect the mobile version

* Changed the vault timeout modal to have the ability to cancel

* Simplified the reversion of value if the user cancels the change
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
…gOverlayHelper and also added more logging to see when it happens. (bitwarden#1983)
* feat: update auto totp copy setting

* feat: update show icons settings

* feat: update auto add settings

* feat: update settings and options to sentence case

* feat: update translation keys

With the latest changes the translation keys had diverged from their contents.
This commit fixes that.

* fix: revert AndroidManifest changes

* chore: add todo comments to fix negative functions
* PS-920 - Added feedback to user when saving bad formed URLs
* Added feedback to user when trying to perform login with bad formed URL

* PS-920 - Refactor to use AsyncCommand
*(missing file from previous commit)

* PS-920 - Fixed whitespace formatting

* PS-920 - Removed unused method

* PS-920 - Fixed validation
* Added comment for hard coded string

* PS-920 - Removed unused properties
* Fixed url validations
* Refactored method to local function

* PS-920 - Added exception handling and logging
* Added generic error message string to AppResources
…#1971)

* EC-259 Added Account switching on share extension on iOS, also improved performance for this and exception handling

* EC-259 code formatting

* EC-259 Added account switching to Share extension Send view

* EC-259 Fixed navigation on share extension when a forms page is already presented

* EC-259 Fix send text UI update when going from the iOS extension

* EC-259 Improved DateTimeViewModel with helper property to easily setup date and time at the same time and applied on usage
Added Opera GX Support for autofill
…1993)

* EC-324 Added more logging for trying to get more information on list out of range crash on AppCenter

* EC-324 Fix include on iOS.Core.csproj on iOS CollectionView files
…d Autofill on iOS given that the string was the same as Autofill Services and the comparison was misleading. Also refactored so that the action is on each item instead of having to compare to act (bitwarden#1989)
…n#1914)

* settings option to allow screen capture on Android

* Improved code on Screen Capture and added prompt to the user to allow screen capture

* EC-325 Removed async on OnCreate of MainActivity given that's not necessary anymore

Co-authored-by: Federico Maccaroni <fedemkr@gmail.com>
* [SG-467] Fixed url validation so it works with or without http or https

* [SG-467] Validation has been refactored for simplicity.
…den#2000)

* PS-1080 Added new accessibility text property to the custom field bool icon

* PS-1080 Added BoolValue property to the FieldView and added new AccessibilityProperty to bool icon
jlf0dev and others added 7 commits March 10, 2023 12:55
* [EC-1045] lock action if policy and show message

* [EC-1045] add text for policy message

* [EC-1045] add consts to policy service

* [EC-1045] missed a const

* [AC-1045] fix build
…om API (bitwarden#2390)

* [PM-1078] Fingerprint phrase gets calculated from pub key on AuthService instead of coming as a property from the api.
…warden#2417)

* [PM-1431] Do not clear Master password  if login is ongoing.
Co-authored-by: bitwarden-devops-bot <106330231+bitwarden-devops-bot@users.noreply.github.com>
* [PM-1497] Ignore know device api error.
Co-authored-by: bitwarden-devops-bot <106330231+bitwarden-devops-bot@users.noreply.github.com>
@CLAassistant
Copy link

CLAassistant commented Mar 23, 2023

CLA assistant check
All committers have signed the CLA.

@bitwarden-bot
Copy link

Thank you for your contribution! We've added this to our internal Community PR board for review.
ID: PM-1568

@bitwarden-bot bitwarden-bot changed the title Support Mulch browser for autofill [PM-1568] Support Mulch browser for autofill Mar 23, 2023
@fd1f fd1f force-pushed the master branch 3 times, most recently from b69c5d3 to a3f9c24 Compare March 23, 2023 06:52
@jcrabapple
Copy link

Someone please approve this I have been waiting for Bitwarden to work with Mulch! Thank you!

@bepoli
Copy link

bepoli commented Aug 29, 2023

Is this ever going to be merged?
Maybe we need to add some context, since devs may not be aware of Mulch:
Mulch browser is a chromium-based browser developed and maintained by the Divest Computing Group (the same devs of Mull browser, aka us.spotco.fennec_dos, currently supported by Bitwarden).
It is relevant since many users moved from Bromite (a browser currently supported by Bitwarden) to Mulch when the former stopped receiving security updates.

@unoukujou
Copy link

Somebody please add this. Thanks.

@streetsamurai00mi
Copy link

waiting for this to be merged

@dbosompem
Copy link

Hi All, apologies for the long review time. This has been approved by Product Management, and so Engineering will pick this up and assist till it's merged. Thank you for the patience!

@D3SOX
Copy link

D3SOX commented Nov 22, 2023

Hate to be that kind of guy but are there any updates on this? Merging this 5 line PR would save me and other Mulch users hours of time. I'm thinking about compiling it myself now, but is it even possible on Linux?
Edit: Used a KVM/QEMU Windows VM, checked out the latest release, cherry-picked the commit and after a lot of fighting with Microsoft products it works!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.