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

updates default poll url #85

Merged
merged 1 commit into from May 13, 2021
Merged

Conversation

semperry
Copy link
Contributor

@semperry semperry commented May 3, 2021

I appreciate you using swapi, but I work off of a very small scale dyno that I pay for out of pocket, and have no intention of scaling for the kind of traffic this has been sending, and this nearly ddos'd me over the weekend. Httpbin is going to give you what you need, or perhaps put together a small api for your own needs. I'm happy to assist with that if you'd like something done up for your needs there! Thank you.

@cwise89
Copy link
Owner

cwise89 commented May 13, 2021

@semperry, your API was the first api I've ever played with when learning to code, great to "meet" you! Thanks for the PR, super sorry to burden your service like this. We had to switch from the previous polling url because of the same. I'll go ahead and test this and merge this pr to prevent the further trouble for your galaxy :)

@cwise89
Copy link
Owner

cwise89 commented May 13, 2021

lookin good!

@cwise89 cwise89 merged commit 885480c into cwise89:master May 13, 2021
@semperry
Copy link
Contributor Author

Thanks for that! No problem at all. Luckily you were using a HEAD request, it was super easy to filter the traffic and keep the server and db up. Hopefully that api works out a bit better! I honestly wouldn't have even minded at all if I had a bigger server, but I have to make sure everyone has a chance to access the data. Thanks again!

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.

None yet

2 participants