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

Staff channel messages are limited to 299 characters (!pl is a good example) #37

Open
WizardCM opened this issue Apr 21, 2013 · 4 comments
Labels
Milestone

Comments

@WizardCM
Copy link
Member

In minecraft-staff, the response from a command is limited to 299 characters, as shown below.

Should be;

Plugins (68): W4, Privileges, MiniFixes, Memory, CommandViewer, WMABroadcast, RoyalDeath, Vault, Multiverse-Core, uSkyBlock, DevJoined, dynmap, BukkitIRCd, SurvivalGames, SpongeRecipe, Duties, TNTSweeper, ProtocolLib, ConnectFour, MobArena, CleanroomGenerator, Plotty, LagMeter, ZeeSlag, Shortify, WorldEdit, NoCheatPlus, Multiverse-Inventories, CreativeGates, MultiMapsFix, TagAPI, Multiverse-NetherPortals, VanishNoPacket, DeathControl, MCdigr, War, StandardBooks, NCore, Paintball, WorldGuard, TF2, Tetris, SignEdit, ImgMap, MineJong, NBTEdit, UltimateSkyGrid, MonsterMOTD, CommandHelper, AnimalProtect, AntiXRay, The Trashcan, Lockette, NoNewChunk, FirstJoinPlus, ButtonWarp, MobHealth, WorldBorder, Multiverse-Portals, ScriptBlock, Dynmap-WorldGuard, DisguiseCraft, NTheEndAgain, HawkEye, RoyalCommands, ChessCraft, NameTags, LimitedWorldEdit, RoyalChat

Comes through;

Plugins (68): W4, Privileges, MiniFixes, Memory, CommandViewer, WMABroadcast, RoyalDeath, Vault, Multiverse-Core, uSkyBlock, DevJoined, dynmap, BukkitIRCd, SurvivalGames, SpongeRecipe, Duties, TNTSweeper, ProtocolLib, ConnectFour, MobArena, CleanroomGenerator, Plotty, LagMeter, ZeeSlag, Shortify,

@chyyran
Copy link
Contributor

chyyran commented Apr 21, 2013

This might be a problem with IRC rather than the plugin. Not 100% sure though

@WizardCM
Copy link
Member Author

If that turns out to be the case, there are two things we could do here;
A) detect the length then split the message so the whole thing is sent
Or
B) if the whole message doesn't come through, the next message should say something like "The response to your command was longer than we could send. Please try this ingame or from console instead." In red.

@chyyran
Copy link
Contributor

chyyran commented Apr 22, 2013

According to http://www.networksorcery.com/enp/protocol/irc.htm , the max message length for IRC is 512 characters including leading/trailing CRLF, and the protocol message (PRIVMSG and all that). So I think it might be a problem of the IRC protocol

@WizardCM
Copy link
Member Author

WizardCM commented Sep 4, 2014

This has been (at least partly) improved in ttps://github.com/WMCAlliance/BukkitIRCd/commit/43700b3d3a5e4a6c7b166f3e8c32ec41a083f69c - the messages are now split when sent to avoid loss of information. I don't know if colour codes are counted in the IRC protocol (I haven't checked) but in my current method - they are.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants