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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use outfits.openneo-assets.net for outfit images #1

Merged
merged 1 commit into from Nov 17, 2021

Conversation

matchu
Copy link
Contributor

@matchu matchu commented Nov 17, 2021

outfits.openneo-assets.net is the more canonical host for outfit images, and we control the traffic differently sometimes.

Right now, we're working on moving outfit image pressure off of Vercel, and we're only really able to control the outfits.openneo-assets.net with that level of granularity right now.

So, switching to this host should help Neobot take a bit more pressure off our bill! 馃挅 and help clean up my logs to help confirm that we've actually switched users over!

I haven't tested this change, but it seems simple enough? But it might be smart to have this code build an outfit image URL once before shipping imo, depending on how cautious you're feeling!

ty friend~ 馃挏

`outfits.openneo-assets.net` is the more canonical host for outfit images, and we control the traffic differently sometimes.

Right now, we're working on moving outfit image pressure off of Vercel, and we're only really able to control the `outfits.openneo-assets.net` with that level of granularity right now.

So, switching to this host should help Neobot take a bit more pressure off our bill! 馃挅 and help clean up my logs to help confirm that we've actually switched users over!

I haven't tested this change, but it seems simple enough? But it might be smart to have this code build an outfit image URL once before shipping imo, depending on how cautious you're feeling!

ty friend~ 馃挏
@diceroll123
Copy link
Owner

LGTM!

@diceroll123 diceroll123 merged commit 5355874 into diceroll123:master Nov 17, 2021
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