Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PropertiesMigrationListener wrongly reports property as deprecated when has group #42068

Closed
philwebb opened this issue Aug 31, 2024 · 1 comment
Assignees
Labels
type: regression A regression from a previous release
Milestone

Comments

@philwebb
Copy link
Member

From #35774 (comment)

Hi. I am running Spring Boot 3.3.3 and encountering this issue (with same configurations as in the original report). Could this issue be reopen?

Here is a Dockerfile to reproduce this warning log.

FROM eclipse-temurin:21

WORKDIR /tmp/demo

RUN apt-get update \
    && DEBIAN_FRONTEND=noninteractive apt-get install -y --no-install-recommends \
        curl \
        unzip

RUN curl \
        -G https://start.spring.io/starter.zip \
        -d type=gradle-project \
        -d javaVersion=21 \
        -d bootVersion=3.3.3 \
        -o /tmp/demo.zip

RUN unzip /tmp/demo.zip -d /tmp/demo

RUN ./gradlew dependencies

RUN sed -i '/dependencies {/a runtimeOnly("org.springframework.boot:spring-boot-properties-migrator")' build.gradle

RUN echo 'spring.kafka.ssl.trust-store-password=secret' >> ./src/main/resources/application.properties

CMD ["./gradlew", "bootRun"]

Output:

Starting a Gradle Daemon, 1 incompatible and 1 stopped Daemons could not be reused, use --status for details

> Task :bootRun

  .   ____          _            __ _ _
 /\\ / ___'_ __ _ _(_)_ __  __ _ \ \ \ \
( ( )\___ | '_ | '_| | '_ \/ _` | \ \ \ \
 \\/  ___)| |_)| | | | | || (_| |  ) ) ) )
  '  |____| .__|_| |_|_| |_\__, | / / / /
 =========|_|==============|___/=/_/_/_/

 :: Spring Boot ::                (v3.3.3)

2024-08-30T20:34:39.745Z  INFO 142 --- [demo] [           main] com.example.demo.DemoApplication         : Starting DemoApplication using Java 21.0.4 with PID 142 (/tmp/demo/build/classes/java/main started by root in /tmp/demo)
2024-08-30T20:34:39.746Z  INFO 142 --- [demo] [           main] com.example.demo.DemoApplication         : No active profile set, falling back to 1 default profile: "default"
2024-08-30T20:34:39.982Z  INFO 142 --- [demo] [           main] com.example.demo.DemoApplication         : Started DemoApplication in 0.368 seconds (process running for 0.484)
2024-08-30T20:34:39.986Z  WARN 142 --- [demo] [           main] o.s.b.c.p.m.PropertiesMigrationListener  :
The use of configuration keys that have been renamed was found in the environment:

Property source 'Config resource 'class path resource [application.properties]' via location 'optional:classpath:/'':
        Key: spring.kafka.ssl.truststore-password
                Line: 2
                Replacement: spring.kafka.ssl.trust-store-password


Each configuration key has been temporarily mapped to its replacement for your convenience. To silence this warning, please update your configuration to use the new keys.


BUILD SUCCESSFUL in 9s
4 actionable tasks: 4 executed
@philwebb philwebb added the status: waiting-for-triage An issue we've not yet triaged label Aug 31, 2024
@spring-projects spring-projects deleted a comment Aug 31, 2024
@philwebb philwebb added type: bug A general bug and removed status: waiting-for-triage An issue we've not yet triaged labels Aug 31, 2024
@philwebb philwebb added this to the 3.2.x milestone Aug 31, 2024
@philwebb philwebb added type: regression A regression from a previous release and removed type: bug A general bug labels Aug 31, 2024
@philwebb philwebb changed the title PropertiesMigrationListener wrongly reports property as deprecated PropertiesMigrationListener wrongly reports property as deprecated when has group Aug 31, 2024
@philwebb philwebb self-assigned this Aug 31, 2024
@philwebb philwebb modified the milestones: 3.2.x, 3.2.10 Aug 31, 2024
@philwebb
Copy link
Member Author

Thanks for reporting this @arkinmodi. It should be fixed in the next set of releases.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: regression A regression from a previous release
Projects
None yet
Development

No branches or pull requests

1 participant