set public env vars on cookie in middleware, read in client hook #3383

doug-s-nava · 2025-01-02T20:47:32Z

Summary

Fixes #{ISSUE}

Time to review: x mins

Changes proposed

What was added, updated, or removed in this PR.

Context for reviewers

Testing instructions, background context, more in-depth details of the implementation, and anything else you'd like to call out or ask reviewers. Explain how the changes were verified.

Additional information

Screenshots, GIF demos, code examples or output to help show the changes working as expected.

* adds route at /api/auth/callback to be hit by the flask API at the completion of the login flow * adds jwt token parsing logic * adds support for a "SESSION_SECRET" env var * adds a User page for temporary testing

* adds a context provider and hook to allow client components to access up to date logged in user information * adds a NextJS route to return user data decrypted from the session cookie passed up from the client * makes some temporary updates to the feature flags table in order for that page to act as a proof of concept for the functionality * moves some feature flags and loading spinner code to a more suitable location.

Creates a Node route for logging out to log out a user, which * calls the API logout endpoint * removes client side cookie Also adds fetch function to use to call the API logout route and a dummy logout button to use for testing

doug-s-nava and others added 17 commits January 2, 2025 12:20

[Issue #2654] Next login flow backend (#3182)

5a29aad

* adds route at /api/auth/callback to be hit by the flask API at the completion of the login flow * adds jwt token parsing logic * adds support for a "SESSION_SECRET" env var * adds a User page for temporary testing

[Issue #2656] client side logout route (#3275)

21ad857

Creates a Node route for logging out to log out a user, which * calls the API logout endpoint * removes client side cookie Also adds fetch function to use to call the API logout route and a dummy logout button to use for testing

[Unticketed] move fetcher files out of API directory (#3287)

9016bbc

attempt at reset to mvp

5068049

wip w nocache

e5226ed

remove idea of client side env vars

871dcda

set all feature flag values in cookie remove noCache

61afabc

i think this works

a4c6168

cleanup

beba632

adjusted feature flag manager tests, need to fix some broken ones

0f9162f

tests added and all checks but e2e passing

88e0b8e

some fix ups for e2e, hydration and accessibility

7f286ed

remove unused test

8b7dc62

add tf param

971a129

comment cleanup, simplify login button, and add one more test

03f25d3

set public env vars on cookie in middleware, read in client hook

c885340

doug-s-nava force-pushed the feature-flag-manager-refactor branch from 03f25d3 to b8f15d0 Compare January 3, 2025 21:28

Base automatically changed from feature-flag-manager-refactor to feature/nextjs-auth January 6, 2025 20:55

doug-s-nava force-pushed the feature/nextjs-auth branch from e358419 to f73dffe Compare January 7, 2025 15:12

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

set public env vars on cookie in middleware, read in client hook #3383

set public env vars on cookie in middleware, read in client hook #3383

doug-s-nava commented Jan 2, 2025

set public env vars on cookie in middleware, read in client hook #3383

Are you sure you want to change the base?

set public env vars on cookie in middleware, read in client hook #3383

Conversation

doug-s-nava commented Jan 2, 2025

Summary

Time to review: x mins

Changes proposed

Context for reviewers

Additional information