Skip to content

Commit 2f6ce7c

Browse files
authored
Merge pull request #3401 from mansellan/next
Fixes issues with GitHub authentication crashing source control on push.
2 parents fde7744 + a243ad3 commit 2f6ce7c

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

RetailCoder.VBE/UI/SourceControl/SourceControlProviderFactory.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,6 @@ public interface ISourceControlProviderFactory
77
{
88
ISourceControlProvider CreateProvider(IVBProject project);
99
ISourceControlProvider CreateProvider(IVBProject project, IRepository repository);
10-
ISourceControlProvider CreateProvider(IVBProject isAny, IRepository repository, SecureCredentials secureCredentials);
10+
ISourceControlProvider CreateProvider(IVBProject project, IRepository repository, SecureCredentials secureCredentials);
1111
}
1212
}

Rubberduck.SourceControl/GitProvider.cs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -52,13 +52,13 @@ public GitProvider(IVBProject project, IRepository repository, string userName,
5252
_credentialsHandler = (url, user, cred) => _credentials;
5353
}
5454

55-
public GitProvider(IVBProject project, IRepository repository, ICredentials<SecureString> credentials)
55+
public GitProvider(IVBProject project, IRepository repository, ICredentials<SecureString> secureCredentials)
5656
: this(project, repository)
5757
{
5858
_credentials = new SecureUsernamePasswordCredentials()
5959
{
60-
Username = credentials.Username,
61-
Password = credentials.Password
60+
Username = secureCredentials.Username,
61+
Password = secureCredentials.Password
6262
};
6363

6464
_credentialsHandler = (url, user, cred) => _credentials;

0 commit comments

Comments
 (0)