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

Add discourse actor authorization #93

Open
wants to merge 25 commits into
base: main
Choose a base branch
from

Conversation

angusmcleod
Copy link
Contributor

@angusmcleod angusmcleod commented May 1, 2024

@pmusaraj This is ready to go once #84 is merged.

  • Authorizations now have a dedicated model and table (existing mastodon authorizations will be migrated)
  • Discourse actor authorizations are now supported.
  • The interface in user preferences has been improved.

Notes

To authorize a remote Discourse user, the ActivityPub auth redirect for the user's domain has to be in the site setting allowed user api auth redirects:

[url]/ap/auth/redirect/discourse

For example:

https://meta.discourse.org/ap/auth/redirect/discourse

@angusmcleod angusmcleod changed the title Add discourse to discourse verification Add discourse actor authorization May 1, 2024
@angusmcleod angusmcleod marked this pull request as ready for review May 7, 2024 07:43
@angusmcleod
Copy link
Contributor Author

@pmusaraj This one is ready to go.

@angusmcleod angusmcleod force-pushed the add_discourse_to_discourse_verification branch from 07e150f to d46d633 Compare May 30, 2024 10:01
@angusmcleod
Copy link
Contributor Author

angusmcleod commented May 30, 2024

@pmusaraj I've rebased and updated this so it's ready again. This should be the next one you review.

@pmusaraj
Copy link
Contributor

Thanks @angusmcleod, will look into this today!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants