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

Azure Event Hubs Readme and Sample Issue #30555

Open
Yionse opened this issue Jul 29, 2024 · 10 comments
Open

Azure Event Hubs Readme and Sample Issue #30555

Yionse opened this issue Jul 29, 2024 · 10 comments
Assignees
Labels
Client This issue points to a problem in the data-plane of the library. Event Hubs test-manual-pass

Comments

@Yionse
Copy link
Member

Yionse commented Jul 29, 2024

link1
image
link2
image
link3
image
link4
image
link5
image

Reason:

  1. When running link1, link2, and link3 code. After running, there was no response. But when I no longer use the @Azure/identity verification in the sample, but instead use connection verification, it can run successfully.
  2. When running link4 code. AbortController is undefined.
  3. When packing link5 code. Webpack 5 no longer automatically includes polyfills for the Node.js core module.

@joheredi,@jsquire,@deyaaeldeen for notification.

@github-actions github-actions bot added 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 needs-team-attention Workflow: This issue needs attention from Azure service team or SDK team question The issue doesn't require a change to the product in order to be resolved. Most issues start as that Service Attention Workflow: This issue is responsible by Azure service team. labels Jul 29, 2024
Copy link

Thanks for the feedback! We are routing this to the appropriate team for follow-up. cc @kasun04.

@v-xuto v-xuto added needs-team-triage Workflow: This issue needs the team to triage. test-manual-pass and removed question The issue doesn't require a change to the product in order to be resolved. Most issues start as that customer-reported Issues that are reported by GitHub users external to the Azure organization. Service Attention Workflow: This issue is responsible by Azure service team. needs-team-attention Workflow: This issue needs attention from Azure service team or SDK team labels Jul 29, 2024
@jeremymeng jeremymeng removed the needs-team-triage Workflow: This issue needs the team to triage. label Jul 29, 2024
@deyaaeldeen
Copy link
Member

@Yionse for the first three issues, did you make sure to configure service principal correctly following the instructions in https://github.com/Azure/azure-sdk-for-js/blob/main/sdk/eventhub/event-hubs/README.md#use-the-event-hubs-namespace-and-azure-identity? It would be helpful to share what you see in the logs, especially from @azure/identity.

For the forth one, the fix is to delete AbortController from the import statement because it is a global provided by NodeJS and the web API.

I'll take a look at the fifth one.

deyaaeldeen added a commit that referenced this issue Jul 31, 2024
@Yionse
Copy link
Member Author

Yionse commented Aug 1, 2024

@deyaaeldeen thank you for your reply. For the first three issues. I'm sure I configured it according to the instructions in the document.

@deyaaeldeen
Copy link
Member

deyaaeldeen commented Aug 1, 2024

@Yionse thanks for confirming! Could you share the logs then so we can better diagnose the issue you are facing?

@Yionse
Copy link
Member Author

Yionse commented Aug 2, 2024

@Yionse thanks for confirming! Could you share the logs then so we can better diagnose the issue you are facing?

@deyaaeldeen when using '@azure/identity', an error is reported. if 'ConnectString' is used, it will run successfully. The value of fullQualifiedNamespace is https://<eventhubname>.services.windows.net. Below is the running logs.
image

@deyaaeldeen
Copy link
Member

@Yionse The fully qualified namespace shouldn’t have the eventhub name in it, that is why you’re getting a not found error. It should have the namespace instead. I would encourage you to use the existing test resources arm template to create a resource and use the output variable EVENTHUB_FQDN as the environment variable for the sample.

@Yionse
Copy link
Member Author

Yionse commented Aug 5, 2024

@Yionse The fully qualified namespace shouldn’t have the eventhub name in it, that is why you’re getting a not found error. It should have the namespace instead. I would encourage you to use the existing test resources arm template to create a resource and use the output variable EVENTHUB_FQDN as the environment variable for the sample.

@deyaaeldeen The value of fullQualifiedNamespace is <namespace>.services.windows.net. I have now received another error. Below is the running logs.
image

@deyaaeldeen
Copy link
Member

@Yionse this error indicates there is a problem with authentication and I can't know what exactly is the issue without looking at the logs. Could you share the logs?

@Yionse
Copy link
Member Author

Yionse commented Aug 7, 2024

@Yionse this error indicates there is a problem with authentication and I can't know what exactly is the issue without looking at the logs. Could you share the logs?

I have added the Azure Event Hubs Data Owner permission for the resource in the portal, and it can now run normally. Also, how is the progress status of item5?

@Yionse
Copy link
Member Author

Yionse commented Sep 27, 2024

@deyaaeldeen Hi, may I ask about the progress of item5?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Client This issue points to a problem in the data-plane of the library. Event Hubs test-manual-pass
Projects
None yet
Development

No branches or pull requests

4 participants