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

chore: Fix Gitlab raw file location url #363

Merged
merged 1 commit into from
Oct 3, 2022
Merged

chore: Fix Gitlab raw file location url #363

merged 1 commit into from
Oct 3, 2022

Conversation

vinokurig
Copy link
Contributor

@vinokurig vinokurig commented Sep 30, 2022

Signed-off-by: Igor Vinokur ivinokur@redhat.com

What does this PR do?

  • Rework the Gitlab raw file url to return a public raw file url instead of an API request.
  • Trim the gitlabEndpoint if it ends with /

Screenshot/screencast of this PR

What issues does this PR fix or reference?

fixes eclipse-che/che#21696

How to test this PR?

Try to create a factory from a public Gitlab repo without the Gitlab OAuth being configured.

PR Checklist

As the author of this Pull Request I made sure that:

Reviewers

Reviewers, please comment how you tested the PR when approving it.

Signed-off-by: Igor Vinokur <ivinokur@redhat.com>
@vinokurig vinokurig merged commit 5aa763a into main Oct 3, 2022
@vinokurig vinokurig deleted the che-21696 branch October 3, 2022 06:52
@che-bot che-bot added this to the 7.55 milestone Oct 3, 2022
vinokurig added a commit that referenced this pull request Oct 4, 2022
Rework the Gitlab raw file url function to return a public raw file url instead of an API request.
Trim the gitlabEndpoint if it ends with /
vinokurig added a commit that referenced this pull request Oct 4, 2022
Rework the Gitlab raw file url function to return a public raw file url instead of an API request.
Trim the gitlabEndpoint if it ends with /
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.

Factory creation from public GitLab repo fails
3 participants