lrstanley / chix Goto Github PK

🌧 Describe the problem

I'm getting the following error on provider/auth callback

Internal Server Error: could not find a matching session for this request Looks like that's happening on this line

guser, err := gothic.CompleteUserAuth(w, gothic.GetContextWithProvider(r, chi.URLParam(r, "provider")))

I guess I'm missing some configuration for sessions/cookies

⛅ Expected behavior

Expected to get the user object when login completes

🔄 Minimal reproduction

No response

💠 Version: chix

v1.0.0

🖥 Version: Operating system

macos

⚙ Additional context

No response

🤝 Requirements

• I believe the problem I'm facing is a bug, and is not intended behavior. Post here if you're not sure.
• I have confirmed that someone else has not submitted a similar bug report.

🌧 Describe the problem

When attempting to access any of the auth routes set up by the auth handler from chix, I'm getting 404 page not found and not the typical api route not found JSON, but rather just plain text "404 page not found"

⛅ Expected behavior

I expected to be redirected to the github oauth login, but instead got a 404 from the web server.

🔄 Minimal reproduction

1. Setup the provider using the code in additional context
2. Start the web server
3. Navigate to /api/auth/providers/github or any of the endpoints.

💠 Version: chix

v0.0.0-20221102040401-cd09b32313cc

🖥 Version: Operating system

linux/other

⚙ Additional context

	chix.DefaultAPIPrefix = "/api/"

	r := chi.NewRouter()

	goth.UseProviders(
		github.New(
			cli.Flags.Auth.Github.ClientID,
			cli.Flags.Auth.Github.ClientSecret,
			strings.TrimRight(cli.Flags.HTTP.BaseURL, "/")+"/api/auth/providers/github/callback",
		),
	)

	auth := chix.NewAuthHandler[ent.User, int](
		database.NewAuthService(db, cli.Flags.Auth.Github.AllowedUsers),
		cli.Flags.HTTP.ValidationKey,
		cli.Flags.HTTP.EncryptionKey,
	)


	r.Use(
...
		auth.AddToContext
	)

...

r.Mount("/api/auth", auth)

🤝 Requirements

• I believe the problem I'm facing is a bug, and is not intended behavior. Post here if you're not sure.
• I have confirmed that someone else has not submitted a similar bug report.

✨ Describe the feature you'd like

Implement some kind of logic that allows easily wrapping a function in either cron-based or interval-based execution, for use with passing into Run() or RunCtx(). This code is rather trivial to write manually, however, helpers can further simplify this logic.

Might be good to also have this be implemented using something kind of like a query builder, to toggle various features.

E.g:

NewCron(interval).Timeout(timeout).Immediate().Runner(runner)

Could also add support for interval-after-completion (5m after the last one finishes) or interval-regardless-of-duration (always triggers every 5m).

⚠ If implemented, do you think this feature will be a breaking change to users?

No

✨ Describe the feature you'd like

Cloudflare appends CF-Connecting-IP which contains the visitor's actual IP address, this is a useful way to get it if you're using Cloudflare's service.

🌧 Is your feature request related to a problem?

N/A

🔎 Describe alternatives you've considered

N/A

⚠ If implemented, do you think this feature will be a breaking change to users?

No

⚙ Additional context

N/A

🤝 Requirements

• I have confirmed that someone else has not submitted a similar feature request.
• If implemented, I believe this feature will help others, in addition to solving my problems.
• I have looked into alternative solutions to the best of my ability.
• (optional) I would be willing to contribute to testing this feature if implemented, or making a PR to implement this functionality.