UWCS / choob

Compsoc's Highly Over-engineered Object-oriented Bot
http://choob.uwcs.co.uk/
GNU Lesser General Public License v2.1
17 stars 6 forks source link

Authentication does not work on Unreal with Anope services #334

Closed FauxFaux closed 13 years ago

FauxFaux commented 13 years ago

The new authentication code completely fails on the Unreal + Anope combination.

{{{ 1250970679229 :Silver!chatzilla@609275AB.E6BF62D4.4B50E195.IP PRIVMSG Sugnim :plugin.reload feeds 1250970679239 >>>WHOIS silver 1250970679511 :gravel.mozilla.org 311 Sugnim Silver chatzilla 609275AB.E6BF62D4.4B50E195.IP * :James Ross 1250970679511 :gravel.mozilla.org 307 Sugnim Silver :is a registered nick 1250970679511 :gravel.mozilla.org 319 Sugnim Silver :#venkman @#ChatZilla 1250970679511 :gravel.mozilla.org 312 Sugnim Silver gravel.mozilla.org :Amsterdam, Netherlands 1250970679512 :gravel.mozilla.org 317 Sugnim Silver 1 1250969817 :seconds idle, signon time 1250970679512 :gravel.mozilla.org 318 Sugnim silver :End of /WHOIS list. Sent request to auth handler 1250970679579 >>>PRIVMSG NickServ :INFO Silver 1250970680058 :NickServ!services@mozilla.org NOTICE Sugnim :Silver is James Ross 1250970680060 :NickServ!services@mozilla.org NOTICE Sugnim :Silver is currently online. 1250970680061 :NickServ!services@mozilla.org NOTICE Sugnim : Time registered: Apr 06 06:52:50 2005 PDT 1250970680063 :NickServ!services@mozilla.org NOTICE Sugnim :Last quit message: Ping timeout receiveReply Silver is James Ross Checking ^([^\s]+?) is not registered.$ Checking ^Last seen.?: now$ Checking ^Last seen.?:.* Checking ^ End of Info $ None matched, ignoring receiveReply Silver is currently online. Checking ^([^\s]+?) is not registered.$ Checking ^Last seen.?: now$ Checking ^Last seen.?:.* Checking ^ End of Info $ None matched, ignoring receiveReply Time registered: Apr 06 06:52:50 2005 PDT Checking ^([^\s]+?) is not registered.$ Checking ^Last seen.?: now$ Checking ^Last seen.?:.* Checking ^ End of Info $ None matched, ignoring receiveReply Last quit message: Ping timeout Checking ^([^\s]+?) is not registered.$ Checking ^Last seen.?: now$ Checking ^Last seen.?:.* Checking ^ End of Info $ None matched, ignoring Exception invoking method public void plugins.Plugin.Plugin.commandReload(uk.co.uwcs.choob.support.events.Message) Permission denied. Valid authentication must first be obtained. at uk.co.uwcs.choob.modules.SecurityModule.checkAuth(SecurityModule.java:571) at uk.co.uwcs.choob.modules.SecurityModule.checkNickPerm(SecurityModule.java:706) at plugins.Plugin.Plugin.commandReload(Plugin.java:157) at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39) at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25) at java.lang.reflect.Method.invoke(Method.java:597) at uk.co.uwcs.choob.plugins.HaxSunPluginManager$2.run(HaxSunPluginManager.java:432) at java.util.concurrent.ThreadPoolExecutor$Worker.runTask(ThreadPoolExecutor.java:885) at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:907) at java.lang.Thread.run(Thread.java:619) 1250970699582 >>>PRIVMSG Silver :Plugin Plugin went wrong: Permission denied. Valid authentication must first be obtained. }}}

FauxFaux commented 13 years ago

Author: benji

FauxFaux commented 13 years ago

Author: benji Added a quick fix in r1196.

Created ticket #335 to track further improvements needed.

FauxFaux commented 13 years ago

Ticket imported from Trac: http://trac.uwcs.co.uk/choob/ticket/334