FactionChat

FactionChat

125k Downloads

Authme compatibility

MithrandirCraft opened this issue ยท 34 comments

commented

Hi! Could you please take a look at this?:

https://github.com/Xephi/AuthMeReloaded/issues/259

Thanks!


Current authme version: 5.2-SNAPSHOT-b660
Faction chat version: 1.9.12-SNAPSHOT
Current Minecraft-Spigot version: 1.8.8

commented

Thanks MithrandirCraft.

Personally I've tried in the past to removed staff chat as it has nothing to do with Factions but I had too many PMs when I removed it.

i'll see if I can quickly fix it now.

commented

Fixed

commented

ok It might not be fixed. looking at some of the Hastbin there

commented

ok Seems my previous changes would have fixed that error too (http://hastebin.com/imoxiwajem.avrasm)

Going to recompile the plugin using

http://dev.bukkit.org/bukkit-plugins/authme-reloaded/files/75-auth-me-5-0-beta3/

commented

it's a maven repo, you can add it to the pom file without putting the jar into the libs folder ;)

commented

Thanks. :D

commented

I see

fr.xephi.authme.api.API.isAuthenticated(player) is now @deprecated

What would you suggest I use?

commented

never mind (fr.xephi.authme.api.NewAPI.)

commented

ok that's all done. it "should" work find now

commented

Wait a sec, this isn't fixed. I can still read the moderator chat when logged out.

commented

Yeah but if I remember well you check if the allowchat option is true or false, can you try to remove this check?

commented

:-(

commented

Well I have:

if ((myplayer.hasPermission("FactionChat.ModChat") || FactionChat.isDebugger(myplayer.getName())) && AuthMeAPI.isLoggedIn(player))

with fr.xephi.authme.api.NewAPI.getInstance().isAuthenticated(player); to check if logged in

The AuthMe stuff is only enabled if this.getServer().getPluginManager().getPlugin("AuthMe") != null

Could you check this?

commented

Can I use this build? :3

commented

yes, bear in mind its a Development build

commented

thank you <3 I'll report back.

commented

Well... I can still read messages when logged out.

commented

Updated affected versions.

commented

Registred yet logout?
Well that's just what I tried, and I could see myself spamming moderator chat with another account while logged out with an OP account. (Maybe I could see it because I'm OP?)

In any case, I'm trying your last build, will report asap.

commented

make sure it is registered too (altho the new changes should prevent that too)

commented

Ok I tested it myself with only AuthMe, Factions (and massiveCore) and FactionChat.

Unregistered could see ModChat
Registered yet logout could not see ModChat.

Going to fix this now

commented

screenshot_1
screenshot_2

Still can read from outside u_u

commented

this is my /fc mod right?

commented

Those results are using the last build you just gave me.

commented

ok with my test it is blocking correctly. I'll make another build that prints in the console some information. Could you paste it here (when it is being bypassed)?

commented
commented

Hey James, Xephii passed me a build with a pull request he did on FactionChat.

It works fine with it :) So as long as you apply it, the issue should be fixed.

commented

ok pulled Thanks @Xephi

commented

Closed and fixed.
Thanks @MithrandirCraft , @sgdc3, and @Xephi

and everyone from
https://github.com/Xephi/AuthMeReloaded/issues/259

commented

uploaded to dev.bukkit