buttercup / buttercup-browser-extension

:earth_asia: Buttercup browser extension
https://buttercup.pw
MIT License
230 stars 42 forks source link

Fix form popup copy button #Bug #338

Closed ldexterldesign closed 3 years ago

ldexterldesign commented 4 years ago

Hi!

Hope you're well

Thanks for wonderful software

As title

Experiencing this in latest vivaldi and brave but not in firefox which makes me think perhaps it's a chromium issue?

Unexpected behaviour

Unable to copy username/password

Expected behaviour

Able to copy username/password

Screen shot:

Screenshot 2020-05-06 at 12 21 19

FYI highlighting/selecting the field value manually and copying functions OK so it's something to do with the button click event I imagine

If you're reading and:

If you have any issues (e.g. questions/queries) then happy to help

Keep up great work!

Hope this helps & to hear back

Sincerely

hardware: Apple, MacBook Pro (16-inch, 2019) software - operating system: Apple, macOS, 10.15.4 (19E287) software - application: brave, 1.8.86 Chromium: 81.0.4044.129 (Official Build) (64-bit) firefox, 76.0 (64-bit) vivaldi, 3.0.1874.33 (Stable channel) (64-bit)

ldexterldesign commented 4 years ago

FYI the address bar popup (i.e. off the extension button/icon) copy buttons seem to function OK:

Screenshot 2020-05-06 at 20 02 43
perry-mitchell commented 4 years ago

Hi @ldexterldesign - Apologies for not getting back earlier. Are you still able to reproduce this issue? I cannot on the current build.

ldexterldesign commented 4 years ago

2.17.0 yes

Will try remember to follow up when I get access to 2.18.0 but if not then feel free to ping me

Sincerely

hardware: Apple, MacBook Pro (16-inch, 2019) software - operating system: Apple, macOS, 10.15.5 (19F101) software - application: vivaldi, 3.1.1929.45 (Stable channel) (64-bit)

perry-mitchell commented 4 years ago

Is it possible that it's an issue with Vivaldi? Haven't seen any other reports like this for Chrome/Firefox.

ldexterldesign commented 4 years ago

Yes, I guess it's possible

I will wait to see if the issue exists in 2.18.0 before investigating with Vivaldi

Cheers

ldexterldesign commented 4 years ago

Yes, issue exists in 2.18.0, both latest Vivaldi and Brave so, again, looks like a Chromium issue

Anyone else verify this?

Sincerely

perry-mitchell commented 4 years ago

Not reproducible on my Mac nor Linux machine, in Chrome. Haven't yet tested in Brave.

@ldexterldesign Any chance you could try this on another computer?

ldexterldesign commented 4 years ago

Sorry, don't have access to another machine

I broke my policy and installed some Google products for you though, where I can still reproduce the issue

I still think it's a Chromium issue but it could be a quirk concerning my OS install. Still, weird it works in Firefox 😕.

I guess wait and see if anyone else can reproduce...

Sincerely

software - operating system: Apple, macOS, 10.15.5 (19F101) software - application: brave, 1.11.97 vivaldi, 3.1.1929.45 chromium, 86.0.4207.0 chrome, 84.0.4147.89

perry-mitchell commented 4 years ago

Hmm. I have a feeling it’s an OS thing, as I can’t reproduce it on any machine, and we haven’t seen another report. That being said, I’m happy to leave this open until we find what the cause. You could open the inspector in the pop up menu to see if there’s an error when you click the copy button.

ldexterldesign commented 4 years ago

Nothing in console

When I load the iframe src in a new tab the button copies as expected:

Screenshot 2020-07-21 at 00 19 17

After reading Interact with the clipboard a quick poke around in console gives me the impression we're using execCommand() method when we could possibly benefit from using Clipboard API

NB execCommand() is now obsolete in everything but Firefox, which could be the reason the feature is broken in everything else, specifically Chromium

Still doesn't explain a) why you can't reproduce it and b) why it copies in its own tab and in browser toolbar but not in the context of an iframe, which leads me to reason it could be a security thing Chrome has imposed (they're good at that)

🤷‍♂️

ldexterldesign commented 4 years ago

FYI I plan to reinstall macOS fresh ~Oct/Nov so will follow up then regardless

perry-mitchell commented 4 years ago

@ldexterldesign Hmm, but we don't use execCommand to my knowledge: https://github.com/buttercup/buttercup-browser-extension/search?q=execCommand&unscoped_q=execCommand

We use the navigator.clipboard API.

Upon looking further into this, I can see that we actually have a permissions issue:

image

And we don't have that permission set, so that's most certainly an issue - if not the issue.

Thanks for looking into this! Seems we have something we can try at least.

perry-mitchell commented 4 years ago

Ok, so I tested in Brave (1.11.101) on Ubuntu 18 and could not reproduce. I'll still add the permission, but I doubt it'll fix the issue for you.

At this stage I think the issue might be a local one, and not something other users are experiencing (due to the fact that I can't reproduce it across multiple computers and browsers, across multiple vaults). I'll leave it open for you to test on Mac, however.

ldexterldesign commented 3 years ago

I'll still add the permission, but I doubt it'll fix the issue for you.

Is this change in 2.23.0?:

So frustrating nobody else can reproduce this; desperately need to find bandwidth to reinstall macOS and test

Sincerely

perry-mitchell commented 3 years ago

Is this change in 2.23.0

@ldexterldesign yes - here

Happy to try more hands-on tactic's, as well. If you're able to reproduce in a secure machine, I'd be more than happy to use team viewer or something to debug on the fly. Or if you find some way for us to reproduce the issue, I'll do my best to jump on fixing it immediately.

ldexterldesign commented 3 years ago

FYI

I have a feeling it’s an OS thing, as I can’t reproduce it on any machine, and we haven’t seen another report.

Created a fresh macOS account, which shares browser installs but generates new browser profiles - as close as I can get to a macOS reinstall - and same issue so, yes, can only think it's OS specific

Also tested a local vault, in addition to my dropbox one, and same issue

I'll close this issue for now and reopen if it's still an issue after I reinstall macOS

Thank for help offer(s)

Sincerely