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

merge #826

Closed
wants to merge 36 commits into from
Closed

merge #826

wants to merge 36 commits into from

Conversation

Prioq
Copy link

@Prioq Prioq commented Aug 22, 2024

No description provided.

Prioq and others added 30 commits March 31, 2024 02:51
@Prioq Prioq closed this Aug 22, 2024
@alanbixby
Copy link
Member

alanbixby commented Aug 22, 2024

I understand this appears to be an accidental PR; but an obligatory reminder that you shouldn't use a free proxy for authenticated API requests - it is both insecure and will cause your cookie to be invalidated.

https://devforum.roblox.com/t/psa-stop-using-roblox-proxies-roproxy-rprxyxyz-rprxy/1573256

And if you did want to do something like this - there are much better ways to implement this, such as extending http.js and creating noblox option in settings.json to replace .roblox.com at runtime - or use a package like global-agent to proxy all traffic coming out of your node process.

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