issues: 210217148
This data as json
id | node_id | number | title | user | state | locked | assignee | milestone | comments | created_at | updated_at | closed_at | author_association | active_lock_reason | draft | pull_request | body | reactions | performed_via_github_app | state_reason | repo | type |
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
210217148 | MDExOlB1bGxSZXF1ZXN0MTA3OTUwNDg4 | 90 | Rewrote network stack | 12808515 | closed | 0 | 3 | 2017-02-25T07:06:09Z | 2017-02-27T11:28:37Z | 2017-02-27T11:28:37Z | CONTRIBUTOR | 0 | ACEmulator/ACE/pulls/90 | Rewrote network stack to support buffering messages and then dispatching as 1 or more combined or split packets. EncryptedChecksum seems to work across the board. Sending Packets with fragments from different groups does not seem to cause an issue. Packets are queued and sent based on calls from the Engine. Configurable delay (5ms currently) Reworked CachedPackets some, and now clear the list based on AckSeq packets from client. Sending AckSeq packets to the client seems to cause issues currently, need more testing. Moved some things around to fit this new model. Put message Group as a property on each message. And send this in fragment. Sounds now work again. Created new packet classes to encapsulate data in AuthenticationManager and CharacterManager and allow them to be send through the same model (though packets will never be manipulated). | {"url": "https://api.github.com/repos/ACEmulator/ACE/issues/90/reactions", "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0} | 79078680 | pull |