Skip to content

Conversation

DecDuck
Copy link

@DecDuck DecDuck commented Oct 1, 2025

See my little journey here: #203

Adds the new (to me, at least) minecraft.api.profiles.host option to the doc/usage.md

@KoishiMoe
Copy link
Contributor

I would suggest to change the homepage too?

--- a/main.go
+++ b/main.go
@@ -54,6 +54,7 @@ type App struct {
 	AccountURL               string
 	ServicesURL              string
 	SessionURL               string
+	ProfilesURL              string
 	AuthlibInjectorURL       string
 	DB                       *gorm.DB
 	GetURLMutex              *KeyedMutex
@@ -600,6 +601,7 @@ func setup(config *Config) *App {
 		AuthURL:                  Unwrap(url.JoinPath(config.BaseURL, "auth")),
 		ServicesURL:              Unwrap(url.JoinPath(config.BaseURL, "services")),
 		SessionURL:               Unwrap(url.JoinPath(config.BaseURL, "session")),
+		ProfilesURL:              Unwrap(url.JoinPath(config.BaseURL, "profiles")),
 		AuthlibInjectorURL:       Unwrap(url.JoinPath(config.BaseURL, "authlib-injector")),
 		APIURL:                   Unwrap(url.JoinPath(config.BaseURL, DRASL_API_PREFIX)),
 		VerificationSkinTemplate: verificationSkinTemplate,
--- a/view/root.tmpl
+++ b/view/root.tmpl
@@ -122,7 +122,8 @@
 -Dminecraft.api.auth.host={{ .App.AuthURL }}
 -Dminecraft.api.account.host={{ .App.AccountURL }}
 -Dminecraft.api.session.host={{ .App.SessionURL }}
--Dminecraft.api.services.host={{ .App.ServicesURL }}</pre>
+-Dminecraft.api.services.host={{ .App.ServicesURL }}
+-Dminecraft.api.profiles.host={{ .App.ProfilesURL }}</pre>
 
   <p>{{call .T "For example, the full command you use to start the server might be:" }}</p>
   <pre style="word-wrap: break-word; white-space: pre-wrap; overflow-x: auto">
@@ -132,6 +133,7 @@ java -Xmx1024M -Xms1024M \
     -Dminecraft.api.account.host={{ .App.AccountURL }} \
     -Dminecraft.api.session.host={{ .App.SessionURL }} \
     -Dminecraft.api.services.host={{ .App.ServicesURL }} \
+    -Dminecraft.api.profiles.host={{ .App.ProfilesURL }} \
     -jar server.jar nogui</pre>
 
   <h4>{{ call .T "Minecraft 1.7.2 through 1.15.2" }}</h4>

@catfromplan9
Copy link
Contributor

catfromplan9 commented Oct 16, 2025

AFAIK profiles host is api.mojang.com, so isn't this just account host? Correct me if I'm wrong. I think that a documentation and homepage update alone suffices.

@catfromplan9
Copy link
Contributor

catfromplan9 commented Oct 16, 2025

Yeah, there is nothing at all referencing a /profiles endpoint. It seems mojang just decided to rename minecraft.api.account.host. There should be a shortcut put in place where /profiles goes to the same endpoints as /account, but we could also leave it as-is and update solely the documentation.

Here's the section that needs to be duplicated:

drasl/main.go

Lines 317 to 321 in b415a37

e.GET("/account/user/security/location", accountVerifySecurityLocation)
e.GET("/account/users/profiles/minecraft/:playerName", accountPlayerNameToID)
e.POST("/account/profiles/minecraft", accountPlayerNamesToIDs)

Then it should be as simple as updating documentation and homepages. I don't think authlib-injector routes for "profile" should be made, that's its own protocol.

@catfromplan9
Copy link
Contributor

See #207, you should close this

@evan-goode
Copy link
Member

Thanks for investigating, @DecDuck, I merged #207.

@evan-goode evan-goode closed this Oct 17, 2025
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.

4 participants