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

Codeql issue hashicorp #207

Merged
merged 2 commits into from
May 4, 2024
Merged

Codeql issue hashicorp #207

merged 2 commits into from
May 4, 2024

Conversation

sayaliM0412
Copy link
Contributor

@sayaliM0412 sayaliM0412 commented May 1, 2024

fix:.github/workflows/codeql.yml: codeql fails in hashicorp with 401 Unauth .
Successful run when built with adding: maven-settings-xml-action: https://github.com/liquibase/hashicorp-vault-plugin/actions/runs/8913988741

comments on the ticket: https://datical.atlassian.net/browse/DAT-17435?focusedCommentId=141375

@jnewton03
Copy link
Contributor

@sayaliM0412 I just want to confirm:
If an external contributor forks an extension repo that is calling this workflow, they will not be able to access secrets, right?

@sayaliM0412
Copy link
Contributor Author

@sayaliM0412 I just want to confirm: If an external contributor forks an extension repo that is calling this workflow, they will not be able to access secrets, right?

I'll do a test-run and get back

@sayaliM0412
Copy link
Contributor Author

sayaliM0412 commented May 2, 2024

@sayaliM0412 I just want to confirm: If an external contributor forks an extension repo that is calling this workflow, they will not be able to access secrets, right?

I'll do a test-run and get back

Also, we are doing something similar here as well : https://github.com/liquibase/build-logic/blob/main/.github/workflows/os-extension-test.yml#L88
update: No, the external user cannot access the secrets. I think it's not accessible as the secrets are repository-specific and are not inherited or shared between repositories, even if one repository is using workflows from another repository. When an external user forks "hashicorp," they will not have access to the secrets defined in "build-logic".

@jnewton03
Copy link
Contributor

@sayaliM0412 Will forked contributor PRs run successfully?

@sayaliM0412
Copy link
Contributor Author

@sayaliM0412 Will forked contributor PRs run successfully?

yes.

@sayaliM0412 sayaliM0412 merged commit c9d864b into main May 4, 2024
@sayaliM0412 sayaliM0412 deleted the codeql-issue-hashicorp branch May 4, 2024 17:13
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.

2 participants