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.

Sign up for GitHub

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

Jump to bottom

Error when '/plot visit <alias>' #4586

Open
2 tasks done
Folas1337 opened this issue Feb 3, 2025 · 4 comments
Open
2 tasks done

Error when '/plot visit <alias>' #4586

Folas1337 opened this issue Feb 3, 2025 · 4 comments
Labels
Requires Testing This is a new issue which needs to be approved before labeled with "bug"

Comments

@Folas1337
Copy link

Folas1337 commented Feb 3, 2025 •
edited
Loading

Server Implementation

Paper

Server Version

1.21.4

Describe the bug

I get errors like the following when I first set up a plot alias name with '/plot alias set ' and then try to '/plot visit ':

[Mon 15:19:12 ERROR com.mojang.authlib.minecraft.client.MinecraftClient] Got an error with a html body connecting to https://api.minecraftservices.com/minecraft/profile/lookup/bulk/byname: <html>
<head><title>403 Forbidden</title></head>
<body>
<center><h1>403 Forbidden</h1></center>
<hr><center>Microsoft-Azure-Application-Gateway/v2</center>
</body>
</html>
- [Mon 15:19:13 ERROR com.mojang.authlib.minecraft.client.MinecraftClient] Got an error with a html body connecting to https://api.minecraftservices.com/minecraft/profile/lookup/bulk/byname: <html>
<head><title>403 Forbidden</title></head>
<body>
<center><h1>403 Forbidden</h1></center>
<hr><center>Microsoft-Azure-Application-Gateway/v2</center>
</body>
</html>

To Reproduce

  1. Go to a plot of yours
  2. Use '/plot alias set testAlias'
  3. Use '/plot visit testAlias'

Expected behaviour

No error should show up in the console

Screenshots / Videos

No response

Error log (if applicable)

No response

Plot Debugpaste

https://athion.net/ISPaster/paste/view/9a352b28d05f424885fcd715bccbfc00

PlotSquared Version

PlotSquared version 7.4.1-Premium

Checklist

Anything else?

No response
@Folas1337 Folas1337 added the Requires Testing This is a new issue which needs to be approved before labeled with "bug" label Feb 3, 2025
@RedstoneFuture
Copy link
Member

On my test server, the command works well with the default settings.

@Folas1337
Copy link
Author

I admit it seems to no longer be happening for all of the aliases there are but it is still happening for some and I have no idea why.

A user of mine has a plot aliased as "zack_Builds" which works fine and one called "zack_Trees" which throws the error shown above. I hope the developers might have a clue or perhaps with some further information can help out.

@PierreSchwang
Copy link
Member

PierreSchwang commented Feb 22, 2025 •
edited
Loading

The error indicates hitting some kind of client-specific error. Ratelimit should be error 429, so i don't really know when mojang / microsoft throws a forbidden - but i doubt PlotSquared can do anything there (potentially better error handling)

EDIT:
com.mojang.authlib.minecraft.client.MinecraftClient - PS doesn't use mojangs authlib, so def nothing with PS

@PierreSchwang PierreSchwang closed this as not planned Won't fix, can't repro, duplicate, stale Feb 22, 2025
@Folas1337
Copy link
Author

Folas1337 commented Feb 22, 2025 •
edited
Loading

The error indicates hitting some kind of client-specific error. Ratelimit should be error 429, so i don't really know when mojang / microsoft throws a forbidden - but i doubt PlotSquared can do anything there (potentially better error handling)

EDIT: com.mojang.authlib.minecraft.client.MinecraftClient - PS doesn't use mojangs authlib, so def nothing with PS

But it happens every time I use the command and exactly at that time, reproducible at least on my server. You might not be using that exact authlib but perhaps something within PS is triggering it to then lead to this issue.

I'm really hoping you could reconsider and look into this again because I don't even have a clue where it might be coming from and I'm definitely willing to figure out why it might be happening in the first place.

It could very well be a rate limit thing but perhaps we can find a way so that PlotSquared no longer triggers this and no error would be printed anymore.

@PierreSchwang PierreSchwang reopened this Feb 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Requires Testing This is a new issue which needs to be approved before labeled with "bug"
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@RedstoneFuture @PierreSchwang @Folas1337