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

Custom Publish Headers #30

Closed
wants to merge 3 commits into from
Closed

Custom Publish Headers #30

wants to merge 3 commits into from

Conversation

jonnii
Copy link
Owner

@jonnii jonnii commented Oct 15, 2013

Messages can now be adorned with an IHeaders interface which gives each
message you publish access to the headers collection on the default
properties.

Todo

  • Make this a publisher configuration too, so that you don't need to mark up your objects with interfaces.

This will close #28

/cc @tanir @cocowalla

@jonnii
Copy link
Owner Author

jonnii commented Nov 2, 2013

@tanir is this something you're still interested in?

@tanir
Copy link

tanir commented Nov 5, 2013

Yes I am interesting in adding some feature, that allows to set a custom header when publish a message.

@jonnii
Copy link
Owner Author

jonnii commented Nov 6, 2013

@tanir, take a look at the change in this PR, is this the kind of thing you're thinking?

Messages can now be adorned with an IHeaders interface which gives each
message you publish access to the headers collection on the default
properties.
@jonnii
Copy link
Owner Author

jonnii commented Nov 6, 2013

@tanir this is ready to go. you can add an interface to a message to get custom headers, or supply a publisher strategy for setting headers. Let me know if this is what you had in mind and i'll merge/release it.

@jonnii jonnii assigned jonnii and unassigned jonnii Feb 19, 2016
@@ -2,6 +2,7 @@

# Pending

* Custom headers on publish
Copy link
Owner Author

Choose a reason for hiding this comment

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

wat

@jonnii jonnii closed this May 11, 2020
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.

create custom headers when publish
2 participants