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

Initial work for standalone manifest per sponsorable account #176

Merged
merged 15 commits into from
Apr 5, 2024

Conversation

kzu
Copy link
Member

@kzu kzu commented Apr 5, 2024

We'll move to a self-hosted model where each sponsorable account can opt-in to using SL by:

  • Creating a GH OAuth app
  • Forking this repository which provides the backend for manifest signing
  • Leveraging the unified gh sponsors CLI to manage all manifests

This removes the need for cross-sponsorable sync, issues around local-only manifest generation (and therefore, hacking) and others.

This brings the model back to a simpler approach, yet extensible enough that others can leverage it with minimal (although not non-existent) effort.

kzu added 15 commits April 3, 2024 20:32
The new message handler populates the access token in outgoing requests, as showcased in https://github.com/kzu/GitHubFunctions.

The refactored code for the sponsorship inspection is now in the SponsorManager, so we remove all the rest from the function.
We use SponsorLink as a static helper class, so we shouldn't use that as a namespace anywhere.
This is what will tie together the local-only offline check against a one-time generated JWT signed manifest containing the same.
Makes tokens easier to inspect and troubleshoot, especially if the active user has multiple accounts being used simultaneously.
The devlooped/gh-sponsors repo will now just contain the extension for the GH CLI.
Also add default funding file to targets, initial test boilerplate for manifest endpoint.
@kzu kzu added the enhancement New feature or request label Apr 5, 2024
@kzu kzu enabled auto-merge (rebase) April 5, 2024 22:52
@kzu kzu merged commit 87ffe21 into main Apr 5, 2024
4 checks passed
@kzu kzu deleted the dev/standalone branch April 5, 2024 22:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant