-
-
Notifications
You must be signed in to change notification settings - Fork 26
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
SSL setup? #27
Comments
@nununo do you get the invalid certificate error when accessing the .git URL via browser? What is the error message in abapGit? what kind of login method is used when accessing the web part? try checking some of the following settings: perhaps try adding also try modifying the url in https://github.com/larshp/abapGit/wiki/ZABAPGIT_TEST_SSL to see if it works, it might be easier to debug |
Solved. Everything is working now. I configured it in a way that the SSO is no longer in the middle. |
was it a setting in SICF for ZGIT? can you add a few notes on how to fix it if somebody else experiences the same issue in the future? |
Sure! What I did was:
And that was enough to make it work! I hope this helps others! It hadn't tried it before because I was misled to think that the issue was related to SSL. Question: why did you call the service ZGIT instead of ZABAPGITSERVER? It would have been more coherent with the project naming. |
just to have a shorter URL, my guess is it wont conflict with anything else, we can change it if you like |
I think name coherence is very important in any project. If someone finds this service called ZGIT it will not be obvious if it is related to abapGit or abapGitServer. It's ambiguous. I'd give it a more explicit name like ZABAPGITSERVER. If this is possible of course, otherwise... not a big deal ;) |
good idea |
fixed, SICF service renamed from ZGIT to ZABAPGITSERVER, note that ZGIT service has to be deleted manually after pulling |
see abapGit/abapGit#306
@nununo
The text was updated successfully, but these errors were encountered: