-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
[master] submodule issue about https and user #62
Comments
Hey,
And I'm getting this error too
Is it relted too? |
@zivkaziv I guess changing url from https to ssh link to repo should help in your case. But yes, it is related. |
Thank you for the quick answer.. |
I think |
Sorry for being so junior:) |
Do you have this issue on Generally with In my case the solution was |
So do you mean to set the step something like this?
And yes. |
yes, or |
Well.. Unfortunately it didn't work.. It seems that I have to use SSH for it |
@zivkaziv, I've been trying to reproduce your issue but have been unable to do so. Could you please try the following: - uses: actions/checkout@v1
with:
repository: myAccount/myRepository #For example thboop/test-repo
ref: refs/heads/master
token: ${{ secrets.GitHub_PAT }} and make sure you have the secret |
Thank you @thboop for your response.
Do I need to modify something on the private repository\organization in order to support it? |
Hey @zivkaziv, you need add your pat as a secret named |
The point is the user has to be added in https url, because of the error:
But when add URL with a
owner@
and other user (not an owner) will want to dogit submodule update
, then he will be prompted aboutowner
password.So using current method of authorisation with
token
for user which is set in submodule HTTPS URL is not functional.How to solve it? Deploy key? Let set also
user
as parameter in action?Just discovered it works on
actions/checkout@v1
. But issue still valid for@master
and generally worth to implement deploy keys.The text was updated successfully, but these errors were encountered: