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.

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

Prevent returning 401 for other successful OAuth2 plugins #263

Merged
merged 2 commits into from
Jan 12, 2024

Conversation

akirk
Copy link
Contributor

@akirk akirk commented Jan 12, 2024

Fixes akirk/enable-mastodon-apps#43

IndieAuth interferes with the Enable Mastodon Apps plugin when it successfully authenticates the user. This is a follow up to #245 where we checked for a successful authentication but the REST response is still {"error":"invalid_token","error_description":"Invalid access token"} coming from the rest_authentication_errors hook.

Copy link
Member

@pfefferle pfefferle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@pfefferle pfefferle merged commit 816d7ff into indieweb:trunk Jan 12, 2024
6 checks passed
@veeyawn
Copy link

veeyawn commented Jan 13, 2024

Not sure this is fixed yet. I can currently connect with the Enable Mastodon Apps plugin however if I enable the updated IndieAuth plugin (v4.4.2) I'm unable to connect with Mastodon apps (Fedilab, Phanpy, etc). If I deactivate the IndieAuth plugin I'm able to connect with the apps again.

@dshanske
Copy link
Member

@akirk Are you still taking the lead on diagnosing the plugin interaction?

@akirk
Copy link
Contributor Author

akirk commented Jan 14, 2024

Yes, I'm on it and released https://github.com/akirk/enable-mastodon-apps/releases/tag/0.6.4 to address this.

@dshanske
Copy link
Member

Yes, I'm on it and released https://github.com/akirk/enable-mastodon-apps/releases/tag/0.6.4 to address this.

Great. Sorry about the conflict... I thought we got it last time

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.

Cannot connect any client to Wordpress
4 participants