apfunk
diff --git a/‎LibGit2Sharp.Tests/MetaFixture.cs
Lines changed: 2 additions & 2 deletions b/‎LibGit2Sharp.Tests/MetaFixture.cs
Lines changed: 2 additions & 2 deletions
diff --git a/‎LibGit2Sharp/CloneOptions.cs
Lines changed: 7 additions & 1 deletion b/‎LibGit2Sharp/CloneOptions.cs
Lines changed: 7 additions & 1 deletion
diff --git a/‎LibGit2Sharp/Credentials.cs
Lines changed: 38 additions & 0 deletions b/‎LibGit2Sharp/Credentials.cs
Lines changed: 38 additions & 0 deletions
diff --git a/‎LibGit2Sharp/FetchOptions.cs
Lines changed: 7 additions & 1 deletion b/‎LibGit2Sharp/FetchOptions.cs
Lines changed: 7 additions & 1 deletion
diff --git a/‎LibGit2Sharp/Network.cs
Lines changed: 20 additions & 1 deletion b/‎LibGit2Sharp/Network.cs
Lines changed: 20 additions & 1 deletion
diff --git a/‎LibGit2Sharp/PushOptions.cs
Lines changed: 8 additions & 2 deletions b/‎LibGit2Sharp/PushOptions.cs
Lines changed: 8 additions & 2 deletions
diff --git a/‎LibGit2Sharp/RemoteCallbacks.cs
Lines changed: 13 additions & 1 deletion b/‎LibGit2Sharp/RemoteCallbacks.cs
Lines changed: 13 additions & 1 deletion
@@ -109,12 +109,12 @@ public void TypesInLibGit2SharpMustBeExtensibleInATestingContext()
         }
 
         [Fact]
-        public void LibGit2SharpInterfacesCoverAllPublicMembers()
+        public void LibGit2SharpPublicInterfacesCoverAllPublicMembers()
         {
             var methodsMissingFromInterfaces =
                 from t in Assembly.GetAssembly(typeof(IRepository)).GetExportedTypes()
                 where !t.IsInterface
-                where t.GetInterfaces().Any(i => i.Namespace == typeof(IRepository).Namespace)
+                where t.GetInterfaces().Any(i => i.IsPublic && i.Namespace == typeof(IRepository).Namespace)
                 let interfaceTargetMethods = from i in t.GetInterfaces()
                                              from im in t.GetInterfaceMap(i).TargetMethods
                                              select im
 
@@ -7,7 +7,7 @@ namespace LibGit2Sharp
     /// <summary>
     /// Options to define clone behaviour
     /// </summary>
-    public sealed class CloneOptions : IConvertableToGitCheckoutOpts
+    public sealed class CloneOptions : IConvertableToGitCheckoutOpts, ICredentialsProvider
     {
         /// <summary>
         /// Creates default <see cref="CloneOptions"/> for a non-bare clone
@@ -41,6 +41,12 @@ public CloneOptions()
         /// <summary>
         /// Credentials to use for user/pass authentication
         /// </summary>
+        [Obsolete("This will be removed in future release. Use CredentialsProvider.")]
+        public Credentials Credentials { get; set; }
+
+        /// <summary>
+        /// Handler to generate <see cref="LibGit2Sharp.Credentials"/> for authentication.
+        /// </summary>
         public CredentialsHandler CredentialsProvider { get; set; }
 
         #region IConvertableToGitCheckoutOpts
 
@@ -1,5 +1,6 @@
 using System;
 using LibGit2Sharp.Core;
+using LibGit2Sharp.Handlers;
 
 namespace LibGit2Sharp
 {
@@ -19,4 +20,41 @@ public abstract class Credentials
         /// <returns>0 for success, &lt; 0 to indicate an error, &gt; 0 to indicate no credential was acquired.</returns>
         protected internal abstract int GitCredentialHandler(out IntPtr cred, IntPtr url, IntPtr usernameFromUrl, GitCredentialType types, IntPtr payload);
     }
+
+    internal interface ICredentialsProvider
+    {
+        /// <summary>
+        /// The <see cref="Credentials"/> to authenticate with during the push.
+        /// </summary>
+        [Obsolete("This will be removed in future release. Use CredentialsProvider.")]
+        Credentials Credentials { get; }
+
+        /// <summary>
+        /// Handler to generate <see cref="LibGit2Sharp.Credentials"/> for authentication.
+        /// </summary>
+        CredentialsHandler CredentialsProvider { get; }
+    }
+
+    internal static class CredentialsProviderExtensions
+    {
+        public static CredentialsHandler GetCredentialsHandler(this ICredentialsProvider provider)
+        {
+            if (provider == null)
+            {
+                return null;
+            }
+
+            if (provider.CredentialsProvider != null)
+            {
+                return provider.CredentialsProvider;
+            }
+
+            if (provider.Credentials == null)
+            {
+                return null;
+            }
+
+            return (url, user, type) => provider.Credentials;
+        }
+    }
 }
@@ -6,7 +6,7 @@ namespace LibGit2Sharp
     /// <summary>
     /// Collection of parameters controlling Fetch behavior.
     /// </summary>
-    public sealed class FetchOptions
+    public sealed class FetchOptions : ICredentialsProvider
     {
         /// <summary>
         /// Specifies the tag-following behavior of the fetch operation.
@@ -42,6 +42,12 @@ public sealed class FetchOptions
         /// <summary>
         /// Credentials to use for username/password authentication.
         /// </summary>
+        [Obsolete("This will be removed in future release. Use CredentialsProvider.")]
+        public Credentials Credentials { get; set; }
+
+        /// <summary>
+        /// Handler to generate <see cref="LibGit2Sharp.Credentials"/> for authentication.
+        /// </summary>
         public CredentialsHandler CredentialsProvider { get; set; }
     }
 }
@@ -36,6 +36,25 @@ public virtual RemoteCollection Remotes
             get { return remotes.Value; }
         }
 
+        /// <summary>
+        /// List references in a <see cref="Remote"/> repository.
+        /// <para>
+        /// When the remote tips are ahead of the local ones, the retrieved
+        /// <see cref="DirectReference"/>s may point to non existing
+        /// <see cref="GitObject"/>s in the local repository. In that
+        /// case, <see cref="DirectReference.Target"/> will return <c>null</c>.
+        /// </para>
+        /// </summary>
+        /// <param name="remote">The <see cref="Remote"/> to list from.</param>
+        /// <param name="credentials">The optional <see cref="Credentials"/> used to connect to remote repository.</param>
+        /// <returns>The references in the <see cref="Remote"/> repository.</returns>
+        [Obsolete("This will be removed in future release. Use the overload ListReferences(Remote, CredentialsHandler).")]
+        public virtual IEnumerable<DirectReference> ListReferences(Remote remote, Credentials credentials)
+        {
+            return ListReferences(remote,
+                                  credentials == null ? null : new CredentialsHandler((url, user, type) => credentials));
+        }
+
         /// <summary>
         /// List references in a <see cref="Remote"/> repository.
         /// <para>
@@ -272,7 +291,7 @@ public virtual void Push(
             // Load the remote.
             using (RemoteSafeHandle remoteHandle = Proxy.git_remote_load(repository.Handle, remote.Name, true))
             {
-                var callbacks = new RemoteCallbacks(null, null, null, pushOptions.CredentialsProvider);
+                var callbacks = new RemoteCallbacks(null, null, null, pushOptions);
                 GitRemoteCallbacks gitCallbacks = callbacks.GenerateCallbacks();
                 Proxy.git_remote_set_callbacks(remoteHandle, ref gitCallbacks);
 
 
@@ -6,10 +6,16 @@ namespace LibGit2Sharp
     /// <summary>
     /// Collection of parameters controlling Push behavior.
     /// </summary>
-    public sealed class PushOptions
+    public sealed class PushOptions : ICredentialsProvider
     {
         /// <summary>
-        /// The <see cref="Credentials"/> to authenticate with during the push.
+        /// The <see cref="LibGit2Sharp.Credentials"/> to authenticate with during the push.
+        /// </summary>
+        [Obsolete("This will be removed in future release. Use CredentialsProvider.")]
+        public Credentials Credentials { get; set; }
+
+        /// <summary>
+        /// Handler to generate <see cref="LibGit2Sharp.Credentials"/> for authentication.
         /// </summary>
         public CredentialsHandler CredentialsProvider { get; set; }
 
 
@@ -12,6 +12,18 @@ namespace LibGit2Sharp
     /// </summary>
     internal class RemoteCallbacks
     {
+        internal RemoteCallbacks(
+            ProgressHandler onProgress = null,
+            TransferProgressHandler onDownloadProgress = null,
+            UpdateTipsHandler onUpdateTips = null,
+            ICredentialsProvider credentialsProvider = null)
+        {
+            Progress = onProgress;
+            DownloadTransferProgress = onDownloadProgress;
+            UpdateTips = onUpdateTips;
+            CredentialsProvider = credentialsProvider.GetCredentialsHandler();
+        }
+
         internal RemoteCallbacks(
             ProgressHandler onProgress = null,
             TransferProgressHandler onDownloadProgress = null,
@@ -30,7 +42,7 @@ internal RemoteCallbacks(FetchOptions fetchOptions)
             Progress = fetchOptions.OnProgress;
             DownloadTransferProgress = fetchOptions.OnTransferProgress;
             UpdateTips = fetchOptions.OnUpdateTips;
-            CredentialsProvider = fetchOptions.CredentialsProvider;
+            CredentialsProvider = fetchOptions.GetCredentialsHandler();
         }
 
         #region Delegates