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

Support for custom properties passed to the action #1009

Open
cmboling opened this issue Apr 5, 2022 · 3 comments
Open

Support for custom properties passed to the action #1009

cmboling opened this issue Apr 5, 2022 · 3 comments

Comments

@cmboling
Copy link
Contributor

cmboling commented Apr 5, 2022

Include a field to add custom properties to FOSSA analysis. Some users are dealing with custom builds which require custom properties to be passed to the invocation.

@NielsDoucet provided a use case where a custom gradle wrapper is used where custom system properties are passed (e.g. ./gradlew -Dany.system.prop=value)

jobs:
  fossa-scan:
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@v2
      - uses: fossas/fossa-action@main # Use a specific version if locking is preferred
        with:
          api-key: ${{secrets.fossaApiKey}}
          - gradle:
            - system-properties:
              sys.prop1: value1
              sys.prop2: value2
            - properties:
              prop1: value1
              prop2: value2
@NielsDoucet
Copy link

For extra context: https://docs.gradle.org/current/userguide/build_environment.html#sec:gradle_system_properties
The relevant properties to define credentials to download a custom distribution are:

systemProp.gradle.wrapperUser=myuser
systemProp.gradle.wrapperPassword=mypassword

@NielsDoucet
Copy link

NielsDoucet commented Aug 1, 2022

I did some further digging into the issue and I'm now convinced this can't be solved exclusively at the GitHub action level. This requires the fossa cli tool to adapt how it invokes gradle. This issue should probably be moved to the fossa-cli tool repository.

@cmboling cmboling transferred this issue from fossas/fossa-action Aug 1, 2022
@cmboling
Copy link
Contributor Author

cmboling commented Aug 1, 2022

@NielsDoucet Done.

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

2 participants