[Question]: Why was spring-boot-starter-aop removed from dependencies?

Question

[Question]: Why was spring-boot-starter-aop removed from dependencies?

maxl2287 opened this issue 7 months ago · 8 comments

Hi together,

just a short question.
We are using resilience4j, which needs:

<dependency>
	<groupId>org.springframework.boot</groupId>
	<artifactId>spring-boot-starter-aop</artifactId>
</dependency>

With upgrading spring-cloud-dependencies to the latest version 2023.0.2, we faced issues that our Retry-mechanism was not working anymore.
Then we analyzed the dependencies and saw that you have removed the aop-dependency from spring-cloud-openfeign-core (6084609).

As a change we just have included aop by ourself, but we just wanted to know what was the intention of removing this.

Thanks in advance! :)

Answer 1 · 2024-06-03T07:15:47.000Z

Just to confirm that this issue affecting multiple dependent projects. We have added the dependency directly to resolve the issue.

It would be good to know why the dependency was removed and why it was there in the first place. Unfortunately, the Resilience4J annotations are just silently ignored without it (unless you have tests verifying their behaviour, which we fortunately had).

Thanks 🙂

Answer 2 · 2024-06-03T09:46:21.000Z

Just to confirm that this issue affecting multiple dependent projects. We have added the dependency directly to resolve the issue.

It would be good to know why the dependency was removed and why it was there in the first place. Unfortunately, the Resilience4J annotations are just silently ignored without it (unless you have tests verifying their behaviour, which we fortunately had).

Thanks 🙂

@jjlharrison Exactly we had the same issue. This issue only came up by having verification tests, which did not trigger retries at all after upgrading spring-cloud-dependencies :)

Answer 3 · 2024-06-12T06:52:17.000Z

Hello
Are there any news on this topic?
Thanks

Answer 4 · 2024-06-25T11:50:33.000Z

Hi all, we have discover that spring-boot-starter-aop was a legacy dependency in Spring Cloud OpenFeign. The project hasn't used it for a long time and hence it should not be there anymore. Additionally, it was causing an AOT issue (spring-projects/spring-data-mongodb#4637 (comment)) that was in no way related to SC OpenFeign. If the dependency is required by other projects, the users should add it directly if not provided.

Answer 5 · 2024-06-26T10:44:00.000Z

Hello, @OlgaMaciaszek !
Thanks for the explanation, we had a similar issue.
As a suggestion, maybe it worth updating at least minor version in case you modify the structure of dependencies.
We have dependabot grouping patch updates and PR with 10 different updates failed.
It would be easier to find this thread, if we knew that openfeign update was the reason.

Answer 6 · 2024-06-26T10:50:13.000Z

I agree that it would be preferable, however, we do a major/minor release usually just once a year and given the AOT bug, it would have been problematic to wait that much, so since it was a bugfix, we decided to go ahead with it. We're sorry for any inconvenience this might have caused.

Answer 7 · 2024-10-30T10:53:07.000Z

As other says, this goes for all the starters, that are removed. The documentation should at least be updated, as it is clearly wrong pt. (eg https://spring.io/projects/spring-cloud-sleuth)

Answer 8 · 2024-10-30T13:48:17.000Z

@lazee Spring Cloud Sleuth is a separate project. If you've found any other problem in Spring Cloud OpenFeign docs, please create a separate GH issue and we'll fix it.