Juster logging - legger saksnummer i MDC ved kjøring av steg #2564
Conversation
| @@ -33,6 +33,7 @@ public TekniskBehandlingStegVisitor(BehandlingskontrollServiceProvider servicePr | |||
| @Override | |||
| public StegProsesseringResultat prosesser(BehandlingStegModell steg) { | |||
| LOG_CONTEXT.add("fagsak", kontekst.getFagsakId()); // NOSONAR //$NON-NLS-1$ | |||
| LOG_CONTEXT.add("saksnummer", kontekst.getSaksnummer().getVerdi()); // NOSONAR //$NON-NLS-1$ | |||
There was a problem hiding this comment.
Vi har prøvd å strangulere kall til fortsettBehandling(Gjenoppta..) ned til å foregå i prosesstasks, fortrinnsvis fra FortsettBehandlingTask.
Er det ikke like greit å sette loggkonteksten i disse taskene framfor å dra inn nye konsept inn i kjernen?
Sak er ikke et kjernebegrep i behandlingskontroll - den bør klare seg med modell + oppslagsnøkkel, steg, aksjonspunkt og transisjon :-)
Hva ville gjøre det lettere å skille ut behandlingskontroll som eget repo (utover grensesnitt for behandling)? BehandlingskontrollKontekst sin fagsakId brukes så få steder at det frister å skrive den ut :-)
There was a problem hiding this comment.
Tore: foreslår at du heller lar ProsessTaskDispatcher sette LogContext prosess. -
Setter saksnummer som task_parameter (PTData.setSaksnummer(String)). Da logges prosess[fagsak=...]
Behandlingskontroll . prosesserBehandling skal kalles fra task, problem løst og annen setting av LogContext i Visitor kan egentlig fjernes
No description provided.