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

tenants:add add even if tenant already present making duplicates and malfunction of previous one. #1027

Open
amg-web opened this issue Sep 28, 2023 · 1 comment
Labels
bug Something isn't working

Comments

@amg-web
Copy link
Contributor

amg-web commented Sep 28, 2023

tenants:add create new tenant even if tenant with same name and secret already present making duplicates and malfunction of previous one.

How to get:

  1. create tenant docker exec -it scalelite-api ./bin/rake tenants:add[name, secret]
  2. create meeting - everything is ok.
  3. create same tenant again: docker exec -it scalelite-api ./bin/rake tenants:add[name, secret]
    new tenant with new id is added.
    here work unpredictable, moodle can get issues with api requests.
  4. remove second tenant (duplicate): api returns wrong secret

fix:
rename first tenant:
create tenant docker exec -it scalelite-api ./bin/rake tenants:update[id,name2, secret]
rename first tenant to original name:
create tenant docker exec -it scalelite-api ./bin/rake tenants:update[id,name, secret]

@amg-web amg-web added the bug Something isn't working label Sep 28, 2023
@Ithanil
Copy link
Collaborator

Ithanil commented Oct 18, 2023

Ran into this issue as well, albeit by accidentally renaming a tenant to another existing tenant's name with tenants:update. Your method of then fixing the other tenant worked, thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants