-
Notifications
You must be signed in to change notification settings - Fork 213
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
sysPass 2.1.2 - copy to clipboard error #500
Comments
Hi Lukas, which browser are you using? (sorry I missed your previous post) |
FireFox ESR 45.6.0 |
Ok, so you told that this issue is related when clicking on the "Copy to Clpboard" icon that is outside the password box. Does the password contain any special char?, Does it work on the demo site? |
On demo site it is working. One password is -> MaxNCBGLpS |
with Version 2.0 it was working fine... |
Hey Rubén, I found something out. When I use the "copy to clipboard" button direct on page which shows me my entries, it doesn't work. When I click 1st on "show password" button and press then "copy to clipboard" button, this is working fine. I hopy you can understand my bad english :) Br, |
Hey Lukas! I'm doing some tests. I'll be back shortly. Your English is quite good...Mine isn't perfect too :) |
Hi Rubén, i updated to current sysPass version, but the copy to clipboard issue is still present :-( Br, |
I have some problem too ! All logs is clean. Debug not show trouble. |
@maxdie sorry I don't know what you mean :( |
Ok @maxdie ...Now I see it's the same issue, the weird thing is that this works on the demo site, does it work for you? |
Hello. |
How we can help you Rubén to do some troubleshooting? |
Now I see the error on the demo site, it's weird because it didn't happen before. I'll take a deeper look. |
I love your work and your progress ! ;-) |
Thanks, I've been testing on Firefox and chrome on Windows and Linux and it didn't fail. I could reproduce the issue on Firefox on android. |
@nuxsmin do you have any idea how to fix this problem? |
@lukasp2 I've been doing some tests and finally I guessed that some mobile browsers don't implement a feature that is required to run a "text copy" action: https://developer.mozilla.org/en-US/docs/Web/API/document/execCommand The doc tells that Safari and Firefox for iOS don't implement the "execCommand" feature, so the "copy-to-clipboard" feature won't work until being implemented. I don't know if you are testing on a mobile device, but if it were the case, now you have the answer. Is it? BTW, I'm trying to find out why it doesn't work on Android mobile devices using Firefox/Chrome. @maxdie |
@nuxsmin I'm trying it with FireFox ESR 45.6.0, IE 11.0.9600.18537 and Vivaldi 1.6.689.40 running on Windows 7 Enterprise SP1 64bit. With all Browsers I get same error while trying to copy password to clipboard When I use the "copy to clipboard" button direct on page which shows me my entries, it doesn't work. When I click 1st on "show password" button and press then "copy to clipboard" button, this is working fine. Where is the difference between this both "copy to clipboard" functions? Thank you, |
@lukasp2 the main difference is that the first one (using the direct button) gets the password from an AJAX request, passes it to the Javascript plugin to copy it to the clipboard. The second one, performs an Ajax request, BUT it sets the password in a field that is shown (password box) and then, when you press the "copy" button, it catches the password from the field and copies it to the clipboard. To get the clipboard working, the action for copying the text (in this case the password), need to be executed in the user space, because there are some security restrictions that avoid to copy any text in a background process. It's a bit complicated :) From: https://developer.mozilla.org/en-US/Add-ons/WebExtensions/Interact_with_the_clipboard
|
Okay, also my FireFox ESR Release it not up to date. I will try that @Home with current FireFox / Google Chrome releases. |
Update: I tested it on my MacBook (macOS Sierria 10.12.3) with Chrome 57 and FireFox 52....same issue |
I'm still working on a solution. This is a browser security behavior that is planned to be redesigned to allow asynchronous requests to perform clipboard actions if the user grants permission before using it. |
For me, still have error on my install version 2.1.4 I ma using Chrome How can we help to debug ? When using developer tools |
Hello, since this issue occurs in a random way and the current clipboard behavior is a bit restrictive (see a comment above), I could figure out that it happens when the response from the server is delayed, so the password from the Ajax request is not set. The current behavior is as follows:
Currently a new clipboard API is being developed (working draft), so Ajax actions will be allowed to send text to the clipboard (https://www.w3.org/TR/clipboard-apis/#integration-with-other-scripts-and-events) In the meantime, I think that the current session encryption could be the cause of the issue, because it reduces the general performance in terms of speed when performing requests. |
Could you try out the latest commit in the master branch? |
When I activate session encryption, then is copy to clipboard working for me with newest Vivaldi Browser. |
Works for me, encryption enabled or not. So now, I am waiting the new official release. |
That sounds nice |
It's working by me too. 👍 |
Ummm...that sounds to be more likely to a browser update, because there weren't any changes to the clipboard functions code since a few commits ago. The latest commit only implements some new features like enable/disable PHP session encryption and secondary groups accesses. Thanks for the feedback! 👍 |
Ok, but i have tested it before and after upgrading from 2.1.1 to 2.1.4 and then to the latest master branch. I haven't closed my browser or updated it (chrome)... |
Ok, in your case, there were some intermediate releases that implemented some fixes to that behavior. |
Solved. Thanks to everyone for the feedback! |
Doesn't work for me, with Opera 47 (lastest) Win x86.
|
Hi,
I can't copy any password to clipboard. I'm getting a error:
No entries regarding this in syspass.log available.
The text was updated successfully, but these errors were encountered: