Started by GitHub push by md-5Started by GitHub push by md-5916107d6b011db98e549b401ae56daa01d2701ad287107d6b011db98e549b401ae56daa01d2701ad287origin/master107d6b011db98e549b401ae56daa01d2701ad287107d6b011db98e549b401ae56daa01d2701ad287origin/master107d6b011db98e549b401ae56daa01d2701ad287107d6b011db98e549b401ae56daa01d2701ad287origin/mastergit://github.com/SpigotMC/BungeeCord.git0019testReportBungeeCord.jarBungeeCord.jarbootstrap/target/BungeeCord.jarcmd_alert.jarcmd_alert.jarmodule/cmd-alert/target/cmd_alert.jarcmd_find.jarcmd_find.jarmodule/cmd-find/target/cmd_find.jarcmd_list.jarcmd_list.jarmodule/cmd-list/target/cmd_list.jarcmd_send.jarcmd_send.jarmodule/cmd-send/target/cmd_send.jarcmd_server.jarcmd_server.jarmodule/cmd-server/target/cmd_server.jarreconnect_yaml.jarreconnect_yaml.jarmodule/reconnect-yaml/target/reconnect_yaml.jarfalse#916112204162405BungeeCord #9162014-06-11_02-10-07falsefalse916-1SUCCESS1402467007000https://hub.spigotmc.org/jenkins/job/BungeeCord/916/proxy/src/main/java/net/md_5/bungee/netty/PipelineUtils.javaf23691df232fcd7ae92d693a726fa1f053018ce81402430913000https://hub.spigotmc.org/jenkins/user/md_5md_5git@md-5.netReenable throttle since we only support 1.7+ 2014-06-11T06:08:33+1000f23691df232fcd7ae92d693a726fa1f053018ce8Reenable throttle since we only support 1.7+editproxy/src/main/java/net/md_5/bungee/netty/PipelineUtils.javaapi/src/main/java/net/md_5/bungee/api/ServerPing.java107d6b011db98e549b401ae56daa01d2701ad2871402430960000https://hub.spigotmc.org/jenkins/user/md_5md_5git@md-5.netFix ServerPing NPE w/ String favicons Currently, passing a null favicon String to the ServerPing(Protocol, Players, String, String) constructor causes a NPE. However, passing a null `Favicon` object to the corresponding constructor does not cause one. Setting the favicon String using the setFavicon(String) method doesn't cause a NPE either. Therefore, the NPE thrown by the constructor is inconsistent and should be avoided. Please find a sample NPE here: http://newpaste.md-5.net/pmtqjc8vl (Note the `null` favicon) This PR changes the documented (unintended?) behaviour by adding a null check before passing the favicon String to the alternative `Favicon` object constructor. This makes the constructor consistent with the other one and the `setFavicon(String)` method. This also adds compatibility for old (made before Favicon API) plugins passing `null` favicon Strings (and expecting no favicon to be displayed instead of a NPE). Thanks! 2014-06-11T06:09:20+1000107d6b011db98e549b401ae56daa01d2701ad287Fix ServerPing NPE w/ String faviconseditapi/src/main/java/net/md_5/bungee/api/ServerPing.javagithttps://hub.spigotmc.org/jenkins/user/md_5md_53.0.4