Dev

[BruceGoodGuy]

No description provided.

[Copilot]

Pull request overview

This PR wires up a basic user registration + email confirmation + authentication flow in the Go app, adds an HTML verification email template, introduces Redis Bloom-filter–based email existence checks, and updates Redis/rate-limit infrastructure to support that flow.

Changes:

• Added verification email templating and updated the mail sender to render HTML templates.
• Implemented user registration staging in Redis + confirmation endpoint + JWT auth flow (with refresh tokens stored in Redis).
• Switched Redis client to github.com/redis/go-redis/v9, introduced Bloom filter seeding on startup, and updated Docker Compose to Redis Stack.

Reviewed changes

Copilot reviewed 16 out of 20 changed files in this pull request and generated 23 comments.

Show a summary per file

File	Description
README.md	Adds a placeholder README line.
go-app/tmp/build-errors.log	Removes a tracked build error log.
go-app/templates/verify.html	Adds the HTML verification email template.
go-app/pkg/mail/gomail.go	Updates mail sender to render and send HTML templates.
go-app/pkg/cache/redis.go	Migrates Redis client import to go-redis v9.
go-app/internal/user/service.go	Adds auth/token logic and user service methods.
go-app/internal/user/repository.go	Adds staged registration in Redis, Bloom filter seeding/checks, confirmation, refresh-token storage.
go-app/internal/user/model.go	Adds a secondary user struct type.
go-app/internal/user/handler.go	Adds verify/confirm/auth endpoints and additional request handling.
go-app/internal/user/dto.go	Adds DTOs for verify/confirm/auth flows.
go-app/internal/test/repository.go	Migrates Redis client import to go-redis v9.
go-app/internal/middleware/rate_limit.go	Makes rate limiting configurable and constructs limiter store/client.
go-app/internal/app/application.go	Injects mail into user repo, adds Bootstrap seeding, updates routes + rate limits.
go-app/go.mod	Updates dependencies (JWT, validator, go-redis v9, limiter).
go-app/go.sum	Updates dependency checksums.
go-app/cmd/api/main.go	Calls application bootstrap before routes.
docker-compose.yml	Switches Redis image to redis-stack-server.
.gitignore	Ignores go-app/tmp/.
.github/instructions/review.instructions.md	Adds repository code review standards/instructions.

Comments suppressed due to low confidence (2)

go-app/internal/middleware/rate_limit.go:23

• RateLimit creates a brand-new Redis client (and does a Ping) every time the middleware is constructed, which happens multiple times in Routes(). This wastes connections and increases startup latency; inject a shared Redis client/store into the middleware instead of calling cache.Connect() here.

func RateLimit(numberRequest int, period string) gin.HandlerFunc {
	rate, err := limiter.NewRateFromFormatted(fmt.Sprintf("%d-%s", numberRequest, period))
	if err != nil {
		panic(err)
	}

	client := cache.Connect()
	store, err := redis.NewStore(client)

go-app/internal/middleware/rate_limit.go:26

• This middleware panics on rate parsing/store creation errors. Panics crash the server and are hard to operate; prefer returning an error during app startup (and failing fast with a clear error) rather than panicking from inside middleware construction.

func RateLimit(numberRequest int, period string) gin.HandlerFunc {
	rate, err := limiter.NewRateFromFormatted(fmt.Sprintf("%d-%s", numberRequest, period))
	if err != nil {
		panic(err)
	}

	client := cache.Connect()
	store, err := redis.NewStore(client)
	if err != nil {
		panic(err)
	}