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

fix xml ctypes in store_loot #10612

Merged
merged 1 commit into from Sep 10, 2018
Merged

fix xml ctypes in store_loot #10612

merged 1 commit into from Sep 10, 2018

Conversation

h00die
Copy link
Contributor

@h00die h00die commented Sep 10, 2018

A few modules save .xml files as 'text/plain' ctype, this fixes them to 'text/xml'. Most likely makes 0 difference, but it bothered me.

@h00die h00die added the bug label Sep 10, 2018
@wvu wvu self-assigned this Sep 10, 2018
@wvu
Copy link
Contributor

wvu commented Sep 10, 2018

Very good. I should have run through the GPP modules in #10507.

wvu added a commit to wvu/metasploit-framework that referenced this pull request Sep 10, 2018
@wvu wvu merged commit 39a2d9d into rapid7:master Sep 10, 2018
@wvu
Copy link
Contributor

wvu commented Sep 10, 2018

3ec4d2f

msjenkins-r7 pushed a commit that referenced this pull request Sep 10, 2018
@wvu
Copy link
Contributor

wvu commented Sep 10, 2018

Release Notes

This fix changes an incorrect text/plain content type for store_loot calls involving XML. The corrected content type is text/xml.

wvu added a commit that referenced this pull request Sep 10, 2018
Also update #10612 to align with these changes.
msjenkins-r7 pushed a commit that referenced this pull request Sep 10, 2018
Also update #10612 to align with these changes.
@tdoan-r7 tdoan-r7 added the rn-fix release notes fix label Sep 26, 2018
@h00die h00die deleted the xml branch November 15, 2018 22:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants