Skip to content

  • Projects
  • Groups
  • Snippets
  • Help
  • Sign in / Register
Minds Frontend
Minds Frontend
  • Project overview
  • Repository
  • Issues 403
  • Merge Requests 67
  • 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 28 minutes ago by Ben Hayward@benhayward.ben
Report abuse

Logged out update markers calls #2577

  • Overview 0
  • 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 6 commits behind the target branch
Open in Web IDE
Pipeline #117269261 running for 79aa76a4 on fix/logged-out-update-markers-2577
              Requires 3 more approvals from Devs, Deployers, and QA.
              Martin Santangelo Rami Albatal Xander Miller Brian Hatchet Emiliano Balbuena
              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 28 minutes ago

                added scoped labels

              Please register or sign in to reply
              0 Assignees
              None
              None
              Milestone
              None
              Time tracking
              No estimate or time spent
              2
              Labels
              Squad::Yellow Status::Awaiting Review
              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