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

WIP: Fiberplane Auth integration #319

Draft
wants to merge 22 commits into
base: main
Choose a base branch
from
Draft

WIP: Fiberplane Auth integration #319

wants to merge 22 commits into from

Conversation

brettimus
Copy link
Contributor

@brettimus brettimus commented Oct 14, 2024

PR is WIP.

image

The basic flow/components are:

  • Deploy auth service (fp-services), which:

    • Has credentials of GitHub OAuth application
    • Can create JWTs for users who log in with GitHub
    • Can forward JWTs to local studio
    • Has authenticated routes for AI request generation
  • Configure Studio to run a tiny local auth server in the background, which gives us a predictable port to communicate with the auth service mentioned above

  • Modify Studio UI and API to allow users to log in

  • Modify Studio settings page for AI Request Generation to include details about using FP as the "model"

TODO

  • Implement AI calls agains FP service
  • Show remaining requests in UI
  • Propagate expiry

Copy link

pkg-pr-new bot commented Oct 14, 2024

Open in Stackblitz

pnpm add https://pkg.pr.new/fiberplane/fpx/@fiberplane/studio@319
pnpm add https://pkg.pr.new/fiberplane/fpx/@fiberplane/hono-otel@319

commit: e6f062f

export const FPX_PORT = +(process.env.FPX_PORT ?? 8788);

export const FPX_AUTH_BASE_URL =
process.env.FPX_AUTH_BASE_URL || "http://localhost:3578";
Copy link
Contributor Author

Choose a reason for hiding this comment

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

TODO: Change default

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.

1 participant