Modified a bunch of services to accept the new access rights and push them...
1 open thread
Closes #235
Please test the implementation if it starts on your computer and create a second user, give him read access on a private database and see if it works
-
Also fix bug where changing the password leads to a wrong password + tag combination at the broker service
Edited by Martin Weise
Merge request reports
Activity
added priority::3 scope::Backend status::Doing + 1 deleted label
requested review from @tahaj96
assigned to @grantnert48
added status::Done label
added 51 commits
-
723ec51a...9d789d53 - 50 commits from branch
dev
- 432a2035 - Merge branch 'dev' into 235-give-access-to-more-than-one-user
-
723ec51a...9d789d53 - 50 commits from branch
- Resolved by Martin Weise
- Resolved by Martin Weise
- Resolved by Martin Weise
31 this.databaseMapper = databaseMapper; 26 32 this.databaseService = databaseService; 27 33 } 28 34 29 35 @Override 30 public void create(Long containerId, Long databaseId) throws DatabaseNotFoundException, DatabaseConnectionException, DatabaseMalformedException { 36 public void create(Long containerId, Long databaseId, Principal principal) throws DatabaseNotFoundException, 37 DatabaseConnectionException, DatabaseMalformedException, UserNotFoundException { 31 38 final Database database = databaseService.findById(containerId, databaseId); 39 final User root = databaseMapper.containerToPrivilegedUser(database.getContainer()); 32 40 /* create */ 33 final ComboPooledDataSource dataSource = getDataSource(database.getContainer().getImage(), database.getContainer(), database); 41 final ComboPooledDataSource dataSource = getDataSource(database.getContainer().getImage(), database.getContainer(), database, root); 34 42 try { 35 43 final Connection connection = dataSource.getConnection(); 44 executeQuery(connection, "USE `" + database.getInternalName() + "`"); changed this line in version 11 of the diff
added 1 commit
- d6afc6f6 - Fixed the password change for amqp + service at once
added 14 commits
-
d6afc6f6...30bd00fb - 13 commits from branch
dev
- 19827117 - Merge branch 'dev' into 235-give-access-to-more-than-one-user
-
d6afc6f6...30bd00fb - 13 commits from branch
Please register or sign in to reply