Skip to content
This repository has been archived by the owner on Jan 10, 2025. It is now read-only.

Commit

Permalink
OPHYK-100 Change debug level logs to info level as they would have ma…
Browse files Browse the repository at this point in the history
…de debugging the issue much easier
  • Loading branch information
rce committed Jun 28, 2024
1 parent 9b07216 commit 6cdbc15
Show file tree
Hide file tree
Showing 7 changed files with 13 additions and 13 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -123,12 +123,12 @@ public Iterable<Rekisterointi> listaaRekisteroinnit(
@ApiParam("rekisteröityvän organisaation nimi (tai sen osa)") @RequestParam(value = "hakutermi", required = false) String hakutermi) {
List<String> organisaatioOidit = haeOrganisaatioOidit(authentication.getAuthorities());
if (onOphVirkailija(organisaatioOidit)) {
LOGGER.debug("Käyttäjällä on oikeus nähdä kaikki rekisteröinnit.");
LOGGER.info("Käyttäjällä on oikeus nähdä kaikki rekisteröinnit.");
return rekisterointiService.listByTilaAndOrganisaatio(tila, hakutermi);
}
List<String> kunnat = virkailijanKunnat(organisaatioOidit);
if (kunnat.isEmpty()) {
LOGGER.debug("Käyttäjällä ei ole oikeutta nähdä yhdenkään kunnan rekisteröintejä.");
LOGGER.info("Käyttäjällä ei ole oikeutta nähdä yhdenkään kunnan rekisteröintejä.");
return List.of();
}
return rekisterointiService.listByTilaAndKunnatAndOrganisaatio(
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ public ScheduledTaskDao(JdbcTemplate template) {
public Set<TaskMonitoringService.TaskFailure> haeEpaonnistumiset() {
Set<TaskMonitoringService.TaskFailure> epaonnistumiset = new HashSet<>();
for (TaskMonitoringService.MonitoredTaskType tyyppi : TaskMonitoringService.MonitoredTaskType.values()) {
LOGGER.debug("Haetaan yli {} kertaa epäonnistuneet taskit tyyppiä: {}",
LOGGER.info("Haetaan yli {} kertaa epäonnistuneet taskit tyyppiä: {}",
TaskMonitoringService.MAX_ALLOWED_FAILURES, tyyppi.taskName);
template.query(
FAILING_TASKS_QUERY,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -120,7 +120,7 @@ List<OrganisaatioNimi> organisaatioNimet(KielistettyNimi kielistettyNimi) {
String yritysMuotoKoodiUriToNimi(String uri) {
String nimi = null;
if (YRITYSMUOTOURI_PATTERN.matcher(uri).matches()) {
LOGGER.debug("Muunnetaan yritysmuotokoodi: {}", uri);
LOGGER.info("Muunnetaan yritysmuotokoodi: {}", uri);
Koodi koodi = yritysmuotoUriToKoodi.get(uri);
if (koodi != null) {
nimi = koodi.nimi.get(DEFAULT_KIELI_KOODI_ARVO);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -18,14 +18,14 @@ public class PermissionService implements PermissionEvaluator {

@Override
public boolean hasPermission(Authentication authentication, Object targetDomainObject, Object permission) {
LOGGER.debug("PermissionService#hasPermission({}, {}, {})", authentication.getName(), targetDomainObject, permission);
LOGGER.info("PermissionService#hasPermission({}, {}, {})", authentication.getName(), targetDomainObject, permission);
LOGGER.warn("Target '{}' is not supported with permission '{}'", targetDomainObject, permission);
return false;
}

@Override
public boolean hasPermission(Authentication authentication, Serializable targetId, String targetType, Object permission) {
LOGGER.debug("PermissionService#hasPermission({}, {}, {}, {})", authentication.getName(), targetId, targetType, permission);
LOGGER.info("PermissionService#hasPermission({}, {}, {}, {})", authentication.getName(), targetId, targetType, permission);
if (permission instanceof String) {
return hasPermission(authentication, targetId, targetType, (String) permission);
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@ public void luoTaiPaivitaOrganisaatio(Long rekisterointiId) {
Rekisterointi rekisterointi = lataaRekisterointi(rekisterointiId);
String oid = rekisterointiOrgFinalizer.luoTaiPaivitaOrganisaatio(rekisterointi);
if (rekisterointi.organisaatio.oid == null) {
LOGGER.debug("Tallennetaan rekisteröintiin luodun organisaation oid: {}", oid);
LOGGER.info("Tallennetaan rekisteröintiin luodun organisaation oid: {}", oid);
rekisterointiRepository.save(rekisterointi.withOrganisaatio(rekisterointi.organisaatio.withOid(oid)));
}
schedulerClient.schedule(
Expand Down Expand Up @@ -77,6 +77,6 @@ private void ajastaPaatosEmail(Long rekisterointiId) {
paatosEmailTask.instance(taskId(paatosEmailTask, rekisterointiId), rekisterointiId),
Instant.now()
);
LOGGER.debug("Päätös-email ajastettu rekisteröinnille {}.", rekisterointiId);
LOGGER.info("Päätös-email ajastettu rekisteröinnille {}.", rekisterointiId);
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,7 @@ public String luoTaiPaivitaOrganisaatio(Rekisterointi rekisterointi) {
OrganisaatioDto createdOrg = organisaatioClient.create(organisaatioService.muunnaOrganisaatio(organisaatio));
LOGGER.info("Luotu uusi organisaatio {} rekisteröinnin pohjalta.", createdOrg.oid);
if (rekisterointi.tyyppi.equals(REKISTEROINTITYYPPI_JOTPA)) {
LOGGER.debug("Luodaan uusi jotpa aliorganisaatio oppilaitostypille ei tiedossa");
LOGGER.info("Luodaan uusi jotpa aliorganisaatio oppilaitostypille ei tiedossa");
OrganisaatioDto jotpaChild = organisaatioClient.create(OrganisaatioDto.jotpaChildOppilaitosFrom(createdOrg));
LOGGER.info("Luotu uusi jotpa aliorganisaatio {}", jotpaChild.oid);

Expand All @@ -82,7 +82,7 @@ private void paivitaVardaTiedot(String organisaatioOid) {
organisaatioClient.save(dto);
LOGGER.info("Organisaation lakkautuspäivämäärä poistettu, oid: {}", organisaatioOid);
} else {
LOGGER.debug("Organisaatioon ei tarvittu muutoksia, oid: {}", organisaatioOid);
LOGGER.info("Organisaatioon ei tarvittu muutoksia, oid: {}", organisaatioOid);
}
}

Expand All @@ -101,12 +101,12 @@ private void paivitaJotpaTiedot(String organisaatioOid) {
dto = organisaatioClient.save(dto);
LOGGER.info("Organisaation lakkautuspäivämäärä poistettu, oid: {}", organisaatioOid);
} else {
LOGGER.debug("Organisaatioon ei tarvittu muutoksia, oid: {}", organisaatioOid);
LOGGER.info("Organisaatioon ei tarvittu muutoksia, oid: {}", organisaatioOid);
}

Collection<OrganisaatioDto> jotpaOrgChilden = organisaatioClient.getOrganisaatioJalkelaisetByOid(dto.oid);
if (jotpaOrgChilden.stream().noneMatch(o -> o.oppilaitostyyppi.equals(JOTPA_CHILD_OPPILAITOSTYYPPI))) {
LOGGER.debug("Jotpa-organisaation päivityksen jälkeen huomattu että aliorganisaatio puuttuu, jotpa organisaation oid: {}", organisaatioOid);
LOGGER.info("Jotpa-organisaation päivityksen jälkeen huomattu että aliorganisaatio puuttuu, jotpa organisaation oid: {}", organisaatioOid);
OrganisaatioDto jotpaOppilaitosDto = OrganisaatioDto.jotpaChildOppilaitosFrom(dto);
OrganisaatioDto jotpaSaved = organisaatioClient.save(jotpaOppilaitosDto);
LOGGER.info("Luotu puuttuva oppilaitostyyppinen aliorganisaatio(uuden oid: {}) Jotpa organisaatiolle(jotpa oid: {})", jotpaSaved.oid, organisaatioOid);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -118,7 +118,7 @@ public Rekisterointi resolve(String paattajaOid, PaatosDto paatosDto, RequestCon
Rekisterointi saved = rekisterointiRepository.save(rekisterointi.withPaatos(paatos));
LOGGER.info("Päätös tallennettu rekisteröinnille: {}", rekisterointi.id);
RekisterointiAuditDto auditAfterDto = new RekisterointiAuditDto(saved.tila);
LOGGER.debug("Rekisteröinnin {} tila päivitetty: {}", saved.id, saved.tila);
LOGGER.info("Rekisteröinnin {} tila päivitetty: {}", saved.id, saved.tila);
eventPublisher.publishEvent(new UpdatedEvent<>(requestContext, "rekisterointi", saved.id,
auditBeforeDto, auditAfterDto));

Expand Down

0 comments on commit 6cdbc15

Please sign in to comment.