Jak bezpečně a efektivně pracovat s více databázemi pro různé organizace?

Druhou databázovou konexi můžeš mít jako závislost bez autowiringu. Ale já bych ještě chvíli zůstal u toho, proč musí mít každá organizace svou vlastní databázi? Skutečně to bude tak velká aplikace, že bys narážel na limity velikostí tabulek? Nebo proč?

A.Venturer napsal(a):

Bezpečnost: Hlavním cílem bylo minimalizovat riziko, že by v případě chyby v kódu mohl uživatel vidět data jiné organizace.

I při oddělených databázích může proplout v kódu chyba, která data z jiné databáze zpřístupní. Navíc bude potřeba „někde“ uchovávat hesla k dílčím databázím, což může přinést ještě větší riziko.

Jednodušší dotazy: Chtěl jsem se vyhnout přidávání podmínek podle organizace do většiny dotazů.

Hodně záleží, jak budou koncipovány modelové vrstvy. Ale celkem ideální je, když nějaká metoda vrací základní selectionu a ostatní metody na ni nabalují. Takže omezení dle organizace bude jen v té jedné základní metodě.

Budoucí rozšíření: Do budoucna jsem uvažoval o možnosti, že by se jednotlivé organizace mohly přímo připojit ke své databázi. Zvažuji, zda by tato potřeba mohla být efektivněji řešena například přes REST API.

Pustit je přímo do databáze bude vždy cesta do pekel. A poměrně přímá. Jakékoliv API je cesta správným směrem.

Další věc k zamyšlení. Proč mají být úvodní loginy v nějaké centrální / main databázi? Proč by neměla mít každá organizace svůj virtual, svou databázi a v ní jen své loginy? Prostě pro N zákazníků udržovat N instancí aplikace+databáze. (to je také velmi častý přístup)

Děkuji za vaše cenné postřehy a otázky! Na základě zvažování jsem se rozhodl použít jednu databázi a rozlišovat data pomocí sloupce organization_id v každé tabulce. Připravil jsem dva přístupy, které řeší přístup k datům.

Jaký přístup byste na základě své praxe upřednostnili? Budu také rád za vaše postřehy – například odhalení potenciálních chyb, návrhy na zlepšení nebo usnadnění implementace.

Logika registrace nového účtu:
Při vytvoření nového uživatelského účtu se přiřadí organization_id = NULL.

PŘÍSTUP #1: Inspirace od Kamila Valenty.
V modelu mám základní selection. Další metody využívají tento výběr.

ActivityModel:

class ActivityModel
{
    private Explorer $database;

    public function __construct(Explorer $database)
    {
        $this->database = $database;
    }

    /**
     * Základní výběr aktivit - získá všechny aktivity přiřazené konkrétní organizaci
     */
    public function getActivitiesForOrganization(int $organization_id)
    {
        return $this->database->table('activities')
            ->where('organization_id', $organization_id);
    }

    public function getActivity(int $id, int $organizationId): ?ActiveRow
    {
        return $this->getActivitiesForOrganization($organizationId)
            ->where('id', $id)
            ->fetch();
    }

ActivityPresenter:

    private function getOrganizationIdForUser(): int
    {
        $userIdentity = $this->getUser()->getIdentity();
        if ($userIdentity === null || $userIdentity->organization_id === null) {
            throw new \Exception('Organizace není dostupná nebo uživatel není přihlášen.');
        }
        return $userIdentity->organization_id;
    }

    public function renderShow(int $id): void
    {
        $organizationId = $this->getOrganizationIdForUser();
        $activity = $this->activityModel->getActivity($id, $organizationId);
        $this->template->activity = $activity;
    }

PŘÍSTUP #2: Jednoduchý ACL.

RegistrationPresenter

    public function renderDefault(): void
    {
        $organizationId = $this->accessControl->getOrganizationId();
        if ($organizationId === null) {
            $this->error('Nemáte přístup k žádné organizaci.');
        }
        $registrations = $this->registrationModel->getRegistrationForOrganization($organizationId);
        $this->template->registrations = $registrations;
    }

    public function renderShow(int $id): void
    {
        $registration = $this->registrationModel->getRegistration($id);

        if (!$registration) {
            $this->error('Registrace nebyla nalezena.');
        }

        $registrationOrganizationId = $registration->organization_id;
        if (!$this->accessControl->isAllowed('view', $registrationOrganizationId)) {
            $this->error('Nemáte oprávnění zobrazit tuto registraci.');
        }

        $this->template->registration = $registration;
    }

RegistrationModel

    public function getRegistrationForOrganization(int $organizationId): array
    {
        return $this->database->table('registrations')
            ->where('organization_id', $organizationId)
            ->fetchAll();
    }

    public function getRegistration(int $id): ?ActiveRow
    {
        return $this->database->table('registrations')
            ->where('id', $id)
            ->fetch();
    }

AccessControl

    public function getOrganizationId(): ?int
    {
        $identity = $this->user->getIdentity();
        return $identity ? $identity->organization_id : null;
    }

    public function isAllowed(string $action, int $organizationId): bool
    {
        if (!$this->user->isLoggedIn()) {
            return false;
        }

        $userOrganizationId = $this->getOrganizationId();
        if ($userOrganizationId === null || $userOrganizationId !== $organizationId) {
            return false;
        }

        return true;
    }

Pokud máte návrh na zcela jiné a lepší řešení, moc rád si ho poslechnu! Všem ještě jednou děkuji.

A.Venturer napsal(a):

ActivityModel:
`php
class ActivityModel
{
private Explorer $database;

public function __construct(Explorer $database)
{
$this->database = $database;
 }

/**
* Základní výběr aktivit – získá všechny aktivity přiřazené konkrétní organizaci
*/
public function getActivitiesForOrganization(int $organization_id)
{
return $this->database->table(‚activities‘)
->where(‚organization_id‘, $organization_id);
 }

public function getActivity(int $id, int $organizationId): ?ActiveRow
{
return $this->getActivitiesForOrganization($organizationId)
->where(‚id‘, $id)
->fetch();
 }

Já bych ID organizace držel v identitě uživatele. ActivityModel si o „user“ požádá v závislostech.

     public function getActivitiesForOrganization()
     {
         return $this->database->table('activities')
             ->where('organization_id', $this->user->getIdentity()->getData()['organization_id']);
     }

     public function getActivity(int $id): ?ActiveRow
     {
         return $this->getActivitiesForOrganization()
             ->where('id', $id)
             ->fetch();
     }