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

[QUERY] SharedAccessSignature not working for Event Hubs with Java SDK v5.1.2. #14595

Closed
3 tasks done
mattbalgeman opened this issue Aug 28, 2020 · 4 comments
Closed
3 tasks done
Assignees
Labels
Client This issue points to a problem in the data-plane of the library. customer-reported Issues that are reported by GitHub users external to the Azure organization. Event Hubs feature-request This issue requires a new behavior in the product in order be resolved. Service Bus

Comments

@mattbalgeman
Copy link

Describe the bug
SharedAccessSignatures aren't working for event hubs for v5.1.2 of the Java SDK.

Exception or Stack Trace
java.lang.IllegalArgumentException: Illegal connection string parameter name: SharedAccessSignature

To Reproduce
Generate a SharedAccessSignature for an event hub and attempt to use that Signature in the ConnectionString.
In previous versions of the SDK, that connection string would look something like this:
Endpoint=sb://<namespacename>.servicebus.windows.net;EntityPath=<eventhubname>;SharedAccessSignature=SharedAccessSignature sr=sb%3A%2F%2F<namespacename>.servicebus.windows.net%2F<eventhubname>&sig=IciALS180<somecharsremoved>&se=159<somenumbersremoved>&skn=SendRule

In the new version of the SDK, ConnectionStringProperties doesn't except the SharedAccessSignature option and throws an exception. I was unable to find any other option that supports SharedAccessSignatures.

Code Snippet
Add the code snippet that causes the issue.

Expected behavior
I would expect the new SDK to have feature parity with the old version and support SharedAccessSignatures. Also, it appears that ConnectionStringBuilder was removed?

Screenshots
If applicable, add screenshots to help explain your problem.

Setup (please complete the following information):

  • IDE : IntelliJ
  • Version of the Library used
com.azure azure-messaging-eventhubs 5.1.2

Additional context
Add any other context about the problem here.

Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

  • Bug Description Added
  • Repro Steps Added
  • Setup information Added
@ghost ghost added needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. customer-reported Issues that are reported by GitHub users external to the Azure organization. question The issue doesn't require a change to the product in order to be resolved. Most issues start as that labels Aug 28, 2020
@joshfree joshfree added Client This issue points to a problem in the data-plane of the library. Event Hubs labels Aug 31, 2020
@ghost ghost removed the needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. label Aug 31, 2020
@joshfree
Copy link
Member

Thanks for filing this github issue @mattbalgeman. @conniey could you please follow up?

@conniey
Copy link
Member

conniey commented Aug 31, 2020

@mattbalgeman Thanks for filing this issue. This is not currently supported in Event Hubs or Service Bus... We do have an open Azure SDK issue to address this:
Azure/azure-sdk#1701

Parent Discussion: Azure/azure-sdk#1701
Related: Azure/azure-sdk-for-js#10851

@conniey conniey added feature-request This issue requires a new behavior in the product in order be resolved. Service Bus and removed question The issue doesn't require a change to the product in order to be resolved. Most issues start as that labels Aug 31, 2020
@mattbalgeman
Copy link
Author

Thanks for the information. Unfortunately, this means that will be need to stick with the old version of the Java SDK until this issue is resolved. We'll watch the referenced discussion for further information.

@srnagar srnagar self-assigned this Aug 31, 2020
@srnagar
Copy link
Member

srnagar commented Sep 14, 2020

@mattbalgeman Support for including SharedAccessSignature in connection string was added in the latest Event Hubs release. Please try the new released version:

azure-messaging-eventhubs - 5.2.0

@srnagar srnagar closed this as completed Sep 14, 2020
@github-actions github-actions bot locked and limited conversation to collaborators Apr 12, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Client This issue points to a problem in the data-plane of the library. customer-reported Issues that are reported by GitHub users external to the Azure organization. Event Hubs feature-request This issue requires a new behavior in the product in order be resolved. Service Bus
Projects
None yet
Development

No branches or pull requests

4 participants