Skip to content

Commit bf4ddab

Browse files
committed
#36 Marke methods deprecated which will be removed after code refactoring
Signed-off-by: Sven Strittmatter <sven.strittmatter@iteratec.com>
1 parent cb4dccc commit bf4ddab

File tree

3 files changed

+9
-0
lines changed

3 files changed

+9
-0
lines changed

src/main/java/io/securecodebox/persistence/defectdojo/service/DefaultImportScanService.java

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -44,6 +44,7 @@ class DefaultImportScanService implements ImportScanService {
4444
new FormHttpMessageConverter(),
4545
new ResourceHttpMessageConverter(),
4646
new MappingJackson2HttpMessageConverter());
47+
@Deprecated
4748
private final SystemPropertyFinder properties = new SystemPropertyFinder();
4849
@Getter
4950
private final String defectDojoUrl;
@@ -230,6 +231,7 @@ ClientHttpRequestFactory createRequestFactoryWithProxyAuthConfig() {
230231
return factory;
231232
}
232233

234+
@Deprecated
233235
private static class SystemPropertyFinder {
234236
private boolean hasProperty(@NonNull final ProxyConfigNames name) {
235237
return System.getProperty(name.getLiterat()) != null;
@@ -244,6 +246,7 @@ private String getProperty(@NonNull final ProxyConfigNames name) {
244246
}
245247
}
246248

249+
@Deprecated
247250
final static class MissingProxyAuthenticationConfig extends RuntimeException {
248251
MissingProxyAuthenticationConfig(ProxyConfigNames name) {
249252
super(String.format("Expected System property '%s' not set!", name.getLiterat()));

src/main/java/io/securecodebox/persistence/defectdojo/service/ImportScanService.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -62,6 +62,7 @@ class ImportScanResponse {
6262
* or none of them!
6363
* </p>
6464
*/
65+
@Deprecated
6566
enum ProxyConfigNames {
6667
HTTP_PROXY_HOST("http.proxyHost"),
6768
HTTP_PROXY_PORT("http.proxyPort"),

src/test/java/io/securecodebox/persistence/defectdojo/service/DefaultImportScanServiceTest.java

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -107,6 +107,7 @@ void shouldConfigureProxySettings_trueIfUserAndPasswordSet() {
107107
}
108108

109109
@Test
110+
@Deprecated
110111
void createRequestFactoryWithProxyAuthConfig_throesExceptionIfUserNotSet() {
111112
System.clearProperty(ProxyConfigNames.HTTP_PROXY_USER.getLiterat());
112113
System.setProperty(ProxyConfigNames.HTTP_PROXY_PASSWORD.getLiterat(), "password");
@@ -121,6 +122,7 @@ void createRequestFactoryWithProxyAuthConfig_throesExceptionIfUserNotSet() {
121122
}
122123

123124
@Test
125+
@Deprecated
124126
void createRequestFactoryWithProxyAuthConfig_throesExceptionIfPasswordNotSet() {
125127
System.setProperty(ProxyConfigNames.HTTP_PROXY_USER.getLiterat(), "user");
126128
System.clearProperty(ProxyConfigNames.HTTP_PROXY_PASSWORD.getLiterat());
@@ -135,6 +137,7 @@ void createRequestFactoryWithProxyAuthConfig_throesExceptionIfPasswordNotSet() {
135137
}
136138

137139
@Test
140+
@Deprecated
138141
void createRequestFactoryWithProxyAuthConfig_throesExceptionIfHostNotSet() {
139142
System.setProperty(ProxyConfigNames.HTTP_PROXY_USER.getLiterat(), "user");
140143
System.setProperty(ProxyConfigNames.HTTP_PROXY_PASSWORD.getLiterat(), "password");
@@ -149,6 +152,7 @@ void createRequestFactoryWithProxyAuthConfig_throesExceptionIfHostNotSet() {
149152
}
150153

151154
@Test
155+
@Deprecated
152156
void createRequestFactoryWithProxyAuthConfig_throesExceptionIfPortNotSet() {
153157
System.setProperty(ProxyConfigNames.HTTP_PROXY_USER.getLiterat(), "user");
154158
System.setProperty(ProxyConfigNames.HTTP_PROXY_PASSWORD.getLiterat(), "password");
@@ -163,6 +167,7 @@ void createRequestFactoryWithProxyAuthConfig_throesExceptionIfPortNotSet() {
163167
}
164168

165169
@Test
170+
@Deprecated
166171
void createRequestFactoryWithProxyAuthConfig_throesExceptionIfPortIsNotInteger() {
167172
System.setProperty(ProxyConfigNames.HTTP_PROXY_USER.getLiterat(), "user");
168173
System.setProperty(ProxyConfigNames.HTTP_PROXY_PASSWORD.getLiterat(), "password");

0 commit comments

Comments
 (0)