kit-data-manager / kit-data-manager/wap-server
Refactor properties handling
- Dominant language
- Java
- Stars
- 4
- Forks
- 3
- PR merge metrics
- No merged PRs in 30d
Description
The current properties handling in `edu.kit.scc.dem.wapsrv.app.WapServerConfig` seems to be overly complicated and may circumvent spring functionalities in regards to properties loading.
I am a bit scared to refactor because of the generation of the application properties file on install - and because I do not have enough knowledge about spring.
Things to consider:
- properties are not defined in a default application properties. It seems to be difficult to extend or change properties without breaking anything.
- wap server specific properties do not have a prefix, which might hinder some of the usual spring properties handling. Fixing/Changing this would break backwards compatibility in regards to existing configuration files in deployment, so a migration would be necessary.
- There is a `updateConfigFromOldVersions` functionality which may rewrite existing properties files. I do not really get what it does / why it is necessary.
Contributor guide
No contributing guide indexed for this repository
Research direction
Start by reading edu.kit.scc.dem.wapsrv.app.WapServerConfig and tracing how properties are loaded and how updateConfigFromOldVersions rewrites existing files. Check the install-time generation of the application properties file and document the compatibility constraints before proposing a refactor. Done means a simpler Spring-compatible configuration path with existing deployment files preserved or migrated safely.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- java, spring
- Domain
- backend
- Issue type
- Refactor
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100