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

Sugar Pond CAS -> Intercode OAuth2 #893

Merged
merged 1 commit into from
May 22, 2022
Merged

Sugar Pond CAS -> Intercode OAuth2 #893

merged 1 commit into from
May 22, 2022

Conversation

nbudin
Copy link
Contributor

@nbudin nbudin commented May 22, 2022

The goal is to switch from using Sugar Pond Accounts for authentication over to using Intercode accounts. There are a few reasons for this:

  • Sugar Pond Accounts is CAS-based, whereas Intercode is an OAuth2/OpenID Connect server, which is much better supported in the Ruby ecosystem
  • Larp Library is now a NEIL project (actually, it basically has been for most of its life - even though it started as an Alleged Entertainment project, it transferred to NEIL within a few months) and should use a NEIL-controlled authentication server
  • Using Intercode for authentication enables some possible future features such as automatically pulling a list of cons where games have run, and/or automatically generating proposals to run Larp Library games at future cons

This is a forward-port of an abandoned feature branch to do this. We've since extracted a gem from that branch called intercode_client that makes the OAuth client stuff reusable, so this new implementation uses that. Otherwise, it's very similar to the older one.

This is a forward-port of an abandoned feature branch to do this.  We've
since extracted a gem from that branch called intercode_client that
makes the OAuth client stuff reusable, so this new implementation uses
that.  Otherwise, it's very similar to the older one.
@nbudin nbudin added enhancement major Major version release labels May 22, 2022
@nbudin nbudin merged commit 59b8c99 into main May 22, 2022
@nbudin nbudin deleted the decassification branch May 22, 2022 18:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement major Major version release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant