Skip to content

Add response headers to ApiResponse to gather API limits information#27

Open
athlan wants to merge 2 commits intogalen:masterfrom
athlan:master
Open

Add response headers to ApiResponse to gather API limits information#27
athlan wants to merge 2 commits intogalen:masterfrom
athlan:master

Conversation

@athlan
Copy link

@athlan athlan commented Sep 25, 2013

ApiResponse has been modified, headers has been stored.

Use getRateLimit() and getRateLimitRemaining() from ApiResponse to get info about current API usage.

@galen
Copy link
Owner

galen commented Sep 30, 2013

Can you show an example of how this can be utilizied?

@athlan
Copy link
Author

athlan commented Sep 30, 2013

Of course. Our service using this library often reaches Instagram's limits so we have to switch API keys by round-robin or read these limits. We also generates the alerts if API client will soon reach limits based on header information (not hardcoded value).

@galen
Copy link
Owner

galen commented Sep 30, 2013

I mean in code. Can you show me how you use it?

On Sep 30, 2013, at 5:44 PM, Piotr Pelczar notifications@github.com wrote:

Of course. Our service using this library often reaches Instagram's limits so we have to switch API keys by round-robin or read these limits. We also generates the alerts if API client will soon reach limits based on header information (not hardcoded value).


Reply to this email directly or view it on GitHub.

@mpatnode
Copy link

I just implemented this without looking at the open PR's, but I did it a little differently. Having it hang off the response isn't very useful to the client. I'll pull and open a PR.

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

Successfully merging this pull request may close these issues.

3 participants

Comments