Skip to content
This repository has been archived by the owner on Dec 10, 2024. It is now read-only.

[A2] Wrong attributes order #41

Open
LucBerge opened this issue Jul 10, 2023 · 1 comment
Open

[A2] Wrong attributes order #41

LucBerge opened this issue Jul 10, 2023 · 1 comment

Comments

@LucBerge
Copy link
Member

LucBerge commented Jul 10, 2023

In the file https://github.com/bot4dofus/Datafus/blob/master/data/A/DofusInvoker/scripts/com/ankamagames/dofus/network/messages/connection/IdentificationMessage.as#L119

The correct serialization order is:

  1. autoconnect
  2. useCertificate
  3. useLoginToken
  4. version
  5. lang
  6. credentials
  7. serverId
  8. sessionOptionalSalt
  9. failedAttempts

Current order is (based on attributes declaration):

  1. version
  2. lang
  3. credentials
  4. serverId
  5. autoconnect
  6. useCertificate
  7. useLoginToken
  8. sessionOptionalSalt
  9. failedAttempts

events.json content:

	"IdentificationMessage": {
		"file": "data/A/DofusInvoker/scripts/com/ankamagames/dofus/network/messages/connection/IdentificationMessage.as",
		"id": "7755",
		"superclass": "NetworkMessage",
		"interfaces": [
			"INetworkMessage"
		],
		"attributes": {
			"version": "Version",
			"lang": "String",
			"credentials": "Vector<VarInt,Byte>",
			"serverId": "Short",
			"autoconnect": "Boolean",
			"useCertificate": "Boolean",
			"useLoginToken": "Boolean",
			"sessionOptionalSalt": "VarLong",
			"failedAttempts": "Vector<Short,VarShort>"
		}
	},
@LucBerge LucBerge changed the title [A2] Wrong order [A2] Wrong attributes order Jul 10, 2023
@kkuette
Copy link

kkuette commented Oct 1, 2023

autoconnect, useCertificate, useLoginToken are wrapped at the beggining

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

No branches or pull requests

2 participants