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

Fix github credencials (pass 2) #821

Merged

Conversation

vrubezhny
Copy link
Contributor

Github project credentials are to be used in wget action when accessing
https://raw.githubusercontent.com in order to minimize restriction limits

Signed-off-by: Victor Rubezhny vrubezhny@redhat.com

@mickaelistria
Copy link
Contributor

Log says Masking supported pattern matches of $GITHUB_API_TOKEN, this would need to be fixed. The $... expression may not be expanded in the withCredentials block, the direct value should be tried instead.

Github project credentials are to be used in `wget` action when accessing
`https://raw.githubusercontent.com` in order to minimize restriction limits

Signed-off-by: Victor Rubezhny <vrubezhny@redhat.com>
@vrubezhny
Copy link
Contributor Author

Log says Masking supported pattern matches of $GITHUB_API_TOKEN, this would need to be fixed. The $... expression may not be expanded in the withCredentials block, the direct value should be tried instead.

Imho, it's OK and suits our needs - we don't want passwords/tokens to be visible in the log.

@vrubezhny vrubezhny merged commit d17337f into eclipse-wildwebdeveloper:master May 12, 2022
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.

None yet

2 participants