Skip to content
Snippets Groups Projects

Modified a bunch of services to accept the new access rights and push them...

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

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • 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() + "`");
  • Martin Weise added 1 commit

    added 1 commit

    Compare with previous version

  • Martin Weise added 1 commit

    added 1 commit

    • d6afc6f6 - Fixed the password change for amqp + service at once

    Compare with previous version

  • Martin Weise added 14 commits

    added 14 commits

    Compare with previous version

  • Martin Weise changed the description

    changed the description

  • Martin Weise approved this merge request

    approved this merge request

  • Martin Weise added 1 commit

    added 1 commit

    Compare with previous version

  • Martin Weise added 1 commit

    added 1 commit

    Compare with previous version

  • Martin Weise added 1 commit

    added 1 commit

    Compare with previous version

  • Martin Weise added 1 commit
  • merged

  • Please register or sign in to reply
    Loading