8bitjonny
09/19/2022, 11:10 AMrp_st
09/19/2022, 11:47 AMrp_st
09/19/2022, 11:48 AM8bitjonny
09/19/2022, 1:32 PMsession.VerifySession
expects to work with the standard http.ResponseWriter
and http.Request
objects, but it reaches way deeper than I thought.
Because session.VerifySession
passes those http/net types down to recipeInstance.APIImpl.VerifySession
which in turn calls options.RecipeImplementation.GetSession
and options.RecipeImplementation.RefreshSession
both of which also use more than a handful of http Cookie or Header utility functions.
So the first proposal of adding fastHttp support to your GoSDK as a PR is fairly ambitious because of all the interfaces and functions that currently rely on http/net EVEN if we only consider only those supertokens methods that are needed for the VerifySession to work.
And the second proposal of making more of the superTokens types and interfaces publicly importable is really nice of you to offer but I'm currently a bit hesitant because I didn't think that I'd need to reimplement that much logic from scratch.8bitjonny
09/19/2022, 1:33 PMrp_st
09/19/2022, 1:45 PMrp_st
09/19/2022, 1:45 PM8bitjonny
09/19/2022, 1:58 PM