Skip to content

Add proxies#8

Draft
c-m-hunt wants to merge 3 commits intoahobsonsayers:mainfrom
c-m-hunt:add-proxies
Draft

Add proxies#8
c-m-hunt wants to merge 3 commits intoahobsonsayers:mainfrom
c-m-hunt:add-proxies

Conversation

@c-m-hunt
Copy link
Copy Markdown

@c-m-hunt c-m-hunt commented Jul 18, 2025

It seems that rate limiting works on a combination of source IP and API key so providing a list of proxy servers can allow us to hit the endpoint ever 30 seconds per proxy. Using NordVPN's proxy list, we can hit the endpoint every 5 seconds.

For tests, set up .env

TWICKETS_API_KEY=xxx
PROXY_USER=xxx
PROXY_PASSWORD=xxx
PROXY_HOST=nl.socks.nordhold.net //Example endpoint
PROXY_PORT=1080

Have added new instantiation method to not change the existing interface.

@c-m-hunt
Copy link
Copy Markdown
Author

While this code works on this repo and potentially adds a level of obscurity to the requests, after testing, it doesn't help the refresh rate.

Happy to dump this PR if there's no need for it

@ahobsonsayers
Copy link
Copy Markdown
Owner

Hi @c-m-hunt, thanks for putting this together and trying it out. Sad that it doesn't seem to help.

If it is the case that it does not help, I think we should close this PR - but I have a question first. Are you still seeing rate limiting if you keep the refetch time to 1 minute? Or are you only seeing it if the refetch time is below 30 seconds?

If it's only below 30 seconds, I might ask: why do you need to do this? What I've discovered is that Twickets batch-add ticket listings to their feed, and they don't do this every minute. In fact, it seems they do this every 2-3 minutes. I guess having a low refetch time would allow you to pick up when this batch add is done sooner, but I'm interested in why you think you need to refetch that often.

Sorry for my questions, but I'm genuinely interested, and I might have an alternative solution if there's a use case for this.

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.

2 participants