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

Payload format and content type #825

Closed
5 tasks done
Tracked by #695
andsel opened this issue Apr 19, 2024 · 1 comment
Closed
5 tasks done
Tracked by #695

Payload format and content type #825

andsel opened this issue Apr 19, 2024 · 1 comment
Assignees
Labels

Comments

@andsel
Copy link
Collaborator

andsel commented Apr 19, 2024

Allow the payload format (binary, text) and a MIME style content type to be specified when a message is published. These are forwarded on to the receiver of the message.

  • handle payload format indicator property Payload format verification and content type properties handling #826
    • write a test to verify payload format indicator property is forwarded as it is by the broker [MQTT-3.3.2-4]
    • if the payload format indicator verify the payload is UTF-8, if not, send PUBACK or PUBREC with 0x99 reason code (Payload format invalid)
  • verify Content type is UTF-8 and close the connection if not (it's a protocol error)
    • test the content-type is forwarded as opaque
    • [ ] test that if content-type is not in UTF-8 then the connection is dropped (This is hard to write a test because all MQTT client accepts a String, and also setting a bad byte sequence to a String, when it's created from raw bytes, that string quotes those chars transforming in ? chars, loosing the original byte sequence in phase of sending a publish)
@andsel andsel changed the title Payload format and content type: Allow the payload format (binary, text) and a MIME style content type to be specified when a message is published. These are forwarded on to the receiver of the message. Payload format and content type Apr 19, 2024
@andsel andsel self-assigned this Apr 19, 2024
@andsel andsel added the mqtt5 label Apr 19, 2024
@andsel
Copy link
Collaborator Author

andsel commented May 25, 2024

Closed by#826

@andsel andsel closed this as completed May 25, 2024
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

1 participant