spring-projects / spring-projects/spring-data-rest

Associations to unexported repositories are broken [DATAREST-376]

Open
#756 0 comments 0 reactions 1 assignee View on GitHub

@odrotbohm is already working on this.

Since Dec 31, 2020.

type: bug
Dominant language
Java
Stars
958
Forks
568
PR merge metrics
No merged PRs in 30d

Description

Olivier Sels opened DATAREST-376 and commented

DATAREST-262 added support for nested associations to embeddables. However this also creates problems with associations to unexported repositories, even if they shouldn't be serialized at all.

With a single association this results in duplicate links with the same name. With another nested relation this eventually results in an infinite loop as LinkCollectingAssociationHandler:doWithAssociation is repeatedly called when links are being discovered.

Removing the else clause resolves this problem for us as the jackson serialization works correctly. However then it breaks tests for DATAREST-262.

I've included my data model and simple test to illustrate the problem


Affects: 2.1.2 (Dijkstra SR2), 2.2 RC1 (Evans)

Attachments:

1 votes, 2 watchers

Contributor guide

Open the contributing guide

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.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.