Skip to content

  • Projects
  • Groups
  • Snippets
  • Help
  • Sign in / Register
Minds Frontend
Minds Frontend
  • Project overview
  • Repository
  • Issues 403
  • Merge Requests 68
  • CI / CD
  • Security & Compliance
  • Packages
  • Analytics
  • Wiki
  • Snippets
  • Members
  • Collapse sidebar
  • Graph
  • Charts
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
  • Minds
  • Minds FrontendMinds Frontend
  • Merge Requests
  • !777

Open
Opened 5 days ago by Ben Hayward@benhayward.ben
Report abuse

Logged out update markers calls #2577

  • Overview 1
  • Commits 11
  • Pipelines 1
  • Changes 11

Summary

Closes #2577

In some weird set of circumstances, it appears that users can call the api/v1/groups/member endpoint while not logged in.

Steps to test

Neither I nor @brianhatchet have been able to replicate this locally.

Estimated Regression Scope

Not really much I can think of. It's just adding a logged in check to a function so that an endpoint with no purpose for a logged-out user, isn't called when logged out.

Request to merge fix/logged-out-update-markers-2577 into master
The source branch is 41 commits behind the target branch
Open in Web IDE
Pipeline #117269261 passed with warnings for 79aa76a4 on fix/logged-out-update-markers-2577
              Requires 3 more approvals from Devs, Deployers, and QA.
              Juan Manuel Solaro Olivia Madrid Martin Santangelo Rami Albatal Mark Harding
              Ready to be merged automatically. Ask someone with write access to this repository to merge this request

              Closes #2577

              Deletes source branch

              • Ben Hayward @benhayward.ben added Squad::Yellow Status::Awaiting Review scoped labels 5 days ago

                added scoped labels

              • Ben Hayward @benhayward.ben added Status::Requires Changes scoped label and automatically removed Status::Awaiting Review label 5 days ago

                added scoped label and automatically removed label

              • Mark Harding
                Mark Harding @markeharding · 9 minutes ago
                Owner

                @benhayward.ben looks like some other stuff got put in here. Can you cleanup before we merge?

              • Mark Harding @markeharding assigned to @benhayward.ben 9 minutes ago

                assigned to @benhayward.ben

              Please register or sign in to reply
              Assignee
              Ben Hayward's avatar
              Ben Hayward @benhayward.ben
              None
              Milestone
              None
              Time tracking
              No estimate or time spent
              2
              Labels
              Squad::Yellow Status::Requires Changes
              Lock merge request
              Unlocked
              10
              10 participants
              user avatar
              user avatar
              user avatar
              user avatar
              user avatar
              user avatar
              user avatar
              Reference: minds/front!777