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

Action not compatible with latest detect version #45

Open
tvcsantos opened this issue Jan 4, 2023 · 2 comments
Open

Action not compatible with latest detect version #45

tvcsantos opened this issue Jan 4, 2023 · 2 comments

Comments

@tvcsantos
Copy link

Hey,

I was trying to use the action for detect 8.4.0. I found out that using RAPID scan mode is currently not working when creating reports.

Error: Failed due to an unhandled error: 'TypeError: Cannot read property 'join' of undefined'

By doing some research, forking and debugging I've found out that is currently failing at:

function createComponentRow(component: IComponentReport): string {
  const violatedPolicies = component.violatedPolicies.join('<br/>')

The reason being is that the latest detect version 8.4.0 (https://github.com/blackducksoftware/synopsys-detect) uses a different blackduck-common-api 2022.10.04 (https://github.com/blackducksoftware/blackduck-common-api), whose model at least for reports is not compatible with the current action.

It seems that the current version of this action is only compatible until the latest major version 7.

We would like to use the action for the latest version. Can you please take this into account on the next release of the action.

Meanwhile I purpose to add a Note on this action stating that it is only compatible up to 7.

If you need something please reach out. Thanks

@tvcsantos
Copy link
Author

PR #46 adds support for Detect v8

@tvcsantos
Copy link
Author

tvcsantos commented Sep 19, 2023

Since this action is no longer maintained and some of the issues were blocking we decided to continue its development on an up to date fork https://github.com/mercedesbenzio/detect-action, that has this issue fixed.

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

No branches or pull requests

1 participant