🐞 Board > Rest Client: 임포트, Non-static 전환 등 Nettee Client의 수정이 덜 반영된 영역을 수정

[silberbullet]

Issues

• Resolves 🐞 Board > Rest Client: 임포트, Non-static 전환 등 Nettee Client의 수정이 덜 반영된 영역을 수정합니다. #100

Description

• 변경된 Nettee Client에 맞추어 컴파일 에러를 해결합니다.

Review Points

• Client Properties가 yml 파일에서 값을 가져올 시, 문자열 내에 공백은 제거하도록 수정하였습니다.

@Slf4j
@ConfigurationProperties("app.client")
public record ClientProperties(
        String baseUrl,
        Map<String, String> url
) {
    public ClientProperties {
        if (baseUrl == null || baseUrl.isEmpty()) {
            baseUrl = "http://localhost:8080";
            log.warn("baseUrl is null or empty");
        } else {
            baseUrl = baseUrl.strip();
        }

        log.debug("baseUrl url: {}", baseUrl);

        for (Entry<String, String> entry : url.entrySet()) {
            entry.setValue(entry.getValue().strip());
            log.debug((entry.getKey() + ": " + entry.getValue()));
        }
    }
}

How Has This Been Tested?

• No test

Additional Notes

🦉 Nettee Client는 이렇게 사용해주세요!

1. Nettee Client는 싱글톤을 기반하여 Bean 주입이 필요합니다.

@Component
@RequiredArgsConstructor
public class SampleCommandNetteeClientAdapter implements SampleCommandNetteeClientPort {
    // ❗ Bean 주입이 필요
    private final NetteeClient netteeClient;

    .....
}

2. Nette Client Adapter는 타 도메인 호출 시, {도메인}api의존을 추천드립니다.

// 예사
dependencies {
    api(project(boardApi))
    api(project(":rest-client"))
}

[merge-simpson]

👍 깔끔하게 잘 수정해 주셨습니다.

잘 수정된 것 같고,
NetteeClient 사용을 위한 설명도 감사드립니다.

팀 내 온보딩 문서 작성에 잘 참고할 수 있어 보이네요 👍

🚀	🏅 Approve !