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 adding git sources without configured credentials #142

Merged
merged 1 commit into from
Oct 12, 2022
Merged

fix adding git sources without configured credentials #142

merged 1 commit into from
Oct 12, 2022

Conversation

mandelsoft
Copy link
Contributor

If (re)sources are added to a component archive the access method is created for validation. This must be possible, even if the technical access requires crednetials, which are not yet provided.

Therefore operations requiring credentials (for private repos) for the github access method have been moved to the methods really requesting access to the content.

What this PR does / why we need it:

Which issue(s) this PR fixes:
Fixes #

Special notes for your reviewer:

Release note:

if (re)sources are added to a component archive the access method is
created for validation. This must be possible, even if the technical
access requires crednetials, which are not yet provided.

Therefore operations requiring credentials (for private repos)
for the github access method have been moved to the methods
really requesting access to the content.
@mandelsoft mandelsoft requested a review from jensh007 October 12, 2022 06:55
@mandelsoft mandelsoft merged commit b9c2d5a into open-component-model:main Oct 12, 2022
@mandelsoft mandelsoft deleted the sources branch October 12, 2022 17:04
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