feat(react,nextjs): expose getIdToken API

[TharakaUJ]

this commit introduces the getIdToken method to both AsgardeoReactClient and AsgardeoNextClient classes.

Purpose

Related Issues

• feat: expose the raw idToken from the SDK #249

Related PRs

• N/A

Checklist

• Followed the CONTRIBUTING guidelines.
• Manual test round performed and verified.
• Documentation provided. (Add links if there are any)
• Unit tests provided. (Add links if there are any)

Security checks

• Followed secure coding standards in http://wso2.com/technical-reports/wso2-secure-engineering-guidelines?
• Confirmed that this PR doesn't commit any keys, passwords, tokens, usernames, or other secrets?

[TharakaUJ]

Since there didn’t seem to be any active work on this issue, I took the liberty of drafting a PR.

It turns out that AuthAPI already exposes a getIdToken() method, so this change simply wires it through and exposes getIdToken() from the React SDK.

Please let me know if you’d like me to update the documentation as well, or if there are any changes you’d like to see in the implementation.

Thanks, and happy new year! 🎉

[brionmario]

Since there didn’t seem to be any active work on this issue, I took the liberty of drafting a PR.

It turns out that AuthAPI already exposes a getIdToken() method, so this change simply wires it through and exposes getIdToken() from the React SDK.

Please let me know if you’d like me to update the documentation as well, or if there are any changes you’d like to see in the implementation.

Thanks, and happy new year! 🎉

Hey @TharakaUJ,

Thanks a lot for taking the time for the contribution & wish you a very happy new year as well.

Couple of things:

• Shall we expose the getIdToken from the AsgardeoContext as well so we can access it via useAsgardeo.
• Also lets do the same for the Next.js SDK. You can send a separate PR if you like for that as well.
• If you can, please add a screenshot of a console.log or something similar with the getIdToken in action.

[TharakaUJ]

Thanks a lot for the feedback!

I’ve updated this PR to also expose getIdToken via AsgardeoContext, so it can be accessed through useAsgardeo. I’ve kept all React-related changes in a single commit and amended the original commit accordingly.

I’ve also attached screenshots showing getIdToken() in action (via a console.log) as requested, please let me know if this is what you had in mind.

For the Next.js SDK changes, would you prefer that as a separate PR? I can prepare that once this one is aligned.

[TharakaUJ]

Also I noticed that there is a conflict in AsgardeoProvider.tsx. I followed the same approach as in commit 4aa3e4c

[asgardeo-github-bot]

⚠️ No Changeset found

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go.

If these changes should result in a version bump, you need to add a changeset.

Refer Release Documentation to learn how to add a changeset.

[brionmario]

LGTM!

Merged.

Thanks @TharakaUJ for your contribution 🙌