pnopjp / jmeter-plugins

Apache JMeter plugins for Microsoft Azure
Apache License 2.0
17 stars 13 forks source link

Adding header info with Evenhub #12

Open tmsrikanth opened 1 year ago

tmsrikanth commented 1 year ago

Hi,

I need to added header info as part of the eventhub message. can you let me know , how it can be added in this plugin

kuniteru commented 1 year ago

Hi tmsrikanth,

By eventhub message header info you are referring to, do you mean content-type, correlationId, messageId, etc.? If so, unfortunately this Event Hubs plugin does not support sending them.

If not them, please tell me what the eventhub message header information indicates.

tmsrikanth commented 1 year ago

Hi, Extra information needs to be passed as part of Eventhub message. This can be passed as a property . Instead of Property, i referred it as header.
Is it possible to add this feature . Similar implementation is done in service bus plugin. Pls do let me know.

Regards, Srikanth

kuniteru commented 1 year ago

Hi,

I have made some modifications to branch AEH0.3.0 to the extent that I can do so immediately.

I haven't done enough testing because I don't have an environment to retrieve properties, but does this meet your requirements?

tmsrikanth commented 1 year ago

Thanks , I tried to create a jar out of the branch and added it into /lib/ext . But unfortunately i am unable to see the sampler listed in the jmeter. Am i missing any steps. I am new to java , hence the question

tmsrikanth commented 1 year ago

Generated the Jar .. Testing it now. will revert on the results

kuniteru commented 1 year ago

I know it's late, but I wrote down the build steps. https://github.com/pnopjp/jmeter-plugins/blob/master/README.md#how-to-build I hope it will be helpful to you.

tmsrikanth commented 1 year ago

Appreciate it Kuniteru,

Json conversation for the property value provided error in processing in the function. I modified the code with hardcoded value of the property using eventData.getProperties().put, and it worked.

tmsrikanth commented 1 year ago

I also have a requirement on sending messages in bytes format . I will try to make the addition to the code.

kuniteru commented 1 year ago

I am glad that your modifications allowed us to meet your requirements. If you don't mind, please let me know what JSON you gave and how you modified it.

I look forward to your contribute on sending byte format messages.

tmsrikanth commented 1 year ago

I am not able to push my branch . Do i need permission.

kuniteru commented 1 year ago

Following the github convention, please fork first, modify the forked code, and then Pull Request it.

I'm not familiar with any reference documents written in English, but how about this content, for example? https://gist.github.com/siddhantprateek/342046de702fd65a381c9991edc45cd5 https://reflectoring.io/github-fork-and-pull/

srikanth-tm commented 1 year ago

Created a pull request .Pls Look into it.