Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(multi-tenancy): upsert groot for namespace after drop data #9113

Closed
wants to merge 1 commit into from

Conversation

8i5dev
Copy link

@8i5dev 8i5dev commented Jul 18, 2024

Description: After executing drop data in a namespace, the namespace cannot be logged in anymore

  1. drop data of a namespace
curl --location 'http://localhost:8080/alter' \
--header 'X-Dgraph-AccessToken: accesstoken' \
--header 'Content-Type: text/plain' \
--data '{"drop_op": "DATA"}
'
  1. login the namespace again
curl --location 'http://127.0.0.1:8080/login' \
--header 'Content-Type: application/json' \
--data '{
    "userid": "groot",
    "password": "password",
    "namespace": 1
}'
  1. get error message Authentication from address10.42.0.1:55404 failed: unable to authenticate: invalidi credentials

function InitializeAcl only upsert groot for namespace 0

@8i5dev 8i5dev requested a review from a team as a code owner July 18, 2024 08:47
@CLAassistant
Copy link

CLAassistant commented Jul 18, 2024

CLA assistant check
All committers have signed the CLA.

@dgraph-bot dgraph-bot added the go Pull requests that update Go code label Jul 18, 2024
@mangalaman93
Copy link
Contributor

Thank you for the PR. Is it possible for you to rebase the PR while I review it?

@mangalaman93
Copy link
Contributor

Thank you for raising this issue, we have fixed it in #9159 with some tests.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
go Pull requests that update Go code
Development

Successfully merging this pull request may close these issues.

5 participants