User Request
Type: bug
Target: Console Application
Submitted by: @aashu2006
Console Request ID: 09a95aa8-ef9f-4185-b757-018a7284ba3a
Description
What happened:
In pkg/api/handlers/notifications.go:123 and 139, both handlers use c.Locals("userID") directly instead of middleware.GetUserID(c). Every other handler in the package uses the middleware abstraction. If the middleware key name ever changes, these silently return nil instead of failing at compile time.
What I expected:
Both handlers should use middleware.GetUserID(c) consistent with all other handlers in the package.
Steps to reproduce:
- Change the middleware userID key name
- Observe GetNotificationConfig and SaveNotificationConfig silently return nil userID
This issue was automatically created from the KubeStellar Console.
User Request
Type: bug
Target: Console Application
Submitted by: @aashu2006
Console Request ID: 09a95aa8-ef9f-4185-b757-018a7284ba3a
Description
What happened:
In
pkg/api/handlers/notifications.go:123and139, both handlers usec.Locals("userID")directly instead ofmiddleware.GetUserID(c). Every other handler in the package uses the middleware abstraction. If the middleware key name ever changes, these silently return nil instead of failing at compile time.What I expected:
Both handlers should use
middleware.GetUserID(c)consistent with all other handlers in the package.Steps to reproduce:
This issue was automatically created from the KubeStellar Console.