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

#51 Registro Delle Transazioni (WIP) #52

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

pokono
Copy link

@pokono pokono commented Oct 27, 2020

Posting this to gather feedback.
Added LoggerEvent and handling.

LoggerEvent può essere usato per scrivere dati di log in una tabella del database.
Al momento questo è quello che sto usando:

create table spid_entries
(
    id                               bigint unsigned auto_increment
        primary key,
    uuid                             char(36)     not null,
    action                           varchar(255) null,
    request_id                       varchar(255) null,
    request_issue_instant            varchar(255) null,
    request_xml                      text         null,
    response_id                      varchar(255) null,
    response_issue_instant           varchar(255) null,
    response_xml                     text         null,
    assertion_id                     varchar(255) null,
    assertion_name_id                varchar(255) null,
    assertion_name_id_name_qualifier varchar(255) null,
    error                            text         null,
    created_at                       timestamp    null,
    updated_at                       timestamp    null,
    constraint spid_entries_uuid_unique
        unique (uuid)
)
    collate = utf8mb4_unicode_ci;

create index spid_entries_action_index
    on spid_entries (action);

create index spid_entries_action_request_id_index
    on spid_entries (action, request_id);

create index spid_entries_assertion_id_index
    on spid_entries (assertion_id);

create index spid_entries_request_id_index
    on spid_entries (request_id);

Ancora non ho aggiornato ne documentazione, ne eventuali tests.
Vorrei sapere che ne pensate, e se siete interessati a mergiare una cosa simile.

Grazie mille,
Ivan

Added LoggerEvent and handling.
@pdavide
Copy link
Member

pdavide commented Nov 1, 2020

Ciao @pokono, innanzitutto grazie per il tuo contributo e per aver sollevato il problema della mancanza di un sistema di log per le transazioni.

Per l'implementazione di questa funzionalità ti propongo l'approccio che ho abbozzato nella #53.
Rispetto alla soluzione che hai presentato qui, mi sembra che quell'altra rispetti una maggiore separation of concerns e quindi un maggiore disaccoppiamento.
Se vuoi lavorarci su, ci sarebbe ancora da:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants