Skip to content
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

[Request] "Copy" button for API key #2056

Closed
b-morgan opened this issue Aug 4, 2017 · 15 comments
Closed

[Request] "Copy" button for API key #2056

b-morgan opened this issue Aug 4, 2017 · 15 comments
Labels
done Done but not yet released request Feature request

Comments

@b-morgan
Copy link

b-morgan commented Aug 4, 2017

When attempting to "copy" the API key, nothing happens in Chrome and Firefox but it works with Edge. I do have LastPass (but Edge doesn't support it).

@GitIssueBot
Copy link

Hi @b-morgan,

It looks like there is some information missing from your bug report that will be needed in order to solve the problem. Read the Contribution Guidelines which will provide you with a template to fill out here so that your bug report is ready to be investigated (I promise I'll go away then too!).

If you did not intend to report a bug but wanted to request a feature or brain storm about some kind of development, please take special note of the title format to use as described in the Contribution Guidelines.

Please do not abuse the bug tracker as a support forum - if you have a question or otherwise need some kind of help or support refer to the Mailinglist or the G+ Community instead of here.

Also make sure you are at the right place - this is the bug tracker of the official version of OctoPrint, not the Raspberry Pi image OctoPi nor any unbundled third party OctoPrint plugins or unofficial versions. Make sure too that you have read through the Frequently Asked Questions and searched the existing tickets for your problem - try multiple search terms please.

I'm marking this one now as needing some more information. Please understand that if you do not provide that information within the next two weeks (until 2017-08-18 14:10 UTC) I'll close this ticket so it doesn't clutter the bug tracker. This is nothing personal, so please just be considerate and help the maintainers solve this problem quickly by following the guidelines linked above. Remember, the less time the devs have to spend running after information on tickets, the more time they have to actually solve problems and add awesome new features. Thank you!

Best regards,
~ Your friendly GitIssueBot

PS: I'm just an automated script, not a human being, so don't expect any replies from me :) Your ticket is read by humans too, I'm just not one of them.

@GitIssueBot GitIssueBot added the incomplete Issue template has not been fully filled out, no further processing until fixed label Aug 4, 2017
@b-morgan
Copy link
Author

b-morgan commented Aug 4, 2017

What were you doing?

When attempting to "copy" the API key, nothing happens in Chrome and Firefox but it works with Edge. I do have LastPass (but Edge doesn't support it).

What did you expect to happen?

Key in clipboard.

What happened instead?

Nothing.

Did the same happen when running OctoPrint in safe mode?

[Try to reproduce your problem in safe mode. You can find information
on how to enable safe mode in the Contribution Guidelines.]

Branch & Commit or Version of OctoPrint

1.3.4

Operating System running OctoPrint

OctoPi, version 0.14.0

Printer model & used firmware incl. version

Lulzbot TAZ 6, RAMBo 1.3L, Marlin 1.0.2.22

Browser and Version of Browser, Operating System running Browser

Windows 10 Professional, Chrome 60.0.3112.90 (Official Build) (64-bit), Firefox 54.0.1 (64-bit)

Link to octoprint.log

octoprint.zip

@markwal
Copy link
Member

markwal commented Aug 4, 2017

I think this is a security fix it some browsers. Some webapps (OneNote comes to mind) pop up a message when you click the copy button that says you have to use Ctrl-C or the right click menu.

@b-morgan
Copy link
Author

b-morgan commented Aug 4, 2017

I tried ctrl-C and right-click Copy. Neither worked (in Chrome and Firefox). Both worked in Edge.

@ntoff
Copy link
Contributor

ntoff commented Aug 4, 2017

Works ok here on chrome running on windows 10 pro

double click the key (or click and drag) to highlight it all -> right click -> copy. Does that not work for you?

@b-morgan
Copy link
Author

b-morgan commented Aug 5, 2017

To reiterate what I said in my previous posts, on Windows 10 Pro, Chrome 60.0.3112 I can highlight the key but right-click Copy does not work for me. Same results using Firefox (right-click Copy doesn't work).

With Microsoft's Edge browser, I can highlight and right-click Copy without any issues.

I think a key piece of this puzzle is that I have LastPass installed which has addons for Chrome and Firefox but does not have one for Edge (or rather, Edge doesn't yet support addons). I did try disabling LastPass in Firefox but that did NOT change the symptoms (right-click Copy doesn't work).

@ntoff
Copy link
Contributor

ntoff commented Aug 5, 2017

Ok well it works in chrome but I just tried firefox and even without lastpass I can't copy the key either. What I CAN do, is drag it onto notepad++ which inserts it into a document and then leaves OctoPrint hanging in limbo on the upload overlay so I guess bug confirmed? But only for firefox. Chrome still works ok for me (I don't use lastpass).

@KC703
Copy link

KC703 commented Aug 5, 2017 via email

@b-morgan
Copy link
Author

b-morgan commented Aug 5, 2017

I'm a little embarrassed. I just tried Chrome again and it works. That leaves us with a Firefox issue which you all have confirmed.

@ntoff
Copy link
Contributor

ntoff commented Aug 11, 2017

I wonder if a little "copy" button should be placed next to the API key, allowing someone to just press that to copy the key into the clipboard.

@GitIssueBot
Copy link

Since apparently some of the required information is still missing, I'm closing this now, sorry. Feel free to request a reopen of this or create a new issue once you can provide all required information.
This is nothing personal. Thank you for your collaboration.

@foosel foosel added the request Feature request label Aug 22, 2017
@foosel foosel changed the title System API key cannot be copied [Request] "Copy" button for API key Aug 22, 2017
@foosel foosel removed the incomplete Issue template has not been fully filled out, no further processing until fixed label Aug 22, 2017
@foosel foosel reopened this Aug 22, 2017
foosel added a commit that referenced this issue Aug 22, 2017
There's now a dedicated button for that.

This should get around issues like #2056. Also increased size
of API key display field (where possible). Fixes #2055
@foosel foosel added the done Done but not yet released label Aug 22, 2017
@foosel foosel added this to the 1.3.6 milestone Aug 22, 2017
@foosel
Copy link
Member

foosel commented Aug 22, 2017

I have no idea what Firefox' problem is here, but regardless, I've added a bunch of copy buttons to the various API key fields (Settings > API, Settings > Access Control, User settings).

image

Pushed to maintenance, soon devel and to be released with 1.3.6.

foosel added a commit that referenced this issue Nov 27, 2017
Further fix for FF issues with disabled input fields as tracked in
#2056 and #2196.
@sepal
Copy link

sepal commented Dec 12, 2017

Just had this issue as well and did a little bit of research:
Apperently this is new in Firefox Quantum (aka 57.0) and is a bug fix, since the HTML5 standard specifies that users shouldn't be able to interact with a disabled inputs.
The API key input should instead use the read only attribute, which allows to copy from, but not manipulate the form elements.

@foosel
Copy link
Member

foosel commented Dec 12, 2017

The API key input should instead use the read only attribute, which allows to copy from, but not manipulate the form elements.

And that was changed in cf25291 and will be part of 1.3.6 as well.

@foosel
Copy link
Member

foosel commented Dec 12, 2017

Closing because 1.3.6 was just released.

@foosel foosel closed this as completed Dec 12, 2017
@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 29, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
done Done but not yet released request Feature request
Projects
None yet
Development

No branches or pull requests

7 participants