Skip to content

Commit 7a20e13

Browse files
committed
#36 Add issue id to fixme tags
Signed-off-by: Sven Strittmatter <sven.strittmatter@iteratec.com>
1 parent e712c8f commit 7a20e13

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

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

+3-3
Original file line numberDiff line numberDiff line change
@@ -85,7 +85,7 @@ private ImportScanResponse createFindings(ScanFile scanFile, String endpoint, lo
8585
// 2. the raw scan result as file
8686
headers.setContentType(MediaType.MULTIPART_FORM_DATA);
8787

88-
// FIXME: Why do we use a multi value map here? Do we need multiple values for any given key?
88+
// FIXME: #36 Why do we use a multi value map here? Do we need multiple values for any given key?
8989
final var body = new LinkedMultiValueMap<String, Object>();
9090

9191
body.add("lead", Long.toString(lead));
@@ -100,7 +100,7 @@ private ImportScanResponse createFindings(ScanFile scanFile, String endpoint, lo
100100
body.remove(optionName);
101101
}
102102

103-
// FIXME: Workaround due to type incompatibility of MultiValueMap<String, String> and MultiValueMap<String, Object>.
103+
// FIXME: #36 Workaround due to type incompatibility of MultiValueMap<String, String> and MultiValueMap<String, Object>.
104104
for (final var option : options.entrySet()) {
105105
body.add(option.getKey(), option.getValue());
106106
}
@@ -114,7 +114,7 @@ public String getFilename() {
114114
}
115115
};
116116

117-
// FIXME: Why do we add the whole byte array resource here as object? Is not simply the file name sufficient here? Then we could use <String, String>
117+
// FIXME: #36 Why do we add the whole byte array resource here as object? Is not simply the file name sufficient here? Then we could use <String, String>
118118
// We send the whole file content, so DefectDojo can parse the finding by itself.
119119
body.add("file", contentsAsResource);
120120

0 commit comments

Comments
 (0)