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.

Sign up for GitHub

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

Jump to bottom

Convert AMQP 1.0 props and app props to AMQP 0.9.1 props and headers #10037

Merged
merged 1 commit into from
Dec 4, 2023
Merged

Convert AMQP 1.0 props and app props to AMQP 0.9.1 props and headers #10037

merged 1 commit into from
Dec 4, 2023

Commits on Dec 4, 2023

  1. Convert AMQP 1.0 props and app props to AMQP 0.9.1 props and headers 

    - Timestamps are milliseconds in AMQP 1.0, but in AMQP 0.9.1 it is seconds.
  Fixed by multiplying the timestamp by 1 000.
- Shovel crashed if user_id was set in the message because the encoding
  was as utf8 while it should be a byte array.
- Negative integers were encoded as integers - therefore leading to
  incorrect positive values.
- Float values were not supported by the client.
- Fixed priority header encoding in AMQP 1.0. It was set as uint but it
  should be ubyte.
- Priority of the message is now in the Headers instead of Application
  Properties. This is potentially a breaking change.

Fixes: rabbitmq#7508
    @olikasg @luos
    olikasg authored and luos committed Dec 4, 2023
    Configuration menu
    Copy the full SHA
    8e954ff View commit details
    Browse the repository at this point in the history