Skip to content

Commit

Permalink
chore: Update get accounts endpoint
Browse files Browse the repository at this point in the history
Signed-off-by: Mahendra Paipuri <[email protected]>
  • Loading branch information
mahendrapaipuri committed Jan 4, 2024
1 parent d492685 commit 5dede8c
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions pkg/jobstats/server/server.go
Original file line number Diff line number Diff line change
Expand Up @@ -188,9 +188,9 @@ func (s *JobstatsServer) accounts(w http.ResponseWriter, r *http.Request) {
w.WriteHeader(http.StatusOK)

// Get current user from header
user, _ := s.getUser(r)
loggedUser, dashboardUser := s.getUser(r)
// If no user found, return empty response
if user == "" {
if loggedUser == "" {
response = base.AccountsResponse{
Response: base.Response{
Status: "error",
Expand All @@ -208,9 +208,9 @@ func (s *JobstatsServer) accounts(w http.ResponseWriter, r *http.Request) {
}

// Get all user accounts
accounts, err := s.Accounts(s.dbConfig.JobstatsDBTable, user, s.logger)
accounts, err := s.Accounts(s.dbConfig.JobstatsDBTable, dashboardUser, s.logger)
if err != nil {
level.Error(s.logger).Log("msg", "Failed to fetch accounts", "user", user, "err", err)
level.Error(s.logger).Log("msg", "Failed to fetch accounts", "user", dashboardUser, "err", err)
response = base.AccountsResponse{
Response: base.Response{
Status: "error",
Expand Down

0 comments on commit 5dede8c

Please sign in to comment.