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 Github Login #74

Open
fynnfluegge opened this issue Jul 11, 2023 · 0 comments
Open

Add Github Login #74

fynnfluegge opened this issue Jul 11, 2023 · 0 comments

Comments

@fynnfluegge
Copy link
Owner

fynnfluegge commented Jul 11, 2023

It should be possible to login with Github account.

Todos

  • create Github OAuth 2.0 App
  • add OpenID Identity Provider to Cognito UserPoolClient, since Github doesn't support Single Sign-On
  • add lambda function as proxy for access-token and user-info endpoints

Wrapper for these tasks https://github.com/TimothyJones/github-cognito-openid-wrapper
Good resource https://sst.dev/examples/how-to-add-github-login-to-your-cognito-user-pool.html

rohitg00 added a commit to rohitg00/rocketnotes that referenced this issue Oct 27, 2024
Related to fynnfluegge#74

Add Github login functionality to the application.

* **cdk/rocketnotes.go**
  - Add a new lambda function for Github OAuth 2.0 App.
  - Add OpenID Identity Provider to Cognito UserPoolClient.
  - Update the stack to include the new lambda function.

* **webapp/src/main.ts**
  - Add Github OAuth 2.0 App configuration to the `oauth` object.
  - Update `Auth.configure` to include Github as an identity provider.

* **webapp/src/aws-exports.js**
  - Add Github OAuth 2.0 App configuration to the `awsmobile` object.

---

For more details, open the [Copilot Workspace session](https://copilot-workspace.githubnext.com/fynnfluegge/rocketnotes/issues/74?shareId=XXXX-XXXX-XXXX-XXXX).
rohitg00 added a commit to rohitg00/rocketnotes that referenced this issue Oct 28, 2024
- Added secure token exchange Lambda
- Added GitHub user info Lambda
- Added user profile management with DynamoDB
- Added logging middleware
- Updated CDK stack with new resources
- Removed client secret from frontend
- Fixed Go module structure

Closes fynnfluegge#74
Addresses security concerns from PR fynnfluegge#145
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant