novu icon indicating copy to clipboard operation
novu copied to clipboard

🚀 Tech Feature: Specify the tenant id as HTTP header

Open alexanderhofstaetter opened this issue 3 weeks ago • 1 comments

🔖 Feature description

we seperate our clients by different tenants, and use novu as a central notification plattform. We operate a reverse proxy before novu and we need to map and validate the requests to the corresponding tenants.

🎤 Why is this feature needed ?

We use a custom JWT based auth proxy and we need to verify that the auth client_id matches (or allows) the specific tenant_id.

We currently have to look into the request body which can get pretty annoying, especially

✌️ How do you aim to achieve this?

add a tenant_id header to api calls. when this header is present, no tenant data is allowed in the body and everything gets mapped to the specific tenant.

if the tenant is not present, the tenants get auto created only with the id (additional data can be added later - e.g. in the UI)

🔄️ Additional Information

validating and inspecting http request body is pretty hard

👀 Have you spent some time to check if this feature request has been raised before?

  • [X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

Are you willing to submit PR?

None

alexanderhofstaetter avatar Jun 15 '24 22:06 alexanderhofstaetter