Description of problem: Cloning from git locally (URL starts with git://) works correctly. However, the user is unable to push for security reasons. By default, this should be disabled as it will be frustrating for users who just successfully cloned to now be unable to using the exact same URL. Version-Release number of selected component (if applicable): 6.0.0+ How reproducible: Consistent Actual results: User cannot push using git:// Expected results: User cannot clone using git://, either Additional info:
UberFire master updated to disable git daemon by default: (master) http://github.com/uberfire/uberfire/commit/7aa4fc359
although I understand that it might be confusing to some users who by mistake clone over git instead of ssh but the change reveled another (rather serious) issue - cluster synchronization will not work at all as ssh requires authentication and background tasks are not able to do so. I would vote for reverting this commit to keep the same behavior for cluster as in 6.0. Thoughts?
Would we be able to use a configurable shared ssh login, one that only background processes use? I wouldn't be against enabling the git daemon again as an implementation detail, but I'd rather the user not be directed to clone from it if they can't push. That means removing the options to copy the git address from the UI.
I guess if we provide information for the user that git is read only and ssh is read/write that should provide enough information to choose the right one. wdyt? not sure I understand your question about ssh - behavior should not change for ssh at all.
Specifying git as read only would be sufficient.
BPMS 6.1.0 ER2: Cloned git://git.app.eng.bos.redhat.com/bpms-assets.git from UI. No warning or sign that it is readonly.
If this failed QA, please move it to ASSIGN, otherwise developer won't notice this. (In reply to Pavel Kralik from comment #7) > BPMS 6.1.0 ER2: > Cloned git://git.app.eng.bos.redhat.com/bpms-assets.git from UI. No warning > or sign that it is readonly.
If you clone the any repo from UI it won't be readonly. The readonly mentioned here is related to git daemon.
BPMS 6.1.0.ER6