Skip to content

Protect: Backup to ensure the user object is populated#6892

Merged
samhotchkiss merged 1 commit intomasterfrom
fix/6890
Apr 11, 2017
Merged

Protect: Backup to ensure the user object is populated#6892
samhotchkiss merged 1 commit intomasterfrom
fix/6890

Conversation

@kraftbj
Copy link
Copy Markdown
Contributor

@kraftbj kraftbj commented Apr 4, 2017

Fixes #6890

If a do_action( 'wp_login' ) call is made without the second argument, let's figure out what the user object should be. This defensively codes against an warning we've seen crop up a couple of times.

@kraftbj kraftbj added [Feature] Protect Also known as Brute Force Attack Protection [Pri] Low [Status] Needs Review This PR is ready for review. Bug When a feature is broken and / or not performing as intended labels Apr 4, 2017
@kraftbj kraftbj requested a review from samhotchkiss April 4, 2017 14:47
Copy link
Copy Markdown
Member

@thingalon thingalon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks reasonable to me :)

@kraftbj kraftbj added [Status] Ready to Merge Go ahead, you can push that green button! and removed [Status] Needs Review This PR is ready for review. labels Apr 10, 2017
@samhotchkiss samhotchkiss merged commit f5d323e into master Apr 11, 2017
@samhotchkiss samhotchkiss deleted the fix/6890 branch April 11, 2017 03:52
@samhotchkiss samhotchkiss removed the [Status] Ready to Merge Go ahead, you can push that green button! label Apr 11, 2017
jeherve added a commit that referenced this pull request Apr 24, 2017
eliorivero pushed a commit that referenced this pull request Apr 25, 2017
* Changelog: initial commit for 4.9 release.

* Changelog: add #6929

* Changelog: move old changelogs to changelog.txt

* Readme: restore deleted release post link.

The post is now live.

* Changelog: add #6853

* Changelog: add #6856

* Changelog: add #6857

* Changelog: add #6884

* Changelog: add #6885

* Changelog: add #6892

* Changelog: add #6894

* Changelog: add #6898

* Changelog: add #6899

* Changelog: add #6900

* Changelog: add #6909

* Changelog: add #6927

* Changelog: add #6947

* Chagelog: add #6958

* Changelog: add #6961

* Changelog: add #6963

* Changelog: add #6965

* Changelog: add #6986

* Changelog: add #7000

* Changelog: add #7013

* Changelog: add #7015

* Changelog: add #7019

* Changelog: add #7028

* Changelog: add #6998

* Changelog: add #6999

* Changelog: add #7044

* Changelog: add #6881

* Changelog: add #6922

* Changelog: add #6940

* Changelog: add #6962

* Changelog: add #6942

* Changelog: add #6959

* Changelog: add #7018

* Changelog: add #6948

* Changelog: add #6657

* Changelog: add #7030

* Changelog: add #7048

* Changelog: add #7031

* Changelog: add #6990

* Changelog: add #6957

* Changelog: add #7027
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug When a feature is broken and / or not performing as intended [Feature] Protect Also known as Brute Force Attack Protection [Pri] Low

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants