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

Commit

Permalink
Don't put empty version string in message header
Browse files Browse the repository at this point in the history
- implemented protocol version is 4.0
- msgspec v4 doesn't include version in the header
- empty version string is never valid on any version
  • Loading branch information
minrk committed Sep 12, 2015
1 parent dacb7f5 commit 60931a9
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 3 deletions.
2 changes: 1 addition & 1 deletion Kernel/Constants.cs
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,6 @@ public class Constants
{
public const string USERNAME = "icsharp_kernel";

public const string VERSION = "";
public const string VERSION = "4.0";
}
}
3 changes: 1 addition & 2 deletions Kernel/MessageBuilder.cs
Original file line number Diff line number Diff line change
Expand Up @@ -14,8 +14,7 @@ public static Header CreateHeader(string messageType, string session)
Username = Constants.USERNAME,
Session = session,
MessageId = Guid.NewGuid().ToString(),
MessageType = messageType,
Version = Constants.VERSION
MessageType = messageType
};

return newHeader;
Expand Down

0 comments on commit 60931a9

Please sign in to comment.