Skip to content

[PS-1896] Add support for Vanadium in GrapheneOS#2199

Merged
mpbw2 merged 2 commits intobitwarden:masterfrom
flawedworld:vanadium-rebrand
Dec 8, 2022
Merged

[PS-1896] Add support for Vanadium in GrapheneOS#2199
mpbw2 merged 2 commits intobitwarden:masterfrom
flawedworld:vanadium-rebrand

Conversation

@flawedworld
Copy link
Copy Markdown

@flawedworld flawedworld commented Nov 18, 2022

Type of change

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

Objective

GrapheneOS changed the package name of Vanadium from org.chromium.chrome to app.vanadium.browser so a new entry is needed for Bitwarden to work with Vanadium. See: GrapheneOS/Vanadium@fd37911

Code changes

I just added app.vanadium.browser as a supported Chromium browser.

Screenshots

Before you submit

  • Please check for formatting errors (dotnet format --verify-no-changes) (required)
  • Please add unit tests where it makes sense to do so (encouraged but not required)
  • If this change requires a documentation update - notify the documentation team
  • If this change has particular deployment requirements - notify the DevOps team

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Nov 18, 2022

CLA assistant check
All committers have signed the CLA.

@bitwarden-bot
Copy link
Copy Markdown

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

@bitwarden-bot bitwarden-bot changed the title Add support for Vanadium in GrapheneOS [PS-1896] Add support for Vanadium in GrapheneOS Nov 18, 2022
@flawedworld
Copy link
Copy Markdown
Author

If there are any blockers to this pull request please let me know.

@ghost
Copy link
Copy Markdown

ghost commented Dec 8, 2022

@vvolkgang
@mp-bw
I am sorry for tagging you, but it would we very helpful if you could describe to us what we need to do to get this Pull request accepted. Thank you for your time.

@mpbw2 mpbw2 merged commit 2f6fd47 into bitwarden:master Dec 8, 2022
@flawedworld
Copy link
Copy Markdown
Author

Thank you!

@ghost
Copy link
Copy Markdown

ghost commented Jan 13, 2023

@flawedworld Hey, the new version of Bitwarden released. Is the patch working on your device? For me its not working on my Pixel 4a with GrapheneOS.

@BHydden
Copy link
Copy Markdown

BHydden commented Jan 14, 2023

Bitwarden version 2023.1.0
Pixel 6
Android 13
GrapheneOS TQ1A.230105.002.2023011000
Vanadium 109.0.5414.86

Autofill still not working for me either.

@BHydden
Copy link
Copy Markdown

BHydden commented Jan 14, 2023

GrapheneOS changed the package name of Vanadium from org.chromium.chrome to app.vanadium.browser so a new entry is needed for Bitwarden to work with Vanadium

If it's relevant, I installed GrapheneOS before this change, so my Vanadium still has the package name

org.chromium.chrome
versionCode 541408633

@ghost
Copy link
Copy Markdown

ghost commented Jan 14, 2023

Yeah my package is also:

org.chromium.chrome
versionCode 541408633

and i think it is because of this sentence in the patch notes of graphene os:

Vanadium: change app id to app.vanadium.browser while using original-package to keep using org.chromium.chrome for existing installs (until factory reset) to preserve compatibility and user data

But in my opinion there shouldnt be a problem for us in the first place, because if the package wasnt renamed for us and the old package has been supported by bitwarden for years.
So maybe it is a problem inside vanadium?

@flawedworld
Copy link
Copy Markdown
Author

The patch works fine, it merely restores the former behaviour to when we formerly were using "org.chromium.chrome".

@ghost
Copy link
Copy Markdown

ghost commented Jan 14, 2023

@flawedworld But why isnt it working for me and BHydden anymore? As i said, there shouldnt even been a problem as we are using the old naming schemes for Vanadium. But bitwarden is broken for me atleast, since the renaming of Vanadium even though my package is still named the old way.

@BHydden
Copy link
Copy Markdown

BHydden commented Jan 14, 2023

Yep same for me even though I have the old package name Bitwarden autocomplete broke around the time the new name was added for new installs. Not sure how to diagnose further.

@jceloria
Copy link
Copy Markdown

Add me to the list. My wife's Pixel 5 has org.chromium.vanadium and works. My phone has org.chromium.chrome and now no longer works.

@ghost
Copy link
Copy Markdown

ghost commented Jan 16, 2023

@flawedworld
I've seen that you are contributing alot to GrapheneOS. Is it maybe possible for you to have a look at this problem or forward it to the right person?

@flawedworld
Copy link
Copy Markdown
Author

flawedworld commented Jan 17, 2023

Possible that there may be some form of bug in AOSP to do with the package id migration. Needs to be looked into. In the meantime if you can't wait for a fix then try clearing data of Vanadium if that's an option for you.

@johnceloria
Copy link
Copy Markdown

johnceloria commented Jan 17, 2023

I've cleared cache and data a couple of times between reboots and the app is still reporting:

version 109.0.5414.86

org.chromium.chrome
versionCode 541408633

targetSdk 33
minSdk 29

Installed: Dec 31, 2008; 19:00

I've done the same with Bitwarden as well.

@ghost
Copy link
Copy Markdown

ghost commented Jan 17, 2023

I've also cleared the data and cache of both apps (vanadium, bitwarden), sadly the behaviour didnt change and it is still not working.

@Enlinks
Copy link
Copy Markdown

Enlinks commented Jan 22, 2023

Testing on a formated device this unfortunately still isn't working. Works just fine on my other browser (Fennec)

version 109.0.5414.86
app.vanadium.browser
versionCode 541408633
targetSdk 33
minSdk 29
Installed: Jan 1, 2009; 12:00 AM

version 2023.1.0
com.x8bit.bitwarden
versionCode 5786
targetSdk 33
minSdk 21
Installed: Jan 22, 2023; 8:53 PM

@m-delc
Copy link
Copy Markdown

m-delc commented Feb 7, 2023

Bitwarden autofill does not work on my GrapheneOS device

app.vanadium.browser
versionCode 548106133
targetSdk 33
minSdk 29
Updated: Dec 31, 2008; 6:00 PM

version 2023.1.0
com.x8bit.bitwarden
versionCode 5786
targetSdk 33
minSdk 21
Installed: Feb 4, 2023; 11:49 AM
Updated: Feb 6, 2023; 8:11 AM

@Enlinks
Copy link
Copy Markdown

Enlinks commented Feb 10, 2023

Been working with the GrapheneOS team and it looks like to get this to work is a bit tedious but possible.

I was only able to get it working with the new app.vanadium.browser however.

GrapheneOS/Vanadium#244

Try following this instruction from GrapheneOS Testing room:

https://matrix.to/#/!AUltSswJuGkDFXqGti:grapheneos.org/$SGtho6Zk9gv6LGPd981enOVSzTyEyzqVMzRk11rDoDE?via=grapheneos.org&via=matrix.org&via=tchncs.de

Quote:

I have bitwarden autofill working but it was a real pain. In the bitwarden settings for auto-fill services, I needed to enable all 4 options. Vanadium can't handle inliine autofill but draw-over works. The problem occurs when toggling Use accessibility and Bitwarden is grayed out. Tap it to see that it's restricted. You have to give BW permission by going into Settings>Apps>Bitwarden (not within the app but regular settings). Hopefully there will be three dots upper right hand corner. Select and allow restricted. Now, go back to the bitwarden app and select Use Accessibility again. Tihs time it should not be grayed out. Tap it to enable it. Then, finally, go back to the Bitwarden settings to enable draw over. Follow prompts to allow.

@BHydden
Copy link
Copy Markdown

BHydden commented Feb 10, 2023

I was only able to get it working with the new app.vanadium.browser

I have done all 4 steps and can confirm it does not fix the problem for the old org.chromium.chrome app name

Pixel 6 Oriole TQ1A.230205.002.2023021000

Vanadium Version 110.0.5481.64
org.chromium.chrome
versionCode 548106433

Really hoping to find a solution to this problem that doesn't involve formatting my device.

@raptor170
Copy link
Copy Markdown

Also same issue here, have the newest bitwarden version installed and doesn't work on vanadium, works well on Firefox etc.

Pixel 6
Grapheneos

Vanadium
Version 110.0.5481.65

org.chromium.chrome
versionCode 548106533

@ljrk0
Copy link
Copy Markdown

ljrk0 commented Feb 18, 2023

This is closed as wontfix right now on GrapheneOS side as a better solution is underway at GrapheneOS/Vanadium#276.

The way autofill did work before was to (ab)use accessibility features instead of the actual Auto Fill API to mimick the latter in BitWarden. Actual Auto Fill never worked in Chrome/Chromium/Vanadium/... because it has its own Play AutoFill capabilities. Only this crutch, which also required putting Vanadium on this allow list.

Vanadium will implement Android Auto Fill properly (see above) which should solve the issue.

@BHydden
Copy link
Copy Markdown

BHydden commented Feb 27, 2023

This is fixed now with https://github.com/GrapheneOS/Vanadium/releases/tag/110.0.5481.154.1

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.