Cliient.sync_sso calls a missing route

Continuing the discussion from What exactly does "client.sync_sso" do? (Discourse Ruby API):

https://github.com/discourse/discourse_api/blob/master/lib/discourse_api/api/users.rb#L13-L16

Calls /admin/users/USERNAME, which doesn’t seem to exist. That’s a bug, right? It looks like that route requires the user_id to be included before username.

4 Likes

Is this correct @sam?

Yeah looks buggy probably created prior to us refactoring admin routes

Show me the love! At this rate, I’m never going to get my Tester badge. :slight_smile: (This is my second confirmed bug that didn’t get liked. :cry:)

3 Likes

Can we fix this @blake – looks easy?

1 Like

Okay fixed!

https://github.com/discourse/discourse_api/commit/c82519ffb398c56e14f9331a6b550dad4bcb6f20

Install the latest version of the gem (0.16.1) and you will get the update.

3 Likes