Comments (12)
It was decided that we will use the user's PAT to log them in, sort of like the flow I mentioned here - #8 (comment). I can move forward with the same design I used in my submission, and maybe later when we have a consistent design system, we can edit everything.
from ai.
Are we adding a
Signin with GitHub
button, or using the PAT like we did in our demos? If we are integrating with OpenSauced auth, how can we go about doing that?
Generating and using a PAT for the extension is the safest all-round way of doing it, since ay extension mistakes or hijacking would be visible through token usage, however both methods can be explored for knowledge sharing purposes 🍕
from ai.
We can test **.github.com/**
for redirect strings on beta
from ai.
Are we adding a Signin with GitHub
button, or using the PAT like we did in our demos? If we are integrating with OpenSauced auth, how can we go about doing that?
from ai.
Okay, I can start working on this with PAT login.
from ai.
Here's an auth flow I made for my demo, let me know what can be improved, we haven't made a solid design decision yet too @Anush008
from ai.
@diivi, I've used Supabase's auth flow by redirecting to /auth/v1
, followed by a check for the session-cookie. The session holds the auth-token for API calls.
https://user-images.githubusercontent.com/46051506/232797763-699ea9ac-fd09-4cbc-b25c-5c17a987e53b.gif
from ai.
@diivi, I've used Supabase's auth flow by redirecting to
/auth/v1
, followed by a check for the session-cookie. The session holds the auth-token for API calls
@0-vortex, what do you think of this, security-wise?
I'll try to look at the code once I have access to Anush's repo.
from ai.
@diivi, I've used Supabase's auth flow by redirecting to
/auth/v1
, followed by a check for the session-cookie. The session holds the auth-token for API calls@0-vortex, what do you think of this, security-wise? I'll try to look at the code once I have access to Anush's repo.
IMHO we should set up that but on the development SupaBase, and try to enable another redirect url like https://**.github.com/**
from the beta branch (of either insights or hot endpoints) - if that works it would simplify the log in by a lot 🍕
from ai.
IMHO we should set up that but on the development SupaBase, and try to enable another redirect url like
https://**.github.com/**
from the beta branch (of either insights or hot endpoints) - if that works it would simplify the log in by a lot
Can you help me understand how to set this up, like where I can learn more about how authentication currently works in OpenSauced? I did it through PATs and Anush did it by redirecting directly to the authentication URL, and then checking for the presence of an access token in the cookies.
from ai.
🎉 This issue has been resolved in version 1.0.0-beta.1 🎉
The release is available on GitHub release
Your semantic-release bot 📦🚀
from ai.
🎉 This issue has been resolved in version 1.0.0 🎉
The release is available on GitHub release
Your semantic-release bot 📦🚀
from ai.
Related Issues (20)
- Bug: unable to login - Chrome HOT 9
- Monthly issue metrics report HOT 1
- Feature: add blocking labels to the take action HOT 1
- Bug: AI extension is not showing the PR writing message HOT 7
- Feature: refactor constant naming HOT 3
- Feature: enable AI description generator for all PRs even on comments HOT 2
- Bug: unstable AI Description HOT 3
- Bug: repository chatbot tracker issue HOT 3
- Bug: AI description generator on comments is not enabling the comment button HOT 2
- Bug: repo name is being empty HOT 1
- POC: E2E test with Playwright HOT 9
- Bug: Can't generate refactor, test, and explanation code when clicking the OpenSauced orange button HOT 1
- Bug: OpenSauced orange button doesn't appear on the PR form
- Bug: Can't generate Highlight from an issues HOT 4
- Feature: show a message to user when they are not logged in HOT 2
- fix: Add correct path to the usage guide on README and Docs in the extension, and update "Contributing" and "Community" sections in README HOT 3
- Bug: Lint error: Expected linebreaks to be 'LF' but found 'CRLF' HOT 6
- Bug: Can't run `npm run push` to commit changes
- Feature: Cut over to v2 API HOT 1
- Remove PR Description Generator HOT 2
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from ai.