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

CAUTION: Clef Wave May Cause CPU Burnout #208

Closed
CelticParser opened this issue May 5, 2015 · 1 comment
Closed

CAUTION: Clef Wave May Cause CPU Burnout #208

CelticParser opened this issue May 5, 2015 · 1 comment

Comments

@CelticParser
Copy link

Hi, I discovered If a user logs out and leaves the browser tab open (and active), the Clef Wave ramps the CPU to ~91% until he/she logs back in.

A team member today discovered that their laptop got extremely hot after a few minutes with the login screen left open. We checked the CPU usage in task manager when the WordPress login was left open and the tab active. If the Clef Wave is set as the default "primary login option", the CPU runs between 89% and 94% and when switching tabs (or close the wp login), the CPU will drop to a reasonable rate.

It may be a good idea to implement a redirect sending the user to the home page after logging out and/or add a function to switch from the Clef Wave to the "Log in with your phone" after a timeout interval (or something to that effect). Over the weekend, I personally lost a Chromebook - Apparently the CPU burnt up! After the fact of discovering today's indecent, I remembered on Friday, I logged out of WP and left the tab open. So most likely this was the cause of my Chromebook getting fried. I am concerned that this could become a serious liability if more machines are reported being damaged due to excessive CPU utilization over an extended period of time with this scenario.

Also, on a non-related note: Possibly move the plugin entry point from the upper-level Admin Menu to the Tools sub-menu using the add_options_page. This plugin is more of a "Set one time and forget about it." This would follow WP's best practices by not cluttering up the Admin Panel with "settings-type" menus.

Thank-you very much

-j

@jessepollak
Copy link
Member

Thanks for the heads up on this, we're looking into this issue!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants