Skip to content

Conversation

@soul2zimate
Copy link
Contributor

@soul2zimate soul2zimate commented Nov 6, 2025

Description

Fixes: #193

  • chore: rename trustification references to guacsec
  • chore: rename CodeReady or Red Hat references to Trustify
  • chore: rename Exhort to Trustify DA
  • chore: update http headers
  • chore: remove SNYK and OSS_INDEX refs
  • chore: move package to io.github.guacsec.trustifyda
  • chore: update to trustify-da-api-spec and v5 endpoint

Checklist

  • I have followed this repository's contributing guidelines.
  • I will adhere to the project's code of conduct.

@soul2zimate soul2zimate requested a review from ruromero November 6, 2025 07:23
@soul2zimate soul2zimate force-pushed the guacsec_rename_main branch 2 times, most recently from 542071d to ab919ba Compare November 12, 2025 09:56
Copy link
Collaborator

@ruromero ruromero left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The license headers must be updated too

@soul2zimate
Copy link
Contributor Author

Thank you for reivew @ruromero, I have updated according to your comments. The license update is here #202

Are the V5 endpoints deployed ? the endpoints remain as per openapi.yaml But I see this response:

 Output structure: {
  "providers": {}
}

and error output is empty - only contains empty providers for integration tests.

@ruromero
Copy link
Collaborator

Thank you for reivew @ruromero, I have updated according to your comments. The license update is here #202

Are the V5 endpoints deployed ? the endpoints remain as per openapi.yaml But I see this response:

 Output structure: {
  "providers": {}
}

and error output is empty - only contains empty providers for integration tests.

Ok to the license issue.
It seems the v5 endpoint was not deployed to stage yet. I'll check why.

@soul2zimate soul2zimate force-pushed the guacsec_rename_main branch 2 times, most recently from fd95b33 to 8f8f393 Compare November 13, 2025 09:45
@soul2zimate soul2zimate mentioned this pull request Nov 14, 2025
2 tasks
Copy link
Collaborator

@ruromero ruromero left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@soul2zimate
Copy link
Contributor Author

Thank you for the review and approval, although I am not able to merge this yet due to the required checks failing. The tests are failing because that AnalysisReport returned from backend endpoint has no providers

@ruromero ruromero merged commit 59a2b6f into guacsec:main Nov 14, 2025
6 of 39 checks passed
@soul2zimate soul2zimate deleted the guacsec_rename_main branch November 14, 2025 12:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Rename trustification references to guacsec

2 participants