Merge changes for v1.9.0 BETA #132
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Import changes from master into Indev (v1.8.2)
Bug report by doranchak (forum post no 1136)
Mono Framework does not handle CFB-8 AES encryption mode. So now MCC will now use borrowed code from the BouncyCastle project for handling AES when running on Mono framework, instead of using a dirty workaround to try getting Mono encryption working. Regular .NET framework encryption module will still be used when not running under Mono (eg on Windows or using Wine) Should hopefully fix all the issues encountered on Mono including #41 and finally achieve full compatibility of MCC with Mac and Linux.
Avoid crashing due to bots not properly processing text
- Save & Restore background color when needed - Remove useless color modifications - Fix issue #71
Might help when login/session servers take a long time to respond.
- Add prompt for Minecraft version - Improve offline-mode command prompt - Fix default value on parse error in protocol handler - Fix failed to connect not showing offline prompt
Unix-Tool-Like syntax : user@host Window title can be changed or disabled in INI file
- Better catch in proxy handler - Better catch in StartClient (thx doranchak)
Bug report by doranchak
Thanks doranchak & FantomHD (post no.1193) + Add missing doc for MinecraftVersionPrompt
- Pass minecraft login failure message to AutoRelog bot (suggestion by doranchak) - Fix NullReferenceException in McTcpClient caused by SocketException in ProxyHandler - Refactor error handling code in Program.InitializeClient() - More detailed error messages on network errors.
Add config for automated builds
Build can be configured directly on AppVeyor.
- Merge all error handling code into one method - Fix ConsoleIO not clearing the line being typed upon reset - Update console title upon logging in to the server - Pass "failed to ping this IP" to AutoRelog (thx doranchak)
"Failed to ping this IP" also caused "Failed to determine server version" error, calling HandleFailure() twice.
Fix concurrency issues by using proper locks
Indentation was intentionally left the same in previous commit for clearer diff, this commit only fixes code indentation from last commit
+ minor fixes to ConsoleIO
If the server is sending invalid UUIDs, use an empty UUID instead
Improve speed and handling of %1$s tags Fix prompt in ConsoleIO not being reset to gray
See issue #63 - Also includes minor fixes and optimizations
This bot allows users to add a bot that can detect and respond to certain text. The bot can be enabled/disabled via the ini file. (disabled by default) The bot uses 2 files to let the user set what to pickup and what to respond.
Add AutoRespond Bot
When a very long message is typed, a delay of 2 seconds is by default used before sending parts of the long messages. The delay can be modified or set back to 0 in configuration file, if necessary.
Move FromFile method from bots to ChatBot class Rename file and class, removing space and underscore.
See issue #77
Player list updates on MC 1.8 handler did not take into account the amount of items in the list and were only processing the first item, including when there wasn't any item to process. Unfortunately some weird servers were sending useless empty tab-list updates, causing a crash. Should fix issue #78 and forum posts 1267, 1269, 1284. Thanks dbear20, link3321, gerik43, Darkaegis, k3ldon and Ryan6578 for their bug reports! :)
Give bots access to plugin channels
Fix IndexOutOfRangeException when bot is unloaded on update
The first bug is that the list isn't cleared in some cases, meaning new packets get data from the previous packet if it isn't read fully. Most commonly, this happens with a plugin channel message. The second bug happens because lists don't throw IndexOutOfRangeExceptions, but instead throw ArgumentOutOfRangeExceptions. This caused the catch for ignoring message types to not occur, instead causing the client to crash. This only happens in 1.7.10, where the message type is not included. Most likely, these changes will fix the bugs seen in #114 and #117, although they may be caused by other bugs.
Fix several bugs
In an attempt at fixing #117
… difficult. invaliding the token in a way that it can be refreshed is not documented.
Updated project file to include Cache namespace.
Add session caching by initsuj
Changing constructs that weren't .NET 4.0 compliant. Also fix \n to \r\n line returns in ProtocolHandler.cs
Currently no terrain handling but anything else should work. Related: #125
Went unnoticed until, well, now.
- Memory session cache will be used by default - Verbose messages are now hidden by default - Improve ping procedure's Forge handling - Fix 1.8.9 and 1.9.0 in mcversion setting - SessionCache.cs: fix LF into CRLF - Use 1.9 translations instead of 1.8
Suggested in #129
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
A whole year of commits! With amazing contributions by wonderful contributors! :)