OAuth2 Basic - Legacy replies

[pfaffman (Jay Pfaffman)]

I think that I remember needing to add a scope before as well. I think I solved it by embedding it in the URL.

[clement.bouillier (Clément Bouillier)]

Hi,

I got a problem recently (discourse was working for more than a year…). My custom provider logs show that the RedirectUri sent by Discourse is “https://[host]:80/auth/oauth2_basic/callback”…port was not included before…

Where the port come from? I tried to search in source code, the callback_uri is defined here discourse-oauth2-basic/plugin.rb at ee5bca98f0854362ef5063e7960da426ecbc59fa · discourse/discourse-oauth2-basic · GitHub, but I can’t understand where full_host come from…

Any help welcome. Thanks.

EDIT: I found the full_host definition : omniauth/lib/omniauth/strategy.rb at master · omniauth/omniauth · GitHub but do not understand how I could set the OAuth.config.full_host? Any clues what changed?

[davidmh]

@eviltrout

We can’t just replace the GET with a POST in case some providers only support GET. I could only accept a patch like this if it was a site setting.

I happen to need this feature as well, here’s the PR

Add request methods for token and user_json urls