Bug 1299619

Summary: Business Central Repo Clone Fails To Clone SCP Style SSH URLs
Product: [Retired] JBoss BPMS Platform 6 Reporter: Justin Holmes <jholmes>
Component: Business CentralAssignee: manstis
Status: CLOSED EOL QA Contact: Tomas David <tdavid>
Severity: unspecified Docs Contact:
Priority: high    
Version: 6.2.0CC: alazarot, hmiura, kverlaen, manstis, mproctor, tdavid
Target Milestone: ER1   
Target Release: 6.3.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
: 1299870 (view as bug list) Environment:
Last Closed: 2020-03-27 20:11:12 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1299870    
Attachments:
Description Flags
Error screen shot
none
GithHub Recommended none

Description Justin Holmes 2016-01-18 18:58:44 UTC
Created attachment 1115946 [details]
Error screen  shot

Description of problem:

Git supports two styles of URI's for SSH protocol. Business Central fails with "invalid URL format" when using the SCP style for SSH. This problematic because github by default uses the SCP style for SSH and does not seem to support the SSH style URL anymore.


Version-Release number of selected component (if applicable):
6.2.0 GA

How reproducible:
Everytime 

Steps to Reproduce:
1) Go to the admin tab
2) Click repositories->Clone Repository
3) Fill out the form using any ssh URL protocol provided by the github UI. I used git:sherl0cks/TestKnowledgeRepo.git

Actual results:
See screenshot. Error with "invalid URL format"

Expected results:
It clones the repo just like https URLs.


Additional info:

Comment 2 Mark Proctor 2016-02-11 12:18:03 UTC
Created attachment 1123137 [details]
GithHub Recommended

This is a screenshot showing that business central aligns with the Github recommended standard.

Comment 4 mireynol 2016-02-11 15:46:41 UTC
Has there been a JIRA created for this. Do we have any what the target fix version is?

Comment 5 Justin Holmes 2016-02-11 20:03:21 UTC
RE: GitHub recommendation 

The goal here is to use oncommit githooks to sync with a remote git repo. If customers can accomplish this using an alternative approach, I am OK. That said, we have a few issues:

1) Many customers just use SSH, and most SSH providers for git (including GitHub & GitLab) use the scp style protocol. What github recommends and what customers actually do don't always align.
2) The current implementation of githooks from jGit does not provide a way (that I can find today) to have jGit perform the actual git operations. Instead, jGit delegates to the OS to execute the hook which in turn requires git on the OS to perform the operation. This means if you want to use an oncommit githook to push to a remote a repo, then you need to use ssh or some custom way of storing the username/password, as the username/password in jGit is not available.

Comment 6 Justin Holmes 2016-02-11 20:03:45 UTC
@Mike - 6.2 patch 2, which is early/mid March. You can tell because this issue blocks https://bugzilla.redhat.com/show_bug.cgi?id=1299870 which has the appropriate flag

Comment 10 Kris Verlaenen 2016-02-23 13:32:48 UTC
Keys without passphrase can be supported by relaxing the url validation and should work.

Comment 22 Tomas David 2016-03-17 13:00:28 UTC
Verified with BPMS 6.3.0.ER1.