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

Throttling for B2B calls to notifications-push #19

Open
tosan88 opened this issue Sep 19, 2017 · 0 comments
Open

Throttling for B2B calls to notifications-push #19

tosan88 opened this issue Sep 19, 2017 · 0 comments

Comments

@tosan88
Copy link
Contributor

tosan88 commented Sep 19, 2017

As per the discussion in #14 there would be a need to throttle for B2B accessing /content/notifications-push without affecting Next. From the comments:

tamas-molnar 
This change removes throttling for B2B clients hitting the notifications-push endpoint - taking into consideration that they don't hit Mashery beforehand (which would throttle them), is this a good idea?
--
tosan88
Right, but the solution what we implemented due to Mashery's limitation of not supporting long polling makes it difficult to throttle such requests.
Do you (or somebody else) have something in mind how could we throttle B2B without affecting Next? Or should we wait for Tyk to fix it?
--
lucas42
It doesn't look like Tyk is going to solve this any time soon.
Now that we have API keys being sent, is there a way to throttle the number of open connections based on those?
--
tosan88
Do you have some concrete idea? I only can think of hacking what Mashery would do and I would rather not introduce something that would be the responsibility of the API Gateway
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant