Skip to content

Propagating Content-Type property from transport to Core#1666

Merged
afprtclr merged 2 commits into
masterfrom
feature/supporting-basic-properties
Oct 17, 2025
Merged

Propagating Content-Type property from transport to Core#1666
afprtclr merged 2 commits into
masterfrom
feature/supporting-basic-properties

Conversation

@afprtclr

@afprtclr afprtclr commented Oct 14, 2025

Copy link
Copy Markdown
Contributor

For #1664

@afprtclr afprtclr force-pushed the feature/supporting-basic-properties branch from 29d4bbb to 9cb6901 Compare October 14, 2025 09:30
Comment thread src/NServiceBus.Transport.RabbitMQ.Tests/MessageConverterTests.cs Outdated
@afprtclr afprtclr requested a review from bording October 14, 2025 14:08
@afprtclr afprtclr marked this pull request as ready for review October 15, 2025 07:35
Comment thread src/NServiceBus.Transport.RabbitMQ/Configuration/PropertiesToHeaderMapping.cs Outdated
Comment thread src/NServiceBus.Transport.RabbitMQ/Receiving/MessageConverter.cs Outdated
@afprtclr afprtclr force-pushed the feature/supporting-basic-properties branch from ee37a8c to dc395c3 Compare October 16, 2025 06:57

@bording bording left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes look good, title of the PR still needs to be updated

@afprtclr afprtclr changed the title Handling Content-Type and other properties Propagating Content-Type property from transport to Core Oct 16, 2025
@afprtclr afprtclr merged commit 67a77d0 into master Oct 17, 2025
4 checks passed
@afprtclr afprtclr deleted the feature/supporting-basic-properties branch October 17, 2025 08:36
afprtclr added a commit that referenced this pull request Oct 17, 2025
* Handling Content-Type and other properties

* Leaving Content-Type only

(cherry picked from commit 67a77d0)
afprtclr added a commit that referenced this pull request Oct 17, 2025
* Handling Content-Type and other properties

* Leaving Content-Type only

(cherry picked from commit 67a77d0)

# Conflicts:
#	src/NServiceBus.Transport.RabbitMQ.Tests/MessageConverterTests.cs
afprtclr added a commit that referenced this pull request Oct 17, 2025
* Handling Content-Type and other properties

* Leaving Content-Type only

(cherry picked from commit 67a77d0)

# Conflicts:
#	src/NServiceBus.Transport.RabbitMQ.Tests/MessageConverterTests.cs
(cherry picked from commit 26fabfb)

# Conflicts:
#	src/NServiceBus.Transport.RabbitMQ/Receiving/MessageConverter.cs
afprtclr added a commit that referenced this pull request Oct 17, 2025
* Handling Content-Type and other properties

* Leaving Content-Type only

(cherry picked from commit 67a77d0)
afprtclr added a commit that referenced this pull request Oct 17, 2025
* Handling Content-Type and other properties

* Leaving Content-Type only

(cherry picked from commit 67a77d0)

# Conflicts:
#	src/NServiceBus.Transport.RabbitMQ.Tests/MessageConverterTests.cs
afprtclr added a commit that referenced this pull request Oct 17, 2025
* Handling Content-Type and other properties

* Leaving Content-Type only

(cherry picked from commit 67a77d0)

# Conflicts:
#	src/NServiceBus.Transport.RabbitMQ.Tests/MessageConverterTests.cs
(cherry picked from commit 26fabfb)

# Conflicts:
#	src/NServiceBus.Transport.RabbitMQ/Receiving/MessageConverter.cs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants