Mwalker has uploaded a new change for review. Change subject: Minor updates to FundraiserUnsubscribe. ......................................................................
Minor updates to FundraiserUnsubscribe. Awight had proposed some changes to the last commit but I didn't get around to changing them before the whole thing got merged. So! - We now can control the syslog facility via config param, but by default it's set to LOG_USER - We can control if we want to log outbound XML. This removes a security concern whereby it logged the Silverpop password into the logs. - Fixed a silly IF condition in the Silverpop method. Change-Id: I987e4a0e00ecc18f64b9643a0c62a619e675b27c --- M FundraiserUnsubscribe.php M includes/Logger.php M includes/XmlTransactionProcessor.php M methods/FundraiserUnsubscribeSilverpop.php 4 files changed, 15 insertions(+), 5 deletions(-) git pull ssh://gerrit.wikimedia.org:29418/wikimedia/fundraising/DonationEmailUnsubscribe refs/changes/71/25471/1 -- To view, visit https://gerrit.wikimedia.org/r/25471 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I987e4a0e00ecc18f64b9643a0c62a619e675b27c Gerrit-PatchSet: 1 Gerrit-Project: wikimedia/fundraising/DonationEmailUnsubscribe Gerrit-Branch: master Gerrit-Owner: Mwalker <[email protected]> _______________________________________________ MediaWiki-commits mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
