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.

Sign up for GitHub

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

Jump to bottom

Updated ipmi_users.txt #11612

Merged
merged 1 commit into from Mar 22, 2019
Merged

Updated ipmi_users.txt #11612

merged 1 commit into from Mar 22, 2019

Conversation

bandrel
Copy link
Contributor

@bandrel bandrel commented Mar 22, 2019

Added Admin to ipmi_users.txt

Fix for #11611

@bandrel
Updated ipmi_users.txt
c98ed4b 
Added username 'Admin'
@bcoles bcoles added the easy label Mar 22, 2019
bcoles
bcoles approved these changes Mar 22, 2019
View reviewed changes
Copy link
Contributor

@bcoles bcoles left a comment

Choose a reason for hiding this comment

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

looks legit

@wvu wvu self-assigned this Mar 22, 2019
wvu added a commit to wvu/metasploit-framework that referenced this pull request Mar 22, 2019
@wvu
Land rapid7#11612, ipmi_users.txt update
1c34926
@wvu wvu merged commit c98ed4b into rapid7:master Mar 22, 2019
@wvu
Copy link
Contributor

wvu commented Mar 22, 2019

Release Notes

This adds the Admin user to the ipmi_users.txt word list.

@bandrel bandrel deleted the missing_ipmi_user branch March 22, 2019 16:07
msjenkins-r7 pushed a commit that referenced this pull request Mar 22, 2019
@wvu @msjenkins-r7
Land #11612, ipmi_users.txt update
197ad6c
@gdavidson-r7 gdavidson-r7 added the rn-enhancement release notes enhancement label Apr 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bcoles bcoles bcoles approved these changes

@wvu wvu wvu approved these changes

Assignees

@wvu wvu

Labels
easy rn-enhancement release notes enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@bandrel @wvu @bcoles @gdavidson-r7