carla icon indicating copy to clipboard operation
carla copied to clipboard

Documentation for mentioning github username and password should be included

Open dt1729 opened this issue 3 years ago • 6 comments

https://github.com/carla-simulator/carla/blob/8854804f4d7748e14d937ec763a2912823a7e5f5/Util/Docker/Prerequisites.Dockerfile#L5

We have to mention our username and github key here for the command to execute otherwise it gives Cloning into '/home/carla/UE4.26'... remote: Support for password authentication was removed on August 13, 2021. Please use a personal access token instead. remote: Please see https://github.blog/2020-12-15-token-authentication-requirements-for-git-operations/ for more information. fatal: Authentication failed for 'https://user:[email protected]/CarlaUnreal/UnrealEngine.git/' this error, I do not find any mention of this in any documentation, should I submit a PR with the same? Or is it something that is already mentioned in some documentation?

Regards.

dt1729 avatar Jul 24 '22 07:07 dt1729

Hey @dt1729 how did you solve the login issue? I'd love to file a constructive PR if I knew how to authenticate.

For non-docker I know how to create this GitHub auth token stuff, but for the ue4-docker tooling, it's really a black box for me :sweat_smile:

Thanks for your help in advance :wink:

Bonifatius94 avatar Aug 05 '22 18:08 Bonifatius94

I think I figured it out. When prompted, you need to enter the GitHub access token as "password". This should fix it.

Bonifatius94 avatar Aug 05 '22 18:08 Bonifatius94

I did not get any prompt perhaps because it was a university PC and someone with University had a github info entered there. I had to edit the dockerfile and hardcode that information for my account. If it works on a fresh Linux install this issue shall be closed, we'd need someone to test this maybe?

dt1729 avatar Sep 12 '22 09:09 dt1729

I had to edit the dockerfile and hardcode that information for my account.

I think you're confusing ue4docker with Prerequisites.Dockerfile. You don't need to hardcode the username and password btw. Rather use the --build-arg argument of the docker build command to set an ARG in a Dockerfile 😉

I'm working on a new Dockerfile and wanna propose it soon. Stay tuned 😄

Bonifatius94 avatar Sep 12 '22 14:09 Bonifatius94

I can help write a test for it if you want.

dt1729 avatar Sep 15 '22 19:09 dt1729

@dt1729 Here are my Docker tools as PR: #5750. You're welcome to test it if you want :wink:

Bonifatius94 avatar Sep 15 '22 20:09 Bonifatius94

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Jun 02 '23 02:06 stale[bot]