Skip to content

fix: update mowojang API base URL#323

Merged
DarthGigi merged 3 commits into
SkyCryptWebsite:devfrom
Piorunx22:dev
May 16, 2026
Merged

fix: update mowojang API base URL#323
DarthGigi merged 3 commits into
SkyCryptWebsite:devfrom
Piorunx22:dev

Conversation

@Piorunx22

Copy link
Copy Markdown
Contributor

Description

https://mowojang.matdoes.dev/ returns HTTP 301 redirect to https://mowojang.seraph.si/

Fixed the 3D skin viewer by updating the texture fetch to the new host.

Type of Change

  • 🐛 Bug fix (non-breaking change which fixes an issue)
  • ✨ New feature (non-breaking change which adds functionality)
  • 💥 Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • 🎨 Style/UI changes
  • ♻️ Code refactoring (no functional changes, no api changes)
  • ⚡ Performance improvements
  • 🧪 Adding or updating tests
  • 🔧 Build/CI changes
  • 🧹 Chore (maintenance, dependencies, etc.)

Testing

  • I have tested these changes locally
  • I have added tests that prove my changes work as expected (if appropriate)

Checklist

  • My code follows the project's style guidelines
  • I have used conventional commits for all commit messages
  • I have added necessary documentation (if appropriate)
  • I have added tests that prove my changes work as expected (if appropriate)
  • I have added changesets if my changes should be reflected in the changelog

Code Quality Checks

  • pnpm lint passes without errors
  • pnpm format has been run (or no formatting issues exist)
  • pnpm build completes successfully

@Piorunx22
Piorunx22 requested a review from DarthGigi as a code owner May 16, 2026 07:32
@changeset-bot

changeset-bot Bot commented May 16, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: a260947

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@DuckySoLucky

Copy link
Copy Markdown
Member

@DarthGigi +1

DarthGigi added 2 commits May 16, 2026 17:26
Signed-off-by: Gigi <47110839+DarthGigi@users.noreply.github.com>
@DarthGigi
DarthGigi merged commit 4363f2a into SkyCryptWebsite:dev May 16, 2026
5 checks passed
@DarthGigi

Copy link
Copy Markdown
Member

Thanks!

@Kathund

Kathund commented May 17, 2026

Copy link
Copy Markdown

Even though this was merged I just want to point out something interesting that I found

I'm going to refer to the two instances of mowojang as matjang and seraphjang. Matjang will refer to https://mowojang.matdoes.dev and seraphjang will refer to https://mowojang.seraph.si

matjang seams to be having issues and sometimes returning 500 with the text Internal server error
image
And other times just fully resolving. Other times it just redirects to seraphjang with 301

seraphjang seams to be a proxy of some sorts of matjang as it says in the description of base page (/home - https://mowojang.seraph.si/)

My personal suggestion would be to try matjang and if it fails fall back to serahjang
There could be more that I am missing but this is just some interesting things that I found

@DuckySoLucky

Copy link
Copy Markdown
Member

Even though this was merged I just want to point out something interesting that I found

I'm going to refer to the two instances of mowojang as matjang and seraphjang. Matjang will refer to https://mowojang.matdoes.dev and seraphjang will refer to https://mowojang.seraph.si

matjang seams to be having issues and sometimes returning 500 with the text Internal server error image And other times just fully resolving. Other times it just redirects to seraphjang with 301

seraphjang seams to be a proxy of some sorts of matjang as it says in the description of base page (/home - https://mowojang.seraph.si/)

My personal suggestion would be to try matjang and if it fails fall back to serahjang There could be more that I am missing but this is just some interesting things that I found

At backend I'm using official mojang's API as fallback in case this one fails, so I guess we could do that on the frontend as well?

@Clemintina

Clemintina commented May 21, 2026

Copy link
Copy Markdown

Hello, as the co-owner of the Seraph project. The proxy was a fork of mat's code, we then rewrote it with multiple instances to reduce the error rate his proxy has. As such, he was credited in the official documentation. Our proxy should have no errors, and follows proper conventions unless you hit our original Rust based proxy which is still running his code which uses html errors.

@DarthGigi

Copy link
Copy Markdown
Member

Thanks for your input @Clemintina

After talking to mat on discord, I've decided to completely switch from his (and seraph's) mowojang instance to just using the mojang api directly. As he told me that mojang's api added IP ratelimits.

Since our calls to mowojang are mostly done on the frontend/client side, that means that every call has the user's IP and not ours or mat's/seraph's mowojang instance(s), essentially fixing the rate limit issue as well as taking off some load on you.

@Clemintina

Copy link
Copy Markdown

Thanks for your input @Clemintina

After talking to mat on discord, I've decided to completely switch from his (and seraph's) mowojang instance to just using the mojang api directly. As he told me that mojang's api added IP ratelimits.

Since our calls to mowojang are mostly done on the frontend/client side, that means that every call has the user's IP and not ours or mat's/seraph's mowojang instance(s), essentially fixing the rate limit issue as well as taking off some load on you.

No worries, feel free to use ours if this solution doesn't work well. It adds virtually no load on our servers.

@DarthGigi

Copy link
Copy Markdown
Member

Nevermind I guess
image

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.

5 participants