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

Provide support for SAS token auth in Azure Event Hubs Scaler #2705

Open
yaronpri opened this issue Mar 2, 2022 · 16 comments
Open

Provide support for SAS token auth in Azure Event Hubs Scaler #2705

yaronpri opened this issue Mar 2, 2022 · 16 comments
Labels
azure All issues concerning integration with Azure feature All issues for new features that have been committed to

Comments

@yaronpri
Copy link

yaronpri commented Mar 2, 2022

Report

Based on current documentation https://keda.sh/docs/2.6/scalers/azure-event-hub/

Expected Behavior

To have ability to use SAS token

Actual Behavior

Get only connection string

Steps to Reproduce the Problem

  1. Setup EventHub scaler
  2. Try to use SAS token for the EventHub or for Checkpoint blob storage
  3. Only receive connection string

Logs from KEDA operator

No need

KEDA Version

2.5.0

Kubernetes Version

No response

Platform

No response

Scaler Details

No response

Anything else?

No response

@yaronpri yaronpri added the bug Something isn't working label Mar 2, 2022
@yaronpri yaronpri changed the title No support for SAS token in EventHub Scaler and Checkpoint Storage No support for SAS token in EventHub Scaler Mar 2, 2022
@tomkerkhove tomkerkhove added azure All issues concerning integration with Azure feature-request All issues for new features that have not been committed to and removed bug Something isn't working labels Mar 2, 2022
@tomkerkhove
Copy link
Member

Fair request, adding to backlog. Are you willing to contribute this?

@yaronpri
Copy link
Author

yaronpri commented Mar 5, 2022

For sure, I will try to find some time for this. Thx

@jiujiujiujiujiuaia
Copy link

@yaronpri Do you any branch/commit for proving this feature? Would love to help with you

@tomkerkhove
Copy link
Member

@v-shenoy If you are bored, is this something you want to pick up?

@v-shenoy
Copy link
Contributor

@tomkerkhove I can take them (both event hub and service bus) up next week.

@tomkerkhove
Copy link
Member

That would be awesome, but don't feel pressured!

@sushmithavangala
Copy link
Contributor

@tomkerkhove I Can take up this parallelly with #2920

@stale
Copy link

stale bot commented Jun 26, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 7 days if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale All issues that are marked as stale due to inactivity label Jun 26, 2022
@tomkerkhove
Copy link
Member

Bump

@stale stale bot removed the stale All issues that are marked as stale due to inactivity label Jun 27, 2022
@tomkerkhove tomkerkhove changed the title No support for SAS token in EventHub Scaler Provide support for SAS token auth in Azure Event Hubs Scaler Jun 27, 2022
@stale
Copy link

stale bot commented Aug 31, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 7 days if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale All issues that are marked as stale due to inactivity label Aug 31, 2022
@tomkerkhove tomkerkhove added feature All issues for new features that have been committed to and removed feature-request All issues for new features that have not been committed to labels Aug 31, 2022
@stale stale bot removed the stale All issues that are marked as stale due to inactivity label Aug 31, 2022
@yaronpri
Copy link
Author

Hi @SushmithaVReddy is there a plan to add support for this ?
I saw that #2920 (Azure Service Bus) - was closed but this issue is still valid.

@JorTurFer
Copy link
Member

is this something you have in progress @v-shenoy ?

@v-shenoy
Copy link
Contributor

v-shenoy commented Nov 25, 2022

is this something you have in progress @v-shenoy ?

I was planning to do this. But last I checked the SDK that we use does not support SAS tokens. I am not sure if that has changed.

@v-shenoy
Copy link
Contributor

is this something you have in progress @v-shenoy ?

I was planning to do this. But last I checked the SDK that we use does not support SAS tokens. I am not sure if that has changed.

Okay, I am taking a look at the docs, and there is an interface that mentions SAS tokens. I will take a look and send a PR if it's supported.

@v-shenoy
Copy link
Contributor

After going through the docs for azure-event-hubs-go/v3, it relies on the azure-amqp-common-go/sas package for a SAS token provider which requires either providing the key name & value, or a connection string with the same format as the one KEDA uses. Not the SAS token format.

@yaronpri Maybe you know more about using the SAS tokens of the form you provided with the SDK that we use in KEDA?

@yaronpri
Copy link
Author

Hi @v-shenoy from what I see the way to generate SAS token for EventHub is similar to the way it's generated for ServiceBus.
So same concept which was done for ServiceBus could work also for EventHub
https://learn.microsoft.com/en-us/azure/service-bus-messaging/service-bus-sas
https://learn.microsoft.com/en-us/rest/api/eventhub/generate-sas-token

@v-shenoy v-shenoy removed their assignment Sep 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
azure All issues concerning integration with Azure feature All issues for new features that have been committed to
Projects
Status: To Do
Development

No branches or pull requests

6 participants