new-coder icon indicating copy to clipboard operation
new-coder copied to clipboard

Published 20 hours ago • verified publisher icon econchick

API tutorial - giantbomb API now requires user agent

Open nickdavidhaynes opened this issue 8 years ago • 3 comments

Was taking a look at the API tutorial and kept getting 403 return when calling the giantbomb API. Googling uncovered this forum post from a month ago - http://www.giantbomb.com/forums/api-developers-3017/api-call-throws-error-403-forbidden-1496714/?page=1#js-message-8232912

Looks like giantbomb API now requires a (arbitrary) user agent. Adding this line fixes the 403.

nickdavidhaynes avatar Mar 16 '16 17:03 nickdavidhaynes

LGTM, thanks for the report and patch!

merwok avatar Mar 16 '16 18:03 merwok

I was going to add some similar code to ensure that people knew they had to use a user-agent for the giatbomb api. As of a couple days ago, @nickdavidhaynes 's changes hadn't been pulled into @econchick 's master branch. Are there any plans to do this? How does that work?

skadiet avatar Jul 30 '17 11:07 skadiet

I tend to give feedback on PRs but leave the final review and approval to @econchick

merwok avatar Jul 31 '17 16:07 merwok