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

feat(server): improve logging #1274

Merged
merged 2 commits into from
Jul 26, 2024
Merged

feat(server): improve logging #1274

merged 2 commits into from
Jul 26, 2024

Conversation

setchy
Copy link
Member

@setchy setchy commented Jul 25, 2024

Add project name to log statements to make it easier to debug

@setchy setchy requested a review from prabhu as a code owner July 25, 2024 21:44
Signed-off-by: Adam Setch <adam.setch@outlook.com>
@setchy setchy force-pushed the feat/improve-server-logging branch from bbd8c68 to 3a33279 Compare July 25, 2024 21:45
server.js Fixed Show fixed Hide fixed
Signed-off-by: Adam Setch <adam.setch@outlook.com>
@@ -178,14 +178,17 @@
let bomNSData = (await createBom(srcDir, reqOptions)) || {};
bomNSData = postProcess(bomNSData, reqOptions);
if (reqOptions.serverUrl && reqOptions.apiKey) {
console.log("Publishing SBOM to Dependency Track");
console.log(
`Publishing SBOM ${reqOptions.projectName} to Dependency Track`,

Check failure

Code scanning / CodeQL

Use of externally-controlled format string High

Format string depends on a
user-provided value
.
Format string depends on a
user-provided value
.
@setchy setchy added mode:server cdxgen used as server enhancement New feature or request labels Jul 25, 2024
@setchy setchy merged commit c10cf16 into master Jul 26, 2024
24 of 25 checks passed
@setchy setchy deleted the feat/improve-server-logging branch July 26, 2024 09:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request mode:server cdxgen used as server
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants