Query on Cognito auth code generated for React frontend

[himan085]

When PDK generates code for the frontend React website, it checks if the function getAuthenticatedUser is defined or not: code pointer
This function is fetched from the context and used inside useEffect as shown below:

const { getAuthenticatedUser } = useCognitoAuthContext();

  useEffect(() => {
    if (getAuthenticatedUser) {
      const authUser = getAuthenticatedUser();
      // code doing stuff using authUser
    }
  }, [getAuthenticatedUser, setUsername, setEmail]);

I am not able to understand why the check if (getAuthenticatedUser) {...} is required. The aws-northstar library already provides a default state for the CognitoAuthContext which defines the getAuthenticatedUser method: code pointer

The same question applies to the sign out code

Please let me know if you have context on this, thanks in advance!

[agdimech]

Hi Himanshu!

This is a great question. The reason is that this code existed prior to this PR in aws-northstar which changed the getAuthenticatorUser from optional to required.

You are 100% right though, these checks are no longer required given aws-northstar guarantees them to be present now - we just haven't changed the website code as it doesn't harm anything :)

Feel free to open a PR to remove this as that would be greatly appreciated!

[agdimech]

Thanks for contributing #683