zowe / zowe/api-layer

Improve Spring configuration

Open
#2,942 21 comments 1 reaction 0 assignees View on GitHub

Nobody has claimed this yet.

Priority: Medium size/M technical excellence V3
Dominant language
Java
Stars
92
Forks
81
Avg merge
1d 9h
Merged PRs (30d)
46

Description

Is your feature request related to a problem? Please describe.
The APIML uses a couple of deprecated configurations of Spring. The aim of this issue is to reduce them and replace them properly.

  • spring.config.useLegacyProcessing
  • spring.mvc.favicon.enabled
    • remove this config value
    • ensure that each service serves an APIML favicon
  • spring.cloud.compatibilityVerifier
    • verify if this configuration value is still required
  • spring.main.allow-circular-references
    • see creating Jersey client in the HttpConfig bean
      • it could be solved by the lazy creation of the bean
    • fields meAsProxy to handle cache aspects
      • could be done by: different type of proxy / post bean factory handler / new aspect
      • find the best way and replace it

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.

Research direction

Start by reviewing the Spring configuration values listed in the issue and the HttpConfig bean, including the Jersey client creation and the meAsProxy fields. Use the linked Spring Boot 2.4 migration guidance to identify replacements. Done means deprecated configuration is removed or replaced, each service serves an APIML favicon, compatibility is verified, and circular references no longer require the setting.

Written by the indexing model from the issue text.

Assessment

Tech stack
java, spring, spring-boot
Domain
backend
Issue type
Refactor
Difficulty
5/5
Estimated time
Over a week
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.