Skip to content
This repository has been archived by the owner on May 7, 2024. It is now read-only.

B 16771 okta login #3

Closed
wants to merge 10 commits into from
Closed

Conversation

cameroncaci
Copy link

Agility Ticket

Summary

As we have had some issues connecting to chamber, we have temporarily stored Okta keys in local environment variables. This PR is not a finalized solution to Okta logins; however, it's to set a new state for development to branch off of. This branch and PR will allow for successful Okta OIDC login and redirect back to the application (All 3 subdomains). Tests are still in progress, and many do not currently work. Again, this is just to set a new state for development branches.

Verification Steps for Reviewers

Successful Okta login redirects on Eula accept.

How to test

  1. Run the client and server
  2. On landing page, accept the Eula to be prompted for Okta login
  3. You should be redirected back to Milmove

… is the product of pair programming and is being shared for team development purposes. There is still much to be done and cleaned up on.
…mains. Login redirect is working as intended. Modified gitignore for better vscode debug support
@cameroncaci
Copy link
Author

Conflicts due to the feature branch being at a later commit than when B-16771 started

…n standalone package. Configured redirecthandler and callback handler to use Okta based secrets and providers to handle authentication for each app. Modified logout handler in preparation for upcoming session changes to reflect Okta implementation. Removed login.gov from handler auth context. Added new dependencies for Okta support. Added support for grant_type auth code flow with Okta auth. Modified sessions to accomodate for new Okta information. Removed most of login.gov support from auth.go, th rest to be pruned out after finalizing sessions.
…login.gov provider with okta. These tests are not finalized.
@deandreJones deandreJones deleted the branch E-04371-Okta-Phase-1 August 10, 2023 13:09
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants