Skip to content
This repository has been archived by the owner on Sep 16, 2019. It is now read-only.

Login Box #141

Closed
DesignTheBox opened this issue Nov 2, 2014 · 5 comments
Closed

Login Box #141

DesignTheBox opened this issue Nov 2, 2014 · 5 comments

Comments

@DesignTheBox
Copy link

Hi Unsure if this is the correct place to comment but i have the foundationpress theme and when i go to login if i put the wrong password or user name the error box does not display the text

@DesignTheBox
Copy link
Author

also once uploaded to live server i cannot login aparently its a theme issue can someone help :)
www.rustalicious.com

@olefredrik
Copy link
Owner

The login error text is disabled for security reasons. However, if you want to display the login error text, you can remove this line add_filter('login_errors',create_function('$a', "return null;")); (on line 82 in library/cleanup.php) and you should be good to go

If you are unable to login, it is very unlikely that this is a theme issue.

@DesignTheBox
Copy link
Author

Ok thanks

I figured out the login issue
I think it was related to WP code I added to change login page styling
On 3 Nov 2014 11:44, "Ole Fredrik Lie" notifications@github.com wrote:

The login error text is disabled for security reasons. However, if you
want to display the login error text, you can remove this line add_filter('login_errors',create_function('$a',
"return null;")); (on line 82 in library/cleanup.php) and you should be
good to go

If you are unable to login, it is very unlikely that this is a theme issue.


Reply to this email directly or view it on GitHub
#141 (comment)
.

@groovenectar
Copy link

Why not make that an option, or ship it commented out? It can be confusing when common login behavior is changed. It appears that something is going wrong, since the error box still shows but is blank... It might be neat if the error box is removed entirely, and only the login box shaking stays, but I still don't see a huge security concern to inform the user about mistyped login...

@olefredrik
Copy link
Owner

@groovenectar : I agree. Fixed #350

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

No branches or pull requests

3 participants