OpenConext / OpenConext/OpenConext-attribute-aggregation

Improve logging

Open
#146 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Java
Stars
1
Forks
3
Avg merge
1d 16h
Merged PRs (30d)
2

Description

  • add access log
  • Fix startup debug log. On startup, AA currently logs:
Jan 28 14:09:14 docker1.prd.ams.surfconext.nl aaserver[831]: 2026-01-28 13:09:14,018  INFO [main] aa.config.AuthorityResolver:31 - Parsed file:///attributeAuthorities.yml with configuration AuthorityConfiguration(authorities=[aa.model.AttributeAuthorityConfiguration@6518436, aa.model.AttributeAuthorityConfiguration@375299, aa.model.AttributeAuthorityConfiguration@bf8a9da0, aa.model.AttributeAuthorityConfiguration@5be7f8c, aa.model.AttributeAuthorityConfiguration@b9722f44, aa.model.AttributeAuthorityConfiguration@6e8d0de3, aa.model.AttributeAuthorityConfiguration@3529a5f3])

This doesn't tell me anything. Instead, dump the config of each of the authorities it has configured, so that I can check if that matches the input config.

  • [] If an exception occurs while processing a request or response, log the full offending request/reponse. For example:
Apr  2 09:23:52 docker4.prd.utr.surfconext.nl aaserver[818]: 2026-04-02 07:23:52,376  WARN [http-nio-8080-exec-19] o.s.w.s.m.s.DefaultHandlerExceptionResolver:254 - Resolved [org.springframework.http.converter.HttpMessageNotReadableException: JSON parse error: Cannot deserialize value of type `java.util.ArrayList<java.lang.String>` from Object value (token `JsonToken.START_OBJECT`)]

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by locating the AuthorityResolver startup logging and the request/response exception-handling paths. Compare the current startup message with the configured authority data, then identify how access logs and offending requests or responses can be emitted. Done means all three requested logging improvements are covered and their output is useful for diagnosing configuration and processing failures.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
backend, observability
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.