From 7f8786945bf5fa1af0ae0603cc7de2b29cab1a2a Mon Sep 17 00:00:00 2001
From: Anh-Duc Le <Anh-Duc_Le@outlook.com>
Date: Wed, 25 Apr 2018 14:39:44 +0200
Subject: [PATCH 01/21] Added README.md and comments for the following:

In order to run the ConsoleTests you'll first need the following:

* You need at least 5 changesets in your repository.
* You need at least 5 issues in your repository.
* You need at least 1 SSH Key on your user
---
 .gitignore                       | 3 ++-
 ConsoleTests/ConsoleTests.csproj | 3 +++
 ConsoleTests/Program.cs          | 2 ++
 ConsoleTests/README.md           | 9 +++++++++
 4 files changed, 16 insertions(+), 1 deletion(-)
 create mode 100644 ConsoleTests/README.md

diff --git a/.gitignore b/.gitignore
index e115e009..9404e7fa 100644
--- a/.gitignore
+++ b/.gitignore
@@ -21,4 +21,5 @@ obj/
 _ReSharper*/
 *sln.dotsettings
 /packages/
-!packages/repositories.config
\ No newline at end of file
+!packages/repositories.config
+/.vs/SharpBucket/v15/Server/sqlite3
diff --git a/ConsoleTests/ConsoleTests.csproj b/ConsoleTests/ConsoleTests.csproj
index 5e715486..bc64a762 100644
--- a/ConsoleTests/ConsoleTests.csproj
+++ b/ConsoleTests/ConsoleTests.csproj
@@ -45,6 +45,9 @@
       <Name>SharpBucket</Name>
     </ProjectReference>
   </ItemGroup>
+  <ItemGroup>
+    <None Include="README.md" />
+  </ItemGroup>
   <Import Project="$(MSBuildToolsPath)\Microsoft.CSharp.targets" />
   <!-- To modify your build process, add your task inside one of the targets below and uncomment it. 
        Other similar extension points exist, see Microsoft.Common.targets.
diff --git a/ConsoleTests/Program.cs b/ConsoleTests/Program.cs
index 0aabb717..a0ba0359 100644
--- a/ConsoleTests/Program.cs
+++ b/ConsoleTests/Program.cs
@@ -116,6 +116,7 @@ private static void TestIssuesEndPoint(SharpBucketV1 sharpBucket)
         {
             var issuesResource = sharpBucket.RepositoriesEndPoint(accountName, repository).IssuesResource();
 
+            // TODO: You need at least 5 issues in the test repository.
             int ISSUE_ID = 5;
             // Issues
             var issues = issuesResource.ListIssues();
@@ -180,6 +181,7 @@ private static void TestRepositoriesEndPoint(SharpBucketV1 sharpBucket)
             var newPage = new Wiki { data = "Hello to my new page" };
             var newWiki = repositoriesEndPoint.PostWiki(newPage, "NewPage");
             var changeSet = repositoriesEndPoint.ListChangeset();
+            // TODO: You need at least 5 changesets in your repository.
             var change = changeSet.changesets[4];
             var getChange = repositoriesEndPoint.GetChangeset(change.node);
             var diffStats = repositoriesEndPoint.GetChangesetDiffstat(change.node);
diff --git a/ConsoleTests/README.md b/ConsoleTests/README.md
new file mode 100644
index 00000000..7c726704
--- /dev/null
+++ b/ConsoleTests/README.md
@@ -0,0 +1,9 @@
+# ConsoleTests
+
+## Prerequisites
+
+In order to run the ConsoleTests you'll first need the following:
+
+* You need at least 5 changesets in your repository.
+* You need at least 5 issues in your repository.
+* You need at least 1 SSH Key on your user

From f10f365dc5d2885d4c8ef472e63ddb64d63fd614 Mon Sep 17 00:00:00 2001
From: Anh-Duc Le <Anh-Duc_Le@outlook.com>
Date: Wed, 25 Apr 2018 14:49:39 +0200
Subject: [PATCH 02/21] Added Serilog nuget and upgraded the NET Framework from
 4.0 => 4.5

---
 ConsoleTests/ConsoleTests.csproj            |  6 +++++-
 ConsoleTests/app.config                     |  3 +++
 SharpBucket/Authentication/IAuthenticate.cs | 19 +++++++++++++++++++
 SharpBucket/SharpBucket.csproj              | 18 ++++++++++++------
 SharpBucket/packages.config                 |  6 +++---
 SharpBucketTests/SharpBucketTests.csproj    | 21 ++++++++++-----------
 SharpBucketTests/packages.config            |  5 ++---
 7 files changed, 54 insertions(+), 24 deletions(-)
 create mode 100644 ConsoleTests/app.config

diff --git a/ConsoleTests/ConsoleTests.csproj b/ConsoleTests/ConsoleTests.csproj
index bc64a762..324bcb4c 100644
--- a/ConsoleTests/ConsoleTests.csproj
+++ b/ConsoleTests/ConsoleTests.csproj
@@ -9,8 +9,9 @@
     <AppDesignerFolder>Properties</AppDesignerFolder>
     <RootNamespace>ConsoleTests</RootNamespace>
     <AssemblyName>ConsoleTests</AssemblyName>
-    <TargetFrameworkVersion>v4.0</TargetFrameworkVersion>
+    <TargetFrameworkVersion>v4.5</TargetFrameworkVersion>
     <FileAlignment>512</FileAlignment>
+    <TargetFrameworkProfile />
   </PropertyGroup>
   <PropertyGroup Condition=" '$(Configuration)|$(Platform)' == 'Debug|AnyCPU' ">
     <PlatformTarget>AnyCPU</PlatformTarget>
@@ -21,6 +22,7 @@
     <DefineConstants>DEBUG;TRACE</DefineConstants>
     <ErrorReport>prompt</ErrorReport>
     <WarningLevel>4</WarningLevel>
+    <Prefer32Bit>false</Prefer32Bit>
   </PropertyGroup>
   <PropertyGroup Condition=" '$(Configuration)|$(Platform)' == 'Release|AnyCPU' ">
     <PlatformTarget>AnyCPU</PlatformTarget>
@@ -30,6 +32,7 @@
     <DefineConstants>TRACE</DefineConstants>
     <ErrorReport>prompt</ErrorReport>
     <WarningLevel>4</WarningLevel>
+    <Prefer32Bit>false</Prefer32Bit>
   </PropertyGroup>
   <ItemGroup>
     <Reference Include="System" />
@@ -46,6 +49,7 @@
     </ProjectReference>
   </ItemGroup>
   <ItemGroup>
+    <None Include="app.config" />
     <None Include="README.md" />
   </ItemGroup>
   <Import Project="$(MSBuildToolsPath)\Microsoft.CSharp.targets" />
diff --git a/ConsoleTests/app.config b/ConsoleTests/app.config
new file mode 100644
index 00000000..51278a45
--- /dev/null
+++ b/ConsoleTests/app.config
@@ -0,0 +1,3 @@
+<?xml version="1.0" encoding="utf-8"?>
+<configuration>
+<startup><supportedRuntime version="v4.0" sku=".NETFramework,Version=v4.5"/></startup></configuration>
diff --git a/SharpBucket/Authentication/IAuthenticate.cs b/SharpBucket/Authentication/IAuthenticate.cs
index cea2f5e4..7dc5244d 100644
--- a/SharpBucket/Authentication/IAuthenticate.cs
+++ b/SharpBucket/Authentication/IAuthenticate.cs
@@ -1,5 +1,6 @@
 using System.Collections.Generic;
 using RestSharp;
+using Serilog;
 
 namespace SharpBucket.Authentication
 {
@@ -13,5 +14,23 @@ public virtual T GetResponse<T>(string url, Method method, T body, IDictionary<s
             var generic = executeMethod.MakeGenericMethod(typeof(T));
             return (T)generic.Invoke(this, new object[] { url, method, body, client, requestParameters });
         }
+
+        /// <summary>
+        /// With SeriLog logging
+        /// </summary>
+        /// <typeparam name="T"></typeparam>
+        /// <param name="logger"></param>
+        /// <param name="url"></param>
+        /// <param name="method"></param>
+        /// <param name="body"></param>
+        /// <param name="requestParameters"></param>
+        /// <returns></returns>
+        public virtual T GetResponse<T>(ILogger logger, string url, Method method, T body, IDictionary<string, object> requestParameters)
+        {
+            var executeMethod = typeof(RequestExecutor).GetMethod("ExecuteRequestWithLogging");
+            var generic = executeMethod?.MakeGenericMethod(typeof(T));
+            return (T)generic?.Invoke(this, new object[] { logger,url, method, body, client, requestParameters
+            });
+        }
     }
 }
\ No newline at end of file
diff --git a/SharpBucket/SharpBucket.csproj b/SharpBucket/SharpBucket.csproj
index c550d455..3da67654 100644
--- a/SharpBucket/SharpBucket.csproj
+++ b/SharpBucket/SharpBucket.csproj
@@ -9,8 +9,9 @@
     <AppDesignerFolder>Properties</AppDesignerFolder>
     <RootNamespace>SharpBucket</RootNamespace>
     <AssemblyName>SharpBucket</AssemblyName>
-    <TargetFrameworkVersion>v4.0</TargetFrameworkVersion>
+    <TargetFrameworkVersion>v4.5</TargetFrameworkVersion>
     <FileAlignment>512</FileAlignment>
+    <TargetFrameworkProfile />
   </PropertyGroup>
   <PropertyGroup Condition=" '$(Configuration)|$(Platform)' == 'Debug|AnyCPU' ">
     <DebugSymbols>true</DebugSymbols>
@@ -20,6 +21,7 @@
     <DefineConstants>DEBUG;TRACE</DefineConstants>
     <ErrorReport>prompt</ErrorReport>
     <WarningLevel>4</WarningLevel>
+    <Prefer32Bit>false</Prefer32Bit>
   </PropertyGroup>
   <PropertyGroup Condition=" '$(Configuration)|$(Platform)' == 'Release|AnyCPU' ">
     <DebugType>pdbonly</DebugType>
@@ -29,16 +31,18 @@
     <ErrorReport>prompt</ErrorReport>
     <WarningLevel>4</WarningLevel>
     <DocumentationFile>bin\Release\SharpBucket.XML</DocumentationFile>
+    <Prefer32Bit>false</Prefer32Bit>
   </PropertyGroup>
   <ItemGroup>
     <Reference Include="Microsoft.CSharp" />
     <Reference Include="Newtonsoft.Json, Version=7.0.0.0, Culture=neutral, PublicKeyToken=30ad4fe6b2a6aeed, processorArchitecture=MSIL">
-      <HintPath>..\packages\Newtonsoft.Json.7.0.1\lib\net40\Newtonsoft.Json.dll</HintPath>
-      <Private>True</Private>
+      <HintPath>..\packages\Newtonsoft.Json.7.0.1\lib\net45\Newtonsoft.Json.dll</HintPath>
     </Reference>
     <Reference Include="RestSharp, Version=105.2.3.0, Culture=neutral, processorArchitecture=MSIL">
-      <HintPath>..\packages\RestSharp.105.2.3\lib\net4\RestSharp.dll</HintPath>
-      <Private>True</Private>
+      <HintPath>..\packages\RestSharp.105.2.3\lib\net45\RestSharp.dll</HintPath>
+    </Reference>
+    <Reference Include="Serilog, Version=2.0.0.0, Culture=neutral, PublicKeyToken=24c2f752a8e58a10, processorArchitecture=MSIL">
+      <HintPath>..\packages\Serilog.2.6.0\lib\net45\Serilog.dll</HintPath>
     </Reference>
     <Reference Include="System" />
     <Reference Include="System.Core" />
@@ -154,7 +158,9 @@
     <Compile Include="V1\SharpBucketV1.cs" />
   </ItemGroup>
   <ItemGroup>
-    <None Include="packages.config" />
+    <None Include="packages.config">
+      <SubType>Designer</SubType>
+    </None>
     <None Include="SharpBucket.nuspec">
       <SubType>Designer</SubType>
     </None>
diff --git a/SharpBucket/packages.config b/SharpBucket/packages.config
index d5dbdcff..37b13519 100644
--- a/SharpBucket/packages.config
+++ b/SharpBucket/packages.config
@@ -1,6 +1,6 @@
 <?xml version="1.0" encoding="utf-8"?>
-
 <packages>
-  <package id="Newtonsoft.Json" version="7.0.1" targetFramework="net40" />
-  <package id="RestSharp" version="105.2.3" targetFramework="net40" />
+  <package id="Newtonsoft.Json" version="7.0.1" targetFramework="net45" />
+  <package id="RestSharp" version="105.2.3" targetFramework="net45" />
+  <package id="Serilog" version="2.6.0" targetFramework="net45" />
 </packages>
\ No newline at end of file
diff --git a/SharpBucketTests/SharpBucketTests.csproj b/SharpBucketTests/SharpBucketTests.csproj
index 9febdacb..57cf1a27 100644
--- a/SharpBucketTests/SharpBucketTests.csproj
+++ b/SharpBucketTests/SharpBucketTests.csproj
@@ -8,7 +8,7 @@
     <AppDesignerFolder>Properties</AppDesignerFolder>
     <RootNamespace>SharBucketTests</RootNamespace>
     <AssemblyName>SharBucketTests</AssemblyName>
-    <TargetFrameworkVersion>v4.0</TargetFrameworkVersion>
+    <TargetFrameworkVersion>v4.5</TargetFrameworkVersion>
     <FileAlignment>512</FileAlignment>
     <ProjectTypeGuids>{3AC096D0-A1C2-E12C-1390-A8335801FDAB};{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}</ProjectTypeGuids>
     <VisualStudioVersion Condition="'$(VisualStudioVersion)' == ''">10.0</VisualStudioVersion>
@@ -20,6 +20,9 @@
     </FileUpgradeFlags>
     <OldToolsVersion>4.0</OldToolsVersion>
     <UpgradeBackupLocation />
+    <TargetFrameworkProfile />
+    <NuGetPackageImportStamp>
+    </NuGetPackageImportStamp>
   </PropertyGroup>
   <PropertyGroup Condition=" '$(Configuration)|$(Platform)' == 'Debug|AnyCPU' ">
     <DebugSymbols>true</DebugSymbols>
@@ -29,6 +32,7 @@
     <DefineConstants>DEBUG;TRACE</DefineConstants>
     <ErrorReport>prompt</ErrorReport>
     <WarningLevel>4</WarningLevel>
+    <Prefer32Bit>false</Prefer32Bit>
   </PropertyGroup>
   <PropertyGroup Condition=" '$(Configuration)|$(Platform)' == 'Release|AnyCPU' ">
     <DebugType>pdbonly</DebugType>
@@ -37,15 +41,14 @@
     <DefineConstants>TRACE</DefineConstants>
     <ErrorReport>prompt</ErrorReport>
     <WarningLevel>4</WarningLevel>
+    <Prefer32Bit>false</Prefer32Bit>
   </PropertyGroup>
   <ItemGroup>
     <Reference Include="nunit.framework, Version=3.9.0.0, Culture=neutral, PublicKeyToken=2638cd05610744eb, processorArchitecture=MSIL">
-      <HintPath>..\packages\NUnit.3.9.0\lib\net40\nunit.framework.dll</HintPath>
-      <Private>True</Private>
+      <HintPath>..\packages\NUnit.3.9.0\lib\net45\nunit.framework.dll</HintPath>
     </Reference>
     <Reference Include="Shouldly, Version=2.8.3.0, Culture=neutral, PublicKeyToken=6042cbcb05cbc941, processorArchitecture=MSIL">
       <HintPath>..\packages\Shouldly.2.8.3\lib\net40\Shouldly.dll</HintPath>
-      <Private>True</Private>
     </Reference>
     <Reference Include="System" />
     <Reference Include="System.Core">
@@ -75,13 +78,9 @@
     <Compile Include="V2\EndPoints\UsersEndPointTests.cs" />
   </ItemGroup>
   <ItemGroup>
-    <ProjectReference Include="..\SharpBucket\SharpBucket.csproj">
-      <Project>{f7c8fef8-4adb-4175-a4d8-2a0598b78786}</Project>
-      <Name>SharpBucket</Name>
-    </ProjectReference>
-  </ItemGroup>
-  <ItemGroup>
-    <None Include="packages.config" />
+    <None Include="packages.config">
+      <SubType>Designer</SubType>
+    </None>
   </ItemGroup>
   <Choose>
     <When Condition="'$(VisualStudioVersion)' == '10.0' And '$(IsCodedUITest)' == 'True'">
diff --git a/SharpBucketTests/packages.config b/SharpBucketTests/packages.config
index d3a98c57..8df699ab 100644
--- a/SharpBucketTests/packages.config
+++ b/SharpBucketTests/packages.config
@@ -1,6 +1,5 @@
 <?xml version="1.0" encoding="utf-8"?>
-
 <packages>
-  <package id="NUnit" version="3.9.0" targetFramework="net40" />
-  <package id="Shouldly" version="2.8.3" targetFramework="net40" />
+  <package id="NUnit" version="3.9.0" targetFramework="net45" />
+  <package id="Shouldly" version="2.8.3" targetFramework="net45" />
 </packages>
\ No newline at end of file

From bbe2e209b7043221518381f86ee1d0f2b5685cae Mon Sep 17 00:00:00 2001
From: Anh-Duc Le <Anh-Duc_Le@outlook.com>
Date: Wed, 25 Apr 2018 14:52:11 +0200
Subject: [PATCH 03/21] SharpBucket/Authentication/RequestExecutor.cs => Added
 ExecuteRequestWithLogging()

---
 SharpBucket/Authentication/RequestExecutor.cs | 64 +++++++++++++++++++
 1 file changed, 64 insertions(+)

diff --git a/SharpBucket/Authentication/RequestExecutor.cs b/SharpBucket/Authentication/RequestExecutor.cs
index 85a269ad..9cc81e90 100644
--- a/SharpBucket/Authentication/RequestExecutor.cs
+++ b/SharpBucket/Authentication/RequestExecutor.cs
@@ -3,6 +3,8 @@
 using System.Net;
 using RestSharp;
 using RestSharp.Deserializers;
+using Serilog;
+using SharpBucket.V2.Pocos;
 
 namespace SharpBucket.Authentication
 {
@@ -87,5 +89,67 @@ private static bool RequestingSimpleType<T>()
         {
             return typeof(T) == typeof(object);
         }
+
+
+        /// <summary>
+        /// With Logging
+        /// </summary>
+        /// <typeparam name="T"></typeparam>
+        /// <param name="logger"></param>
+        /// <param name="url"></param>
+        /// <param name="method"></param>
+        /// <param name="body"></param>
+        /// <param name="client"></param>
+        /// <param name="requestParameters"></param>
+        /// <returns></returns>
+        public static T ExecuteRequestWithLogging<T>(ILogger logger, string url, Method method, T body, RestClient client, IDictionary<string, object> requestParameters)
+            where T : new()
+        {
+            var request = new RestRequest(url, method);
+            if (requestParameters != null)
+            {
+                foreach (var requestParameter in requestParameters)
+                {
+                    request.AddParameter(requestParameter.Key, requestParameter.Value);
+                }
+            }
+
+            if (ShouldAddBody(method))
+            {
+                if (body.GetType() != typeof(Branch))
+                {
+                    request.RequestFormat = DataFormat.Json;
+                    request.AddBody(body);
+                }
+                else
+                {
+                    request.AddObject(body);
+                    request.AddHeader("Content-Type", "multipart/form-data");
+                }
+            }
+
+            //Fixed bug that prevents RestClient for adding custom headers to the request
+            //https://stackoverflow.com/questions/22229393/why-is-restsharp-addheaderaccept-application-json-to-a-list-of-item
+
+            client.ClearHandlers();
+
+            client.AddHandler("application/json", new JsonDeserializer());
+
+            var result = ExectueRequest<T>(method, client, request);
+
+            if (result.ErrorException != null)
+            {
+                throw new WebException("REST client encountered an error: " + result.ErrorMessage, result.ErrorException);
+            }
+            // This is a hack in order to allow this method to work for simple types as well
+            // one example of this is the GetRevisionRaw method
+            if (RequestingSimpleType<T>())
+            {
+                return result.Content as dynamic;
+            }
+
+            logger.Debug($"{result.StatusCode}: {result.StatusDescription}");
+            return result.Data;
+        }
     }
 }
\ No newline at end of file

From 06b508cf6950cf60bdd777234b5e6f72423733ad Mon Sep 17 00:00:00 2001
From: Anh-Duc Le <Anh-Duc_Le@outlook.com>
Date: Wed, 25 Apr 2018 14:59:50 +0200
Subject: [PATCH 04/21] Added ILogger parameter to Sharpbucket.cs methods
 (Send/Get/Post/Put/Delete)

---
 SharpBucket/SharpBucket.cs | 82 ++++++++++++++++++++++++++++++++++++++
 1 file changed, 82 insertions(+)

diff --git a/SharpBucket/SharpBucket.cs b/SharpBucket/SharpBucket.cs
index 7b5fbab4..46c8dd5c 100644
--- a/SharpBucket/SharpBucket.cs
+++ b/SharpBucket/SharpBucket.cs
@@ -2,6 +2,7 @@
 using System.Collections.Generic;
 using System.Net;
 using RestSharp;
+using Serilog;
 using SharpBucket.Authentication;
 using SharpBucket.Utility;
 
@@ -119,6 +120,32 @@ private T Send<T>(T body, Method method, string overrideUrl = null, IDictionary<
             return response;
         }
 
+        /// <summary>
+        /// With Logging
+        /// </summary>
+        /// <typeparam name="T"></typeparam>
+        /// <param name="logger"></param>
+        /// <param name="body"></param>
+        /// <param name="method"></param>
+        /// <param name="overrideUrl"></param>
+        /// <param name="requestParameters"></param>
+        /// <returns></returns>
+        private T Send<T>(ILogger logger, T body, Method method, string overrideUrl = null, IDictionary<string, object> requestParameters = null)
+        {
+            var relativeUrl = overrideUrl;
+            T response;
+            try
+            {
+                response = authenticator.GetResponse(logger, relativeUrl, method, body, requestParameters);
+            }
+            catch (WebException ex)
+            {
+                Console.WriteLine(ex.Message);
+                response = default(T);
+            }
+            return response;
+        }
+        
         internal T Get<T>(T body, string overrideUrl, object requestParameters = null)
         {
             //Convert to dictionary to avoid refactoring the Send method.
@@ -126,19 +153,74 @@ internal T Get<T>(T body, string overrideUrl, object requestParameters = null)
             return Send(body, Method.GET, overrideUrl, parameterDictionary);
         }
 
+        /// <summary>
+        /// With Logging
+        /// </summary>
+        /// <typeparam name="T"></typeparam>
+        /// <param name="logger"></param>
+        /// <param name="body"></param>
+        /// <param name="overrideUrl"></param>
+        /// <param name="requestParameters"></param>
+        /// <returns></returns>
+        internal T Get<T>(ILogger logger, T body, string overrideUrl, object requestParameters = null)
+        {
+            //Convert to dictionary to avoid refactoring the Send method.
+            var parameterDictionary = requestParameters.ToDictionary();
+            return Send(logger, body, Method.GET, overrideUrl, parameterDictionary);
+        }
+
         internal T Post<T>(T body, string overrideUrl)
         {
             return Send(body, Method.POST, overrideUrl);
         }
 
+        /// <summary>
+        /// With Logging
+        /// </summary>
+        /// <typeparam name="T"></typeparam>
+        /// <param name="logger"></param>
+        /// <param name="body"></param>
+        /// <param name="overrideUrl"></param>
+        /// <returns></returns>
+        internal T Post<T>(ILogger logger, T body, string overrideUrl)
+        {
+            return Send(logger, body, Method.POST, overrideUrl);
+        }
+
         internal T Put<T>(T body, string overrideUrl)
         {
             return Send(body, Method.PUT, overrideUrl);
         }
 
+        /// <summary>
+        /// With Logging
+        /// </summary>
+        /// <typeparam name="T"></typeparam>
+        /// <param name="logger"></param>
+        /// <param name="body"></param>
+        /// <param name="overrideUrl"></param>
+        /// <returns></returns>
+        internal T Put<T>(ILogger logger, T body, string overrideUrl)
+        {
+            return Send(logger, body, Method.PUT, overrideUrl);
+        }
+
         internal T Delete<T>(T body, string overrideUrl)
         {
             return Send(body, Method.DELETE, overrideUrl);
         }
+
+        /// <summary>
+        /// With Logging
+        /// </summary>
+        /// <typeparam name="T"></typeparam>
+        /// <param name="logger"></param>
+        /// <param name="body"></param>
+        /// <param name="overrideUrl"></param>
+        /// <returns></returns>
+        internal T Delete<T>(ILogger logger, T body, string overrideUrl)
+        {
+            return Send(logger, body, Method.DELETE, overrideUrl);
+        }
     }
 }
\ No newline at end of file

From c6bf2768509d9b5f14f506abb44b226704e4d8c3 Mon Sep 17 00:00:00 2001
From: Anh-Duc Le <Anh-Duc_Le@outlook.com>
Date: Wed, 25 Apr 2018 15:05:27 +0200
Subject: [PATCH 05/21] Added Serilog Logger to EndPoint

---
 SharpBucket/V2/EndPoints/EndPoint.cs | 81 ++++++++++++++++++++++++++++
 1 file changed, 81 insertions(+)

diff --git a/SharpBucket/V2/EndPoints/EndPoint.cs b/SharpBucket/V2/EndPoints/EndPoint.cs
index a088c107..94a1a69a 100644
--- a/SharpBucket/V2/EndPoints/EndPoint.cs
+++ b/SharpBucket/V2/EndPoints/EndPoint.cs
@@ -3,6 +3,7 @@
 using System.Collections.Generic;
 using System.Diagnostics;
 using System.Dynamic;
+using Serilog;
 
 namespace SharpBucket.V2.EndPoints
 {
@@ -57,6 +58,46 @@ private IEnumerable<List<TValue>> IteratePages<TValue>(string overrideUrl, int p
             while (!String.IsNullOrEmpty(response.next));
         }
 
+        /// <summary>
+        /// Generator that allows lazy access to paginated resources.
+        /// With logging.
+        /// </summary>
+        /// <typeparam name="TValue"></typeparam>
+        /// <param name="logger"></param>
+        /// <param name="overrideUrl"></param>
+        /// <param name="pageLen"></param>
+        /// <param name="requestParameters"></param>
+        /// <returns></returns>
+        private IEnumerable<List<TValue>> IteratePages<TValue>(ILogger logger, string overrideUrl, int pageLen = DEFAULT_PAGE_LEN,
+           IDictionary<string, object> requestParameters = null)
+        {
+            Debug.Assert(!String.IsNullOrEmpty(overrideUrl));
+            //Debug.Assert(!overrideUrl.Contains("?"));
+
+            if (requestParameters == null)
+            {
+                requestParameters = new Dictionary<string, object>();
+            }
+
+            requestParameters["pagelen"] = pageLen;
+
+            IteratorBasedPage<TValue> response;
+            int page = 1;
+            do
+            {
+                response = _sharpBucketV2.Get(logger, new IteratorBasedPage<TValue>(),
+                        overrideUrl.Replace(SharpBucketV2.BITBUCKET_URL, ""), requestParameters);
+                if (response == null)
+                {
+                    break;
+                }
+
+                yield return response.values;
+
+                requestParameters["page"] = ++page;
+            } while (!String.IsNullOrEmpty(response.next));
+        }
+
         /// <summary>
         /// Returns a list of paginated values.
         /// </summary>
@@ -94,5 +135,45 @@ protected List<TValue> GetPaginatedValues<TValue>(string overrideUrl, int max =
 
             return values;
         }
+
+        /// <summary>
+        /// Returns a list of paginated values with Logging.
+        /// </summary>
+        /// <typeparam name="TValue">The type of the value.</typeparam>
+        /// <param name="logger"></param>
+        /// <param name="overrideUrl">The override URL.</param>
+        /// <param name="max">Set to 0 for unlimited size.</param>
+        /// <param name="requestParameters"></param>
+        /// <returns></returns>
+        /// <exception cref="System.Net.WebException">Thrown when the server fails to respond.</exception>
+        protected List<TValue> GetPaginatedValues<TValue>(ILogger logger, string overrideUrl, int max = 0, IDictionary<string, object> requestParameters = null)
+        {
+            bool isMaxConstrained = max > 0;
+
+            int pageLen = (isMaxConstrained && max < DEFAULT_PAGE_LEN) ? max : DEFAULT_PAGE_LEN;
+
+            List<TValue> values = new List<TValue>();
+
+            foreach (var page in IteratePages<TValue>(logger, overrideUrl, pageLen, requestParameters))
+            {
+                if (page == null)
+                {
+                    break;
+                }
+
+                if (isMaxConstrained &&
+                    +values.Count + page.Count >= max)
+                {
+                    values.AddRange(page.GetRange(0, max - values.Count));
+                    Debug.Assert(values.Count == max);
+                    break;
+                }
+
+                values.AddRange(page);
+            }
+
+            return values;
+        }
+
     }
 }
\ No newline at end of file

From f9f10657ba957267006b0add534e4671767d45ba Mon Sep 17 00:00:00 2001
From: Anh-Duc Le <Anh-Duc_Le@outlook.com>
Date: Wed, 25 Apr 2018 15:36:21 +0200
Subject: [PATCH 06/21] Added logging to repositoriesEndPoint

---
 SharpBucket/SharpBucket.cs                    |   2 +-
 .../V2/EndPoints/RepositoriesEndPoint.cs      | 584 ++++++++++++++++++
 2 files changed, 585 insertions(+), 1 deletion(-)

diff --git a/SharpBucket/SharpBucket.cs b/SharpBucket/SharpBucket.cs
index 46c8dd5c..6acc8caf 100644
--- a/SharpBucket/SharpBucket.cs
+++ b/SharpBucket/SharpBucket.cs
@@ -145,7 +145,7 @@ private T Send<T>(ILogger logger, T body, Method method, string overrideUrl = nu
             }
             return response;
         }
-        
+
         internal T Get<T>(T body, string overrideUrl, object requestParameters = null)
         {
             //Convert to dictionary to avoid refactoring the Send method.
diff --git a/SharpBucket/V2/EndPoints/RepositoriesEndPoint.cs b/SharpBucket/V2/EndPoints/RepositoriesEndPoint.cs
index 681462a4..6ce02e7c 100644
--- a/SharpBucket/V2/EndPoints/RepositoriesEndPoint.cs
+++ b/SharpBucket/V2/EndPoints/RepositoriesEndPoint.cs
@@ -1,5 +1,6 @@
 using System;
 using System.Collections.Generic;
+using Serilog;
 using SharpBucket.V2.Pocos;
 using Comment = SharpBucket.V2.Pocos.Comment;
 using Repository = SharpBucket.V2.Pocos.Repository;
@@ -35,6 +36,22 @@ public List<Repository> ListRepositories(string accountName, int max = 0)
             return GetPaginatedValues<Repository>(overrideUrl, max);
         }
 
+        /// <summary>
+        /// With Logging.
+        /// List of repositories associated with an account and project. If the caller is properly authenticated and authorized, 
+        /// this method returns a collection containing public and private repositories. 
+        /// Otherwise, this method returns a collection of the public repositories. 
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="accountName">The account whose repositories you wish to get.</param>
+        /// <param name="max">The maximum number of items to return. 0 returns all items.</param>
+        /// <returns></returns>
+        public List<Repository> ListRepositories(ILogger logger, string accountName, int max = 0)
+        {
+            var overrideUrl = _baseUrl + accountName + "/";
+            return GetPaginatedValues<Repository>(logger, overrideUrl, max);
+        }
+
         /// <summary>
         /// List of all the public repositories on Bitbucket.  This produces a paginated response. 
         /// Pagination only goes forward (it's not possible to navigate to previous pages) and navigation is done by following the URL for the next page.
@@ -47,6 +64,19 @@ public List<Repository> ListPublicRepositories(int max = 0)
             return GetPaginatedValues<Repository>(_baseUrl, max);
         }
 
+        /// <summary>
+        /// With Logging.
+        /// List of all the public repositories on Bitbucket.  This produces a paginated response. 
+        /// Pagination only goes forward (it's not possible to navigate to previous pages) and navigation is done by following the URL for the next page.
+        /// The returned repositories are ordered by creation date, oldest repositories first. Only public repositories are returned.
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="max">The maximum number of items to return. 0 returns all items.</param>
+        /// <returns></returns>
+        public List<Repository> ListPublicRepositories(ILogger logger, int max = 0)
+        {
+            return GetPaginatedValues<Repository>(logger, _baseUrl, max);
+        }
         #endregion
 
         #region Repository resource
@@ -71,24 +101,78 @@ internal Repository GetRepository(string accountName, string repository)
             return _sharpBucketV2.Get(new Repository(), overrideUrl);
         }
 
+        /// <summary>
+        /// With Logging
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="accountName"></param>
+        /// <param name="repository"></param>
+        /// <returns></returns>
+        internal Repository GetRepository(ILogger logger, string accountName, string repository)
+        {
+            var overrideUrl = GetRepositoryUrl(accountName, repository, null);
+            return _sharpBucketV2.Get(logger, new Repository(), overrideUrl);
+        }
+
         internal Repository PutRepository(Repository repo, string accountName, string repository)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, null);
             return _sharpBucketV2.Put(repo, overrideUrl);
         }
 
+        /// <summary>
+        /// With Logging
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="repo"></param>
+        /// <param name="accountName"></param>
+        /// <param name="repository"></param>
+        /// <returns></returns>
+        internal Repository PutRepository(ILogger logger, Repository repo, string accountName, string repository)
+        {
+            var overrideUrl = GetRepositoryUrl(accountName, repository, null);
+            return _sharpBucketV2.Put(repo, overrideUrl);
+        }
+
         internal Repository PostRepository(Repository repo, string accountName)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repo.name, null);
             return _sharpBucketV2.Post(repo, overrideUrl);
         }
 
+        /// <summary>
+        /// With Logging
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="repo"></param>
+        /// <param name="accountName"></param>
+        /// <returns></returns>
+        internal Repository PostRepository(ILogger logger, Repository repo, string accountName)
+        {
+            var overrideUrl = GetRepositoryUrl(accountName, repo.name, null);
+            return _sharpBucketV2.Post(logger, repo, overrideUrl);
+        }
+
         internal Repository DeleteRepository(string accountName, string repository)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, null);
             return _sharpBucketV2.Delete(new Repository(), overrideUrl);
         }
 
+        /// <summary>
+        /// With Logging
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="accountName"></param>
+        /// <param name="repository"></param>
+        /// <returns></returns>
+        internal Repository DeleteRepository(ILogger logger, string accountName, string repository)
+        {
+            var overrideUrl = GetRepositoryUrl(accountName, repository, null);
+            return _sharpBucketV2.Delete(logger, new Repository(), overrideUrl);
+        }
+
+
         private string GetRepositoryUrl(string accountName, string repository, string append)
         {
             var format = _baseUrl + "{0}/{1}/{2}";
@@ -101,12 +185,40 @@ internal List<Watcher> ListWatchers(string accountName, string repository, int m
             return GetPaginatedValues<Watcher>(overrideUrl, max);
         }
 
+        /// <summary>
+        /// With Logging
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="accountName"></param>
+        /// <param name="repository"></param>
+        /// <param name="max"></param>
+        /// <returns></returns>
+        internal List<Watcher> ListWatchers(ILogger logger, string accountName, string repository, int max = 0)
+        {
+            var overrideUrl = GetRepositoryUrl(accountName, repository, "watchers");
+            return GetPaginatedValues<Watcher>(logger, overrideUrl, max);
+        }
+
         internal List<Fork> ListForks(string accountName, string repository, int max = 0)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "forks");
             return GetPaginatedValues<Fork>(overrideUrl, max);
         }
 
+        /// <summary>
+        /// With Logging
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="accountName"></param>
+        /// <param name="repository"></param>
+        /// <param name="max"></param>
+        /// <returns></returns>
+        internal List<Fork> ListForks(ILogger logger, string accountName, string repository, int max = 0)
+        {
+            var overrideUrl = GetRepositoryUrl(accountName, repository, "forks");
+            return GetPaginatedValues<Fork>(logger, overrideUrl, max);
+        }
+
         #endregion
 
         #region Pull Requests Resource
@@ -122,24 +234,80 @@ internal List<PullRequest> ListPullRequests(string accountName, string repositor
             return GetPaginatedValues<PullRequest>(overrideUrl, max);
         }
 
+        /// <summary>
+        /// With Logging
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="accountName"></param>
+        /// <param name="repository"></param>
+        /// <param name="max"></param>
+        /// <returns></returns>
+        internal List<PullRequest> ListPullRequests(ILogger logger, string accountName, string repository, int max)
+        {
+            var overrideUrl = GetRepositoryUrl(accountName, repository, "pullrequests/");
+            return GetPaginatedValues<PullRequest>(logger, overrideUrl, max);
+        }
+
         internal PullRequest PostPullRequest(string accountName, string repository, PullRequest pullRequest)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "pullrequests/");
             return _sharpBucketV2.Post(pullRequest, overrideUrl);
         }
 
+        /// <summary>
+        /// With Logging
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="accountName"></param>
+        /// <param name="repository"></param>
+        /// <param name="pullRequest"></param>
+        /// <returns></returns>
+        internal PullRequest PostPullRequest(ILogger logger, string accountName, string repository, PullRequest pullRequest)
+        {
+            var overrideUrl = GetRepositoryUrl(accountName, repository, "pullrequests/");
+            return _sharpBucketV2.Post(logger, pullRequest, overrideUrl);
+        }
+
         internal PullRequest PutPullRequest(string accountName, string repository, PullRequest pullRequest)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "pullrequests/");
             return _sharpBucketV2.Put(pullRequest, overrideUrl);
         }
 
+        /// <summary>
+        /// With Logging
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="accountName"></param>
+        /// <param name="repository"></param>
+        /// <param name="pullRequest"></param>
+        /// <returns></returns>
+        internal PullRequest PutPullRequest(ILogger logger, string accountName, string repository, PullRequest pullRequest)
+        {
+            var overrideUrl = GetRepositoryUrl(accountName, repository, "pullrequests/");
+            return _sharpBucketV2.Put(logger, pullRequest, overrideUrl);
+        }
+
         internal List<Activity> GetPullRequestLog(string accountName, string repository, int max = 0)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "pullrequests/activity/");
             return GetPaginatedValues<Activity>(overrideUrl, max);
         }
 
+        /// <summary>
+        /// With Logging
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="accountName"></param>
+        /// <param name="repository"></param>
+        /// <param name="max"></param>
+        /// <returns></returns>
+        internal List<Activity> GetPullRequestLog(ILogger logger, string accountName, string repository, int max = 0)
+        {
+            var overrideUrl = GetRepositoryUrl(accountName, repository, "pullrequests/activity/");
+            return GetPaginatedValues<Activity>(logger, overrideUrl, max);
+        }
+
         #endregion
 
         #region Pull Request Resource
@@ -150,47 +318,158 @@ internal PullRequest GetPullRequest(string accountName, string repository, int p
             return _sharpBucketV2.Get(new PullRequest(), overrideUrl);
         }
 
+        /// <summary>
+        /// With Logging
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="accountName"></param>
+        /// <param name="repository"></param>
+        /// <param name="pullRequestId"></param>
+        /// <returns></returns>
+        internal PullRequest GetPullRequest(ILogger logger, string accountName, string repository, int pullRequestId)
+        {
+            var overrideUrl = GetRepositoryUrl(accountName, repository, "pullrequests/" + pullRequestId + "/");
+            return _sharpBucketV2.Get(logger, new PullRequest(), overrideUrl);
+        }
+
+
         internal List<Commit> ListPullRequestCommits(string accountName, string repository, int pullRequestId, int max = 0)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "pullrequests/" + pullRequestId + "/commits/");
             return GetPaginatedValues<Commit>(overrideUrl, max);
         }
 
+        /// <summary>
+        /// With Logging
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="accountName"></param>
+        /// <param name="repository"></param>
+        /// <param name="pullRequestId"></param>
+        /// <param name="max"></param>
+        /// <returns></returns>
+        internal List<Commit> ListPullRequestCommits(ILogger logger, string accountName, string repository, int pullRequestId, int max = 0)
+        {
+            var overrideUrl = GetRepositoryUrl(accountName, repository, "pullrequests/" + pullRequestId + "/commits/");
+            return GetPaginatedValues<Commit>(logger, overrideUrl, max);
+        }
+
+
         internal PullRequestInfo ApprovePullRequest(string accountName, string repository, int pullRequestId)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "pullrequests/" + pullRequestId + "/approve/");
             return _sharpBucketV2.Post(new PullRequestInfo(), overrideUrl);
         }
 
+        /// <summary>
+        /// With Logging
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="accountName"></param>
+        /// <param name="repository"></param>
+        /// <param name="pullRequestId"></param>
+        /// <returns></returns>
+        internal PullRequestInfo ApprovePullRequest(ILogger logger, string accountName, string repository, int pullRequestId)
+        {
+            var overrideUrl = GetRepositoryUrl(accountName, repository, "pullrequests/" + pullRequestId + "/approve/");
+            return _sharpBucketV2.Post(logger, new PullRequestInfo(), overrideUrl);
+        }
+
         internal object RemovePullRequestApproval(string accountName, string repository, int pullRequestId)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "pullrequests/" + pullRequestId + "/approve/");
             return _sharpBucketV2.Delete(new PullRequestInfo(), overrideUrl);
         }
+        /// <summary>
+        /// With Logging
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="accountName"></param>
+        /// <param name="repository"></param>
+        /// <param name="pullRequestId"></param>
+        /// <returns></returns>
+        internal object RemovePullRequestApproval(ILogger logger, string accountName, string repository, int pullRequestId)
+        {
+            var overrideUrl = GetRepositoryUrl(accountName, repository, "pullrequests/" + pullRequestId + "/approve/");
+            return _sharpBucketV2.Delete(logger, new PullRequestInfo(), overrideUrl);
+        }
 
         internal object GetDiffForPullRequest(string accountName, string repository, int pullRequestId)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "pullrequests/" + pullRequestId + "/diff/");
             return _sharpBucketV2.Get(new Object(), overrideUrl);
         }
+        /// <summary>
+        /// With Logging
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="accountName"></param>
+        /// <param name="repository"></param>
+        /// <param name="pullRequestId"></param>
+        /// <returns></returns>
+        internal object GetDiffForPullRequest(ILogger logger, string accountName, string repository, int pullRequestId)
+        {
+            var overrideUrl = GetRepositoryUrl(accountName, repository, "pullrequests/" + pullRequestId + "/diff/");
+            return _sharpBucketV2.Get(logger, new Object(), overrideUrl);
+        }
 
         internal List<Activity> GetPullRequestActivity(string accountName, string repository, int pullRequestId, int max = 0)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "pullrequests/" + pullRequestId + "/activity/");
             return GetPaginatedValues<Activity>(overrideUrl, max);
         }
+        /// <summary>
+        /// With Logging
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="accountName"></param>
+        /// <param name="repository"></param>
+        /// <param name="pullRequestId"></param>
+        /// <param name="max"></param>
+        /// <returns></returns>
+        internal List<Activity> GetPullRequestActivity(ILogger logger, string accountName, string repository, int pullRequestId, int max = 0)
+        {
+            var overrideUrl = GetRepositoryUrl(accountName, repository, "pullrequests/" + pullRequestId + "/activity/");
+            return GetPaginatedValues<Activity>(logger, overrideUrl, max);
+        }
 
         internal Merge AcceptAndMergePullRequest(string accountName, string repository, int pullRequestId)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "pullrequests/" + pullRequestId + "/merge/");
             return _sharpBucketV2.Post(new Merge(), overrideUrl);
         }
+        /// <summary>
+        /// With Logging
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="accountName"></param>
+        /// <param name="repository"></param>
+        /// <param name="pullRequestId"></param>
+        /// <returns></returns>
+        internal Merge AcceptAndMergePullRequest(ILogger logger, string accountName, string repository, int pullRequestId)
+        {
+            var overrideUrl = GetRepositoryUrl(accountName, repository, "pullrequests/" + pullRequestId + "/merge/");
+            return _sharpBucketV2.Post(logger, new Merge(), overrideUrl);
+        }
 
         internal Merge DeclinePullRequest(string accountName, string repository, int pullRequestId)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "pullrequests/" + pullRequestId + "/decline/");
             return _sharpBucketV2.Get(new Merge(), overrideUrl);
         }
+        /// <summary>
+        /// With Logging
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="accountName"></param>
+        /// <param name="repository"></param>
+        /// <param name="pullRequestId"></param>
+        /// <returns></returns>
+        internal Merge DeclinePullRequest(ILogger logger, string accountName, string repository, int pullRequestId)
+        {
+            var overrideUrl = GetRepositoryUrl(accountName, repository, "pullrequests/" + pullRequestId + "/decline/");
+            return _sharpBucketV2.Get(logger, new Merge(), overrideUrl);
+        }
 
         internal List<Comment> ListPullRequestComments(string accountName, string repository, int pullRequestId, int max = 0)
         {
@@ -198,12 +477,42 @@ internal List<Comment> ListPullRequestComments(string accountName, string reposi
             return GetPaginatedValues<Comment>(overrideUrl, max);
         }
 
+        /// <summary>
+        /// With Logging
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="accountName"></param>
+        /// <param name="repository"></param>
+        /// <param name="pullRequestId"></param>
+        /// <param name="max"></param>
+        /// <returns></returns>
+        internal List<Comment> ListPullRequestComments(ILogger logger, string accountName, string repository, int pullRequestId, int max = 0)
+        {
+            var overrideUrl = GetRepositoryUrl(accountName, repository, "pullrequests/" + pullRequestId + "/comments/");
+            return GetPaginatedValues<Comment>(logger, overrideUrl, max);
+        }
+
         internal Comment GetPullRequestComment(string accountName, string repository, int pullRequestId, int commentId)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "pullrequests/" + pullRequestId + "/comments/" + commentId + "/");
             return _sharpBucketV2.Get(new Comment(), overrideUrl);
         }
 
+        /// <summary>
+        /// With Logging
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="accountName"></param>
+        /// <param name="repository"></param>
+        /// <param name="pullRequestId"></param>
+        /// <param name="commentId"></param>
+        /// <returns></returns>
+        internal Comment GetPullRequestComment(ILogger logger, string accountName, string repository, int pullRequestId, int commentId)
+        {
+            var overrideUrl = GetRepositoryUrl(accountName, repository, "pullrequests/" + pullRequestId + "/comments/" + commentId + "/");
+            return _sharpBucketV2.Get(logger, new Comment(), overrideUrl);
+        }
+
         #endregion
 
         #region Branch Restrictions resource
@@ -214,30 +523,99 @@ internal List<BranchRestriction> ListBranchRestrictions(string accountName, stri
             return GetPaginatedValues<BranchRestriction>(overrideUrl, max);
         }
 
+        /// <summary>
+        /// With Logging
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="accountName"></param>
+        /// <param name="repository"></param>
+        /// <param name="max"></param>
+        /// <returns></returns>
+        internal List<BranchRestriction> ListBranchRestrictions(ILogger logger, string accountName, string repository, int max = 0)
+        {
+            var overrideUrl = GetRepositoryUrl(accountName, repository, "branch-restrictions/");
+            return GetPaginatedValues<BranchRestriction>(logger, overrideUrl, max);
+        }
+
         internal BranchRestriction PostBranchRestriction(string accountName, string repository, BranchRestriction restriction)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "branch-restrictions/");
             return _sharpBucketV2.Post(restriction, overrideUrl);
         }
 
+        /// <summary>
+        /// With Logging
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="accountName"></param>
+        /// <param name="repository"></param>
+        /// <param name="restriction"></param>
+        /// <returns></returns>
+        internal BranchRestriction PostBranchRestriction(ILogger logger, string accountName, string repository, BranchRestriction restriction)
+        {
+            var overrideUrl = GetRepositoryUrl(accountName, repository, "branch-restrictions/");
+            return _sharpBucketV2.Post(logger, restriction, overrideUrl);
+        }
+
         internal BranchRestriction GetBranchRestriction(string accountName, string repository, int restrictionId)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "branch-restrictions/" + restrictionId);
             return _sharpBucketV2.Get(new BranchRestriction(), overrideUrl);
         }
 
+        /// <summary>
+        /// With Logging
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="accountName"></param>
+        /// <param name="repository"></param>
+        /// <param name="restrictionId"></param>
+        /// <returns></returns>
+        internal BranchRestriction GetBranchRestriction(ILogger logger, string accountName, string repository, int restrictionId)
+        {
+            var overrideUrl = GetRepositoryUrl(accountName, repository, "branch-restrictions/" + restrictionId);
+            return _sharpBucketV2.Get(logger, new BranchRestriction(), overrideUrl);
+        }
+
         internal BranchRestriction PutBranchRestriction(string accountName, string repository, BranchRestriction restriction)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "branch-restrictions/" + restriction.id);
             return _sharpBucketV2.Put(restriction, overrideUrl);
         }
 
+        /// <summary>
+        /// With Logging
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="accountName"></param>
+        /// <param name="repository"></param>
+        /// <param name="restriction"></param>
+        /// <returns></returns>
+        internal BranchRestriction PutBranchRestriction(ILogger logger, string accountName, string repository, BranchRestriction restriction)
+        {
+            var overrideUrl = GetRepositoryUrl(accountName, repository, "branch-restrictions/" + restriction.id);
+            return _sharpBucketV2.Put(logger, restriction, overrideUrl);
+        }
         internal BranchRestriction DeleteBranchRestriction(string accountName, string repository, int restrictionId)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "branch-restrictions/" + restrictionId);
             return _sharpBucketV2.Delete(new BranchRestriction(), overrideUrl);
         }
 
+        /// <summary>
+        /// With Logging
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="accountName"></param>
+        /// <param name="repository"></param>
+        /// <param name="restrictionId"></param>
+        /// <returns></returns>
+        internal BranchRestriction DeleteBranchRestriction(ILogger logger, string accountName, string repository, int restrictionId)
+        {
+            var overrideUrl = GetRepositoryUrl(accountName, repository, "branch-restrictions/" + restrictionId);
+            return _sharpBucketV2.Delete(logger, new BranchRestriction(), overrideUrl);
+        }
+
         #endregion
 
         #region Diff resource
@@ -248,12 +626,40 @@ internal object GetDiff(string accountName, string repository, object options)
             return _sharpBucketV2.Get(new object(), overrideUrl);
         }
 
+        /// <summary>
+        /// With Logging
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="accountName"></param>
+        /// <param name="repository"></param>
+        /// <param name="options"></param>
+        /// <returns></returns>
+        internal object GetDiff(ILogger logger, string accountName, string repository, object options)
+        {
+            var overrideUrl = GetRepositoryUrl(accountName, repository, "diff/" + options);
+            return _sharpBucketV2.Get(logger, new object(), overrideUrl);
+        }
+
         internal object GetPatch(string accountName, string repository, object options)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "patch/" + options);
             return _sharpBucketV2.Get(new object(), overrideUrl);
         }
 
+        /// <summary>
+        /// With Logging
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="accountName"></param>
+        /// <param name="repository"></param>
+        /// <param name="options"></param>
+        /// <returns></returns>
+        internal object GetPatch(ILogger logger, string accountName, string repository, object options)
+        {
+            var overrideUrl = GetRepositoryUrl(accountName, repository, "patch/" + options);
+            return _sharpBucketV2.Get(logger, new object(), overrideUrl);
+        }
+
         #endregion
 
         #region Commits Resource
@@ -268,54 +674,191 @@ internal List<Commit> ListCommits(string accountName, string repository, string
             return GetPaginatedValues<Commit>(overrideUrl, max);
         }
 
+        /// <summary>
+        /// With Logging
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="accountName"></param>
+        /// <param name="repository"></param>
+        /// <param name="branchortag"></param>
+        /// <param name="max"></param>
+        /// <returns></returns>
+        internal List<Commit> ListCommits(ILogger logger, string accountName, string repository, string branchortag = null, int max = 0)
+        {
+            var overrideUrl = GetRepositoryUrl(accountName, repository, "commits/");
+            if (!string.IsNullOrEmpty(branchortag))
+            {
+                overrideUrl += branchortag;
+            }
+            return GetPaginatedValues<Commit>(logger, overrideUrl, max);
+        }
+
         internal Commit GetCommit(string accountName, string repository, string revision)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "commit/" + revision);
             return _sharpBucketV2.Get(new Commit(), overrideUrl);
         }
 
+        /// <summary>
+        /// With Logging
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="accountName"></param>
+        /// <param name="repository"></param>
+        /// <param name="revision"></param>
+        /// <returns></returns>
+        internal Commit GetCommit(ILogger logger, string accountName, string repository, string revision)
+        {
+            var overrideUrl = GetRepositoryUrl(accountName, repository, "commit/" + revision);
+            return _sharpBucketV2.Get(logger, new Commit(), overrideUrl);
+        }
+
         internal List<Comment> ListCommitComments(string accountName, string repository, string revision, int max = 0)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "commits/" + revision + "/comments/");
             return GetPaginatedValues<Comment>(overrideUrl, max);
         }
 
+        /// <summary>
+        /// With Logging
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="accountName"></param>
+        /// <param name="repository"></param>
+        /// <param name="revision"></param>
+        /// <param name="max"></param>
+        /// <returns></returns>
+        internal List<Comment> ListCommitComments(ILogger logger, string accountName, string repository, string revision, int max = 0)
+        {
+            var overrideUrl = GetRepositoryUrl(accountName, repository, "commits/" + revision + "/comments/");
+            return GetPaginatedValues<Comment>(logger, overrideUrl, max);
+        }
+
         internal object GetCommitComment(string accountName, string repository, string revision, int commentId)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "commits/" + revision + "/comments/" + revision + "/" + commentId + "/");
             return _sharpBucketV2.Get(new object(), overrideUrl);
         }
 
+        /// <summary>
+        /// With Logging
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="accountName"></param>
+        /// <param name="repository"></param>
+        /// <param name="revision"></param>
+        /// <param name="commentId"></param>
+        /// <returns></returns>
+        internal object GetCommitComment(ILogger logger, string accountName, string repository, string revision, int commentId)
+        {
+            var overrideUrl = GetRepositoryUrl(accountName, repository, "commits/" + revision + "/comments/" + revision + "/" + commentId + "/");
+            return _sharpBucketV2.Get(logger, new object(), overrideUrl);
+        }
+
         internal object ApproveCommit(string accountName, string repository, string revision)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "commits/" + revision + "/approve/");
             return _sharpBucketV2.Post(new object(), overrideUrl);
         }
 
+        /// <summary>
+        /// With Logging
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="accountName"></param>
+        /// <param name="repository"></param>
+        /// <param name="revision"></param>
+        /// <returns></returns>
+        internal object ApproveCommit(ILogger logger, string accountName, string repository, string revision)
+        {
+            var overrideUrl = GetRepositoryUrl(accountName, repository, "commits/" + revision + "/approve/");
+            return _sharpBucketV2.Post(logger, new object(), overrideUrl);
+        }
+
         internal object DeleteCommitApproval(string accountName, string repository, string revision)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "commits/" + revision + "/approve/");
             return _sharpBucketV2.Delete(new object(), overrideUrl);
         }
 
+        /// <summary>
+        /// With Logging
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="accountName"></param>
+        /// <param name="repository"></param>
+        /// <param name="revision"></param>
+        /// <returns></returns>
+        internal object DeleteCommitApproval(ILogger logger, string accountName, string repository, string revision)
+        {
+            var overrideUrl = GetRepositoryUrl(accountName, repository, "commits/" + revision + "/approve/");
+            return _sharpBucketV2.Delete(logger, new object(), overrideUrl);
+        }
+
         internal object AddNewBuildStatus(string accountName, string repository, string revision, BuildInfo buildInfo)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "commit/" + revision + "/statuses/build/");
             return _sharpBucketV2.Post(buildInfo, overrideUrl);
         }
 
+        /// <summary>
+        /// With Logging
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="accountName"></param>
+        /// <param name="repository"></param>
+        /// <param name="revision"></param>
+        /// <param name="buildInfo"></param>
+        /// <returns></returns>
+        internal object AddNewBuildStatus(ILogger logger, string accountName, string repository, string revision, BuildInfo buildInfo)
+        {
+            var overrideUrl = GetRepositoryUrl(accountName, repository, "commit/" + revision + "/statuses/build/");
+            return _sharpBucketV2.Post(logger, buildInfo, overrideUrl);
+        }
+
         internal BuildInfo GetBuildStatusInfo(string accountName, string repository, string revision, string key)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "commit/" + revision + "/statuses/build/" + key);
             return _sharpBucketV2.Get(new BuildInfo(), overrideUrl);
         }
 
+        /// <summary>
+        /// With Logging
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="accountName"></param>
+        /// <param name="repository"></param>
+        /// <param name="revision"></param>
+        /// <param name="key"></param>
+        /// <returns></returns>
+        internal BuildInfo GetBuildStatusInfo(ILogger logger, string accountName, string repository, string revision, string key)
+        {
+            var overrideUrl = GetRepositoryUrl(accountName, repository, "commit/" + revision + "/statuses/build/" + key);
+            return _sharpBucketV2.Get(logger, new BuildInfo(), overrideUrl);
+        }
+
         internal object ChangeBuildStatusInfo(string accountName, string repository, string revision, string key, BuildInfo buildInfo)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "commit/" + revision + "/statuses/build/" + key);
             return _sharpBucketV2.Put(buildInfo, overrideUrl);
         }
 
+        /// <summary>
+        /// With Logging
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="accountName"></param>
+        /// <param name="repository"></param>
+        /// <param name="revision"></param>
+        /// <param name="key"></param>
+        /// <param name="buildInfo"></param>
+        /// <returns></returns>
+        internal object ChangeBuildStatusInfo(ILogger logger, string accountName, string repository, string revision, string key, BuildInfo buildInfo)
+        {
+            var overrideUrl = GetRepositoryUrl(accountName, repository, "commit/" + revision + "/statuses/build/" + key);
+            return _sharpBucketV2.Put(logger, buildInfo, overrideUrl);
+        }
+
         #endregion
 
         #region Default Reviewer Resource
@@ -326,6 +869,19 @@ internal object PutDefaultReviewer(string accountName, string repository, string
             return _sharpBucketV2.Put(new object(), overrideUrl);
         }
 
+        /// <summary>
+        /// With Logging
+        /// </summary>
+        /// <param name="accountName"></param>
+        /// <param name="repository"></param>
+        /// <param name="targetUsername"></param>
+        /// <returns></returns>
+        internal object PutDefaultReviewer(ILogger logger, string accountName, string repository, string targetUsername)
+        {
+            var overrideUrl = GetRepositoryUrl(accountName, repository, "default-reviewers/" + targetUsername);
+            return _sharpBucketV2.Put(logger, new object(), overrideUrl);
+        }
+
         #endregion
 
         #region Branch Resource
@@ -341,6 +897,20 @@ internal List<Branch> ListBranches(string accountName, string repository, int ma
             return GetPaginatedValues<Branch>(overrideUrl, max);
         }
 
+        /// <summary>
+        /// With Logging
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="accountName"></param>
+        /// <param name="repository"></param>
+        /// <param name="max"></param>
+        /// <returns></returns>
+        internal List<Branch> ListBranches(ILogger logger, string accountName, string repository, int max = 0)
+        {
+            var overrideUrl = GetRepositoryUrl(accountName, repository, "refs/branches/");
+            return GetPaginatedValues<Branch>(logger, overrideUrl, max);
+        }
+
         #endregion
 
         #region Tag Resource
@@ -356,6 +926,20 @@ internal List<Tag> ListTags(string accountName, string repository, int max = 0)
             return GetPaginatedValues<Tag>(overrideUrl, max);
         }
 
+        /// <summary>
+        /// With Logging
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="accountName"></param>
+        /// <param name="repository"></param>
+        /// <param name="max"></param>
+        /// <returns></returns>
+        internal List<Tag> ListTags(ILogger logger, string accountName, string repository, int max = 0)
+        {
+            var overrideUrl = GetRepositoryUrl(accountName, repository, "refs/tags/");
+            return GetPaginatedValues<Tag>(logger, overrideUrl, max);
+        }
+
         #endregion
     }
 }
\ No newline at end of file

From 7bc5fc96992aee8d282138ce9bbfb64159c2afc9 Mon Sep 17 00:00:00 2001
From: Anh-Duc Le <Anh-Duc_Le@outlook.com>
Date: Wed, 25 Apr 2018 15:58:30 +0200
Subject: [PATCH 07/21] Logging added to repository Resource.

 DeleteRepository returns HttpStatusCode instead of Repository.
---
 .../V2/EndPoints/RepositoriesEndPoint.cs      |  17 +-
 .../V2/EndPoints/RepositoryResource.cs        | 274 +++++++++++++++++-
 SharpBucketTests/SharpBucketTests.csproj      |   9 +
 SharpBucketTests/packages.config              |   1 +
 4 files changed, 289 insertions(+), 12 deletions(-)

diff --git a/SharpBucket/V2/EndPoints/RepositoriesEndPoint.cs b/SharpBucket/V2/EndPoints/RepositoriesEndPoint.cs
index 6ce02e7c..3bcb781f 100644
--- a/SharpBucket/V2/EndPoints/RepositoriesEndPoint.cs
+++ b/SharpBucket/V2/EndPoints/RepositoriesEndPoint.cs
@@ -1,5 +1,6 @@
 using System;
 using System.Collections.Generic;
+using System.Net;
 using Serilog;
 using SharpBucket.V2.Pocos;
 using Comment = SharpBucket.V2.Pocos.Comment;
@@ -153,10 +154,10 @@ internal Repository PostRepository(ILogger logger, Repository repo, string accou
             return _sharpBucketV2.Post(logger, repo, overrideUrl);
         }
 
-        internal Repository DeleteRepository(string accountName, string repository)
+        internal HttpStatusCode DeleteRepository(string accountName, string repository)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, null);
-            return _sharpBucketV2.Delete(new Repository(), overrideUrl);
+            return _sharpBucketV2.Delete(new HttpStatusCode(), overrideUrl);
         }
 
         /// <summary>
@@ -166,10 +167,10 @@ internal Repository DeleteRepository(string accountName, string repository)
         /// <param name="accountName"></param>
         /// <param name="repository"></param>
         /// <returns></returns>
-        internal Repository DeleteRepository(ILogger logger, string accountName, string repository)
+        internal HttpStatusCode DeleteRepository(ILogger logger, string accountName, string repository)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, null);
-            return _sharpBucketV2.Delete(logger, new Repository(), overrideUrl);
+            return _sharpBucketV2.Delete(logger, new HttpStatusCode(), overrideUrl);
         }
 
 
@@ -596,10 +597,10 @@ internal BranchRestriction PutBranchRestriction(ILogger logger, string accountNa
             var overrideUrl = GetRepositoryUrl(accountName, repository, "branch-restrictions/" + restriction.id);
             return _sharpBucketV2.Put(logger, restriction, overrideUrl);
         }
-        internal BranchRestriction DeleteBranchRestriction(string accountName, string repository, int restrictionId)
+        internal HttpStatusCode DeleteBranchRestriction(string accountName, string repository, int restrictionId)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "branch-restrictions/" + restrictionId);
-            return _sharpBucketV2.Delete(new BranchRestriction(), overrideUrl);
+            return _sharpBucketV2.Delete(new HttpStatusCode(), overrideUrl);
         }
 
         /// <summary>
@@ -610,10 +611,10 @@ internal BranchRestriction DeleteBranchRestriction(string accountName, string re
         /// <param name="repository"></param>
         /// <param name="restrictionId"></param>
         /// <returns></returns>
-        internal BranchRestriction DeleteBranchRestriction(ILogger logger, string accountName, string repository, int restrictionId)
+        internal HttpStatusCode DeleteBranchRestriction(ILogger logger, string accountName, string repository, int restrictionId)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "branch-restrictions/" + restrictionId);
-            return _sharpBucketV2.Delete(logger, new BranchRestriction(), overrideUrl);
+            return _sharpBucketV2.Delete(logger, new HttpStatusCode(), overrideUrl);
         }
 
         #endregion
diff --git a/SharpBucket/V2/EndPoints/RepositoryResource.cs b/SharpBucket/V2/EndPoints/RepositoryResource.cs
index 3baf7979..726fd8ba 100644
--- a/SharpBucket/V2/EndPoints/RepositoryResource.cs
+++ b/SharpBucket/V2/EndPoints/RepositoryResource.cs
@@ -1,4 +1,6 @@
 using System.Collections.Generic;
+using System.Net;
+using Serilog;
 using SharpBucket.V2.Pocos;
 
 namespace SharpBucket.V2.EndPoints
@@ -34,19 +36,51 @@ public Repository GetRepository()
             return _repositoriesEndPoint.GetRepository(_accountName, _repository);
         }
 
+        /// <summary>
+        /// With Logging.
+        /// Returns a single repository.
+        /// </summary>
+        /// <returns></returns>
+        public Repository GetRepository(ILogger logger)
+        {
+            return _repositoriesEndPoint.GetRepository(logger, _accountName, _repository);
+        }
+
         /// <summary>
         /// Removes a repository.  
         /// </summary>
         /// <returns></returns>
-        public Repository DeleteRepository()
+        public HttpStatusCode DeleteRepository()
         {
             return _repositoriesEndPoint.DeleteRepository(_accountName, _repository);
         }
 
+
+        /// <summary>
+        /// With Logging.
+        /// Removes a repository.  
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <returns></returns>
+        public HttpStatusCode DeleteRepository(ILogger logger)
+        {
+            return _repositoriesEndPoint.DeleteRepository(logger, _accountName, _repository);
+        }
+
         public Repository PostRepository(Repository repository)
         {
             return _repositoriesEndPoint.PostRepository(repository, _accountName);
         }
+        /// <summary>
+        /// With Logging
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="repository"></param>
+        /// <returns></returns>
+        public Repository PostRepository(ILogger logger, Repository repository)
+        {
+            return _repositoriesEndPoint.PostRepository(logger, repository, _accountName);
+        }
 
         /// <summary>
         /// Gets the list of accounts watching a repository. 
@@ -57,6 +91,15 @@ public List<Watcher> ListWatchers()
             return _repositoriesEndPoint.ListWatchers(_accountName, _repository);
         }
 
+        /// <summary>
+        /// Gets the list of accounts watching a repository. 
+        /// </summary>
+        /// <returns></returns>
+        public List<Watcher> ListWatchers(ILogger logger)
+        {
+            return _repositoriesEndPoint.ListWatchers(logger, _accountName, _repository);
+        }
+
         /// <summary>
         /// List of repository forks, This call returns a repository object for each fork.
         /// </summary>
@@ -66,6 +109,15 @@ public List<Fork> ListForks()
             return _repositoriesEndPoint.ListForks(_accountName, _repository);
         }
 
+        /// <summary>
+        /// List of repository forks, This call returns a repository object for each fork.
+        /// </summary>
+        /// <returns></returns>
+        public List<Fork> ListForks(ILogger logger)
+        {
+            return _repositoriesEndPoint.ListForks(logger, _accountName, _repository);
+        }
+
         #endregion
 
         #region Pull Requests Resource
@@ -94,11 +146,23 @@ public PullRequestsResource PullRequestsResource()
         /// List the information associated with a repository's branch restrictions. 
         /// </summary>
         /// <returns></returns>
-        public object ListBranchRestrictions()
+        public List<BranchRestriction> ListBranchRestrictions()
         {
             return _repositoriesEndPoint.ListBranchRestrictions(_accountName, _repository);
         }
 
+        /// More info:
+        /// https://confluence.atlassian.com/display/BITBUCKET/branch-restrictions+Resource
+        /// <summary>
+        /// With Logging.
+        /// List the information associated with a repository's branch restrictions. 
+        /// </summary>
+        /// <returns></returns>
+        public List<BranchRestriction> ListBranchRestrictions(ILogger logger)
+        {
+            return _repositoriesEndPoint.ListBranchRestrictions(logger, _accountName, _repository);
+        }
+
         /// <summary>
         /// Creates restrictions for the specified repository. You should specify a Content-Header with this call. 
         /// </summary>
@@ -109,16 +173,40 @@ public BranchRestriction PostBranchRestriction(BranchRestriction restriction)
             return _repositoriesEndPoint.PostBranchRestriction(_accountName, _repository, restriction);
         }
 
+        /// <summary>
+        /// With Logging.
+        /// Creates restrictions for the specified repository. You should specify a Content-Header with this call. 
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="restriction">The branch restriction.</param>
+        /// <returns></returns>
+        public BranchRestriction PostBranchRestriction(ILogger logger, BranchRestriction restriction)
+        {
+            return _repositoriesEndPoint.PostBranchRestriction(logger, _accountName, _repository, restriction);
+        }
+
         /// <summary>
         /// Gets the information associated with specific restriction. 
         /// </summary>
         /// <param name="restrictionId">The restriction's identifier.</param>
         /// <returns></returns>
-        public object GetBranchRestriction(int restrictionId)
+        public BranchRestriction GetBranchRestriction(int restrictionId)
         {
             return _repositoriesEndPoint.GetBranchRestriction(_accountName, _repository, restrictionId);
         }
 
+        /// <summary>
+        /// With Logging.
+        /// Gets the information associated with specific restriction. 
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="restrictionId">The restriction's identifier.</param>
+        /// <returns></returns>
+        public BranchRestriction GetBranchRestriction(ILogger logger, int restrictionId)
+        {
+            return _repositoriesEndPoint.GetBranchRestriction(logger, _accountName, _repository, restrictionId);
+        }
+
         /// <summary>
         /// Updates a specific branch restriction. You cannot change the kind value with this call. 
         /// </summary>
@@ -129,15 +217,37 @@ public BranchRestriction PutBranchRestriction(BranchRestriction restriction)
             return _repositoriesEndPoint.PutBranchRestriction(_accountName, _repository, restriction);
         }
 
+        /// <summary>
+        /// With Logging
+        /// Updates a specific branch restriction. You cannot change the kind value with this call. 
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="restriction">The branch restriction.</param>
+        /// <returns></returns>
+        public BranchRestriction PutBranchRestriction(ILogger logger, BranchRestriction restriction)
+        {
+            return _repositoriesEndPoint.PutBranchRestriction(logger, _accountName, _repository, restriction);
+        }
+
         /// <summary>
         /// Deletes the specified restriction.  
         /// </summary>
         /// <param name="restrictionId">The restriction's identifier.</param>
         /// <returns></returns>
-        public object DeleteBranchRestriction(int restrictionId)
+        public HttpStatusCode DeleteBranchRestriction(int restrictionId)
         {
             return _repositoriesEndPoint.DeleteBranchRestriction(_accountName, _repository, restrictionId);
         }
+        /// <summary>
+        /// With Logging
+        /// Deletes the specified restriction.  
+        /// </summary>
+        /// <param name="restrictionId">The restriction's identifier.</param>
+        /// <returns></returns>
+        public HttpStatusCode DeleteBranchRestriction(ILogger logger, int restrictionId)
+        {
+            return _repositoriesEndPoint.DeleteBranchRestriction(logger, _accountName, _repository, restrictionId);
+        }
 
         #endregion
 
@@ -155,6 +265,20 @@ public object GetDiff(object options)
             return _repositoriesEndPoint.GetDiff(_accountName, _repository, options);
         }
 
+        /// More info:
+        /// https://confluence.atlassian.com/display/BITBUCKET/diff+Resource
+        /// <summary>
+        /// With Logging
+        /// Gets the diff for the current repository.  
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="options">The diff options.</param>
+        /// <returns></returns>
+        public object GetDiff(ILogger logger, object options)
+        {
+            return _repositoriesEndPoint.GetDiff(logger, _accountName, _repository, options);
+        }
+
         /// <summary>
         /// Gets the patch for an individual specification. 
         /// </summary>
@@ -165,6 +289,18 @@ public object GetPatch(object options)
             return _repositoriesEndPoint.GetPatch(_accountName, _repository, options);
         }
 
+        /// <summary>
+        /// With Logging
+        /// Gets the patch for an individual specification. 
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="options">The patch options.</param>
+        /// <returns></returns>
+        public object GetPatch(ILogger logger, object options)
+        {
+            return _repositoriesEndPoint.GetPatch(logger, _accountName, _repository, options);
+        }
+
         #endregion
 
         #region Commits resource
@@ -182,6 +318,21 @@ public List<Commit> ListCommits(string branchortag = null, int max = 0)
         {
             return _repositoriesEndPoint.ListCommits(_accountName, _repository, branchortag, max);
         }
+        /// More info:
+        /// https://confluence.atlassian.com/display/BITBUCKET/commits+or+commit+Resource
+        /// <summary>
+        /// Gets the commit information associated with a repository. 
+        /// By default, this call returns all the commits across all branches, bookmarks, and tags. The newest commit is first. 
+        /// With Logging
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="branchortag">The branch or tag to get, for example, master or default.</param>
+        /// <param name="max">Values greater than 0 will set a maximum number of records to return. 0 or less returns all.</param>
+        /// <returns></returns>
+        public List<Commit> ListCommits(ILogger logger, string branchortag = null, int max = 0)
+        {
+            return _repositoriesEndPoint.ListCommits(logger, _accountName, _repository, branchortag, max);
+        }
 
         /// <summary>
         /// Gets the information associated with an individual commit. 
@@ -193,6 +344,18 @@ public Commit GetCommit(string revision)
             return _repositoriesEndPoint.GetCommit(_accountName, _repository, revision);
         }
 
+        /// <summary>
+        /// Gets the information associated with an individual commit. 
+        /// With Logging
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="revision">The commit's SHA1.</param>
+        /// <returns></returns>
+        public Commit GetCommit(ILogger logger, string revision)
+        {
+            return _repositoriesEndPoint.GetCommit(logger, _accountName, _repository, revision);
+        }
+
         /// <summary>
         /// List of comments on the specified commit.
         /// </summary>
@@ -203,6 +366,18 @@ public List<Comment> ListCommitComments(string revision)
             return _repositoriesEndPoint.ListCommitComments(_accountName, _repository, revision);
         }
 
+        /// <summary>
+        /// List of comments on the specified commit.
+        /// With Logging
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="revision">The commit's SHA1.</param>
+        /// <returns></returns>
+        public List<Comment> ListCommitComments(ILogger logger, string revision)
+        {
+            return _repositoriesEndPoint.ListCommitComments(logger, _accountName, _repository, revision);
+        }
+
         /// <summary>
         /// To get an individual commit comment, just follow the object's self link.
         /// </summary>
@@ -214,6 +389,19 @@ public object GetCommitComment(string revision, int commentId)
             return _repositoriesEndPoint.GetCommitComment(_accountName, _repository, revision, commentId);
         }
 
+        /// <summary>
+        /// To get an individual commit comment, just follow the object's self link.
+        /// With Logging
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="revision">The commit's SHA1.</param>
+        /// <param name="commentId">The comment identifier.</param>
+        /// <returns></returns>
+        public object GetCommitComment(ILogger logger, string revision, int commentId)
+        {
+            return _repositoriesEndPoint.GetCommitComment(logger, _accountName, _repository, revision, commentId);
+        }
+
         /// <summary>
         /// Give your approval on a commit.  
         /// You can only approve a comment on behalf of the authenticated account.  This returns the participant object for the current user.
@@ -225,6 +413,19 @@ public object ApproveCommit(string revision)
             return _repositoriesEndPoint.ApproveCommit(_accountName, _repository, revision);
         }
 
+        /// <summary>
+        /// Give your approval on a commit.  
+        /// You can only approve a comment on behalf of the authenticated account.  This returns the participant object for the current user.
+        /// With Logging
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="revision">The commit's SHA1.</param>
+        /// <returns></returns>
+        public object ApproveCommit(ILogger logger, string revision)
+        {
+            return _repositoriesEndPoint.ApproveCommit(logger, _accountName, _repository, revision);
+        }
+
         /// <summary>
         /// Revoke your approval of a commit. You can remove approvals on behalf of the authenticated account. 
         /// </summary>
@@ -235,6 +436,18 @@ public object DeleteCommitApproval(string revision)
             return _repositoriesEndPoint.DeleteCommitApproval(_accountName, _repository, revision);
         }
 
+        /// <summary>
+        /// Revoke your approval of a commit. You can remove approvals on behalf of the authenticated account. 
+        /// With Logging
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="revision">The commit's SHA1.</param>
+        /// <returns></returns>
+        public object DeleteCommitApproval(ILogger logger, string revision)
+        {
+            return _repositoriesEndPoint.DeleteCommitApproval(logger, _accountName, _repository, revision);
+        }
+
         /// <summary>
         /// Creates a new build status against the specified commit. If the specified key already exists, the existing status object will be overwritten.
         /// </summary>
@@ -246,6 +459,19 @@ public object AddNewBuildStatus(string revision, BuildInfo buildInfo)
             return _repositoriesEndPoint.AddNewBuildStatus(_accountName, _repository, revision, buildInfo);
         }
 
+        /// <summary>
+        /// Creates a new build status against the specified commit. If the specified key already exists, the existing status object will be overwritten.
+        /// With Logging
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="revision">The commit's SHA1</param>
+        /// <param name="buildInfo">The new commit status object</param>
+        /// <returns></returns>
+        public object AddNewBuildStatus(ILogger logger, string revision, BuildInfo buildInfo)
+        {
+            return _repositoriesEndPoint.AddNewBuildStatus(logger, _accountName, _repository, revision, buildInfo);
+        }
+
         /// <summary>
         /// Returns the specified build status for a commit.
         /// </summary>
@@ -257,6 +483,19 @@ public BuildInfo GetBuildStatusInfo(string revision, string key)
             return _repositoriesEndPoint.GetBuildStatusInfo(_accountName, _repository, revision, key);
         }
 
+        /// <summary>
+        /// Returns the specified build status for a commit.
+        /// With Logging
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="revision">The commit's SHA1</param>
+        /// <param name="key">The build status' unique key</param>
+        /// <returns></returns>
+        public BuildInfo GetBuildStatusInfo(ILogger logger, string revision, string key)
+        {
+            return _repositoriesEndPoint.GetBuildStatusInfo(logger, _accountName, _repository, revision, key);
+        }
+
         /// <summary>
         /// Used to update the current status of a build status object on the specific commit.
         /// </summary>
@@ -270,6 +509,21 @@ public object ChangeBuildStatusInfo(string revision, string key, BuildInfo build
             return _repositoriesEndPoint.ChangeBuildStatusInfo(_accountName, _repository, revision, key, buildInfo);
         }
 
+        /// <summary>
+        /// Used to update the current status of a build status object on the specific commit.
+        /// With Logging
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="revision">The commit's SHA1</param>
+        /// <param name="key">The build status' unique key</param>
+        /// <param name="buildInfo">The new commit status object</param>
+        /// <returns></returns>
+        /// /// <remarks>This operation can also be used to change other properties of the build status: state, name, description, url, refname. The key cannot be changed.</remarks>
+        public object ChangeBuildStatusInfo(ILogger logger, string revision, string key, BuildInfo buildInfo)
+        {
+            return _repositoriesEndPoint.ChangeBuildStatusInfo(logger, _accountName, _repository, revision, key, buildInfo);
+        }
+
         #endregion
 
         #region Default Reviewer Resource
@@ -284,6 +538,18 @@ public object PutDefaultReviewer(string targetUsername)
             return _repositoriesEndPoint.PutDefaultReviewer(_accountName, _repository, targetUsername);
         }
 
+        /// <summary>
+        /// Adds a user as the default review for pull requests on a repository.
+        /// With Logging
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="targetUsername">The user to add as the default reviewer.</param>
+        /// <returns></returns>
+        public object PutDefaultReviewer(ILogger logger, string targetUsername)
+        {
+            return _repositoriesEndPoint.PutDefaultReviewer(logger, _accountName, _repository, targetUsername);
+        }
+
         #endregion
     }
 }
\ No newline at end of file
diff --git a/SharpBucketTests/SharpBucketTests.csproj b/SharpBucketTests/SharpBucketTests.csproj
index 57cf1a27..39fb3022 100644
--- a/SharpBucketTests/SharpBucketTests.csproj
+++ b/SharpBucketTests/SharpBucketTests.csproj
@@ -47,6 +47,9 @@
     <Reference Include="nunit.framework, Version=3.9.0.0, Culture=neutral, PublicKeyToken=2638cd05610744eb, processorArchitecture=MSIL">
       <HintPath>..\packages\NUnit.3.9.0\lib\net45\nunit.framework.dll</HintPath>
     </Reference>
+    <Reference Include="Serilog, Version=2.0.0.0, Culture=neutral, PublicKeyToken=24c2f752a8e58a10, processorArchitecture=MSIL">
+      <HintPath>..\packages\Serilog.2.6.0\lib\net45\Serilog.dll</HintPath>
+    </Reference>
     <Reference Include="Shouldly, Version=2.8.3.0, Culture=neutral, PublicKeyToken=6042cbcb05cbc941, processorArchitecture=MSIL">
       <HintPath>..\packages\Shouldly.2.8.3\lib\net40\Shouldly.dll</HintPath>
     </Reference>
@@ -82,6 +85,12 @@
       <SubType>Designer</SubType>
     </None>
   </ItemGroup>
+  <ItemGroup>
+    <ProjectReference Include="..\SharpBucket\SharpBucket.csproj">
+      <Project>{F7C8FEF8-4ADB-4175-A4D8-2A0598B78786}</Project>
+      <Name>SharpBucket</Name>
+    </ProjectReference>
+  </ItemGroup>
   <Choose>
     <When Condition="'$(VisualStudioVersion)' == '10.0' And '$(IsCodedUITest)' == 'True'">
       <ItemGroup>
diff --git a/SharpBucketTests/packages.config b/SharpBucketTests/packages.config
index 8df699ab..4716e363 100644
--- a/SharpBucketTests/packages.config
+++ b/SharpBucketTests/packages.config
@@ -1,5 +1,6 @@
 <?xml version="1.0" encoding="utf-8"?>
 <packages>
   <package id="NUnit" version="3.9.0" targetFramework="net45" />
+  <package id="Serilog" version="2.6.0" targetFramework="net45" />
   <package id="Shouldly" version="2.8.3" targetFramework="net45" />
 </packages>
\ No newline at end of file

From acf27a8c6b183c35f90a366f77c0265cdb6a1c2f Mon Sep 17 00:00:00 2001
From: Anh-Duc Le <Anh-Duc_Le@outlook.com>
Date: Wed, 25 Apr 2018 16:07:56 +0200
Subject: [PATCH 08/21] * Added PutRepository * Added PostBranch * Added
 ListBranch

---
 SharpBucket/V2/EndPoints/BranchResource.cs    |  30 ++++
 .../V2/EndPoints/RepositoriesEndPoint.cs      | 141 +++++++++++++-----
 .../V2/EndPoints/RepositoryResource.cs        |  24 +++
 3 files changed, 161 insertions(+), 34 deletions(-)

diff --git a/SharpBucket/V2/EndPoints/BranchResource.cs b/SharpBucket/V2/EndPoints/BranchResource.cs
index 53e9000f..1d4c976b 100644
--- a/SharpBucket/V2/EndPoints/BranchResource.cs
+++ b/SharpBucket/V2/EndPoints/BranchResource.cs
@@ -1,5 +1,6 @@
 using SharpBucket.V2.Pocos;
 using System.Collections.Generic;
+using Serilog;
 
 namespace SharpBucket.V2.EndPoints
 {
@@ -30,5 +31,34 @@ public List<Branch> ListBranches()
         {
             return _repositoriesEndPoint.ListBranches(_accountName, _repository);
         }
+
+        /// <summary>
+        /// With Logging
+        /// Returns a list of branches for the specific repository
+        /// </summary>
+        /// <returns></returns>
+        public List<Branch> ListBranches(ILogger logger)
+        {
+            return _repositoriesEndPoint.ListBranches(logger, _accountName, _repository);
+        }
+        
+        /// <summary>
+        /// Posts a new branch for the specific repository
+        /// </summary>
+        /// <param name="branch"></param>
+        public void PostBranch(Branch branch)
+        {
+            _repositoriesEndPoint.PostBranch(_accountName, _repository, branch);
+        }
+
+        /// <summary>
+        /// With Logging
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="branch"></param>
+        public void PostBranch(ILogger logger, Branch branch)
+        {
+            _repositoriesEndPoint.PostBranch(logger, _accountName, _repository, branch);
+        }
     }
 }
\ No newline at end of file
diff --git a/SharpBucket/V2/EndPoints/RepositoriesEndPoint.cs b/SharpBucket/V2/EndPoints/RepositoriesEndPoint.cs
index 3bcb781f..f2a712f0 100644
--- a/SharpBucket/V2/EndPoints/RepositoriesEndPoint.cs
+++ b/SharpBucket/V2/EndPoints/RepositoriesEndPoint.cs
@@ -78,6 +78,36 @@ public List<Repository> ListPublicRepositories(ILogger logger, int max = 0)
         {
             return GetPaginatedValues<Repository>(logger, _baseUrl, max);
         }
+
+
+        /// <summary>
+        /// Post a new branch
+        /// </summary>
+        /// <param name="accountName"></param>
+        /// <param name="repository"></param>
+        /// <param name="branch"></param>
+        /// <returns></returns>
+        public Branch PostBranch(string accountName, string repository, Branch branch)
+        {
+            var overrideUrl = GetRepositoryUrl(accountName, repository, "src/");
+            return _sharpBucketV2.Post(branch, overrideUrl);
+        }
+
+
+        /// <summary>
+        /// Post a new branch with Logging
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="accountName"></param>
+        /// <param name="repository"></param>
+        /// <param name="branch"></param>
+        /// <returns></returns>
+        public Branch PostBranch(ILogger logger, string accountName, string repository, Branch branch)
+        {
+            var overrideUrl = GetRepositoryUrl(accountName, repository, "src/");
+            return _sharpBucketV2.Post(logger, branch, overrideUrl);
+        }
+
         #endregion
 
         #region Repository resource
@@ -263,7 +293,8 @@ internal PullRequest PostPullRequest(string accountName, string repository, Pull
         /// <param name="repository"></param>
         /// <param name="pullRequest"></param>
         /// <returns></returns>
-        internal PullRequest PostPullRequest(ILogger logger, string accountName, string repository, PullRequest pullRequest)
+        internal PullRequest PostPullRequest(ILogger logger, string accountName, string repository,
+            PullRequest pullRequest)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "pullrequests/");
             return _sharpBucketV2.Post(logger, pullRequest, overrideUrl);
@@ -283,7 +314,8 @@ internal PullRequest PutPullRequest(string accountName, string repository, PullR
         /// <param name="repository"></param>
         /// <param name="pullRequest"></param>
         /// <returns></returns>
-        internal PullRequest PutPullRequest(ILogger logger, string accountName, string repository, PullRequest pullRequest)
+        internal PullRequest PutPullRequest(ILogger logger, string accountName, string repository,
+            PullRequest pullRequest)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "pullrequests/");
             return _sharpBucketV2.Put(logger, pullRequest, overrideUrl);
@@ -334,7 +366,8 @@ internal PullRequest GetPullRequest(ILogger logger, string accountName, string r
         }
 
 
-        internal List<Commit> ListPullRequestCommits(string accountName, string repository, int pullRequestId, int max = 0)
+        internal List<Commit> ListPullRequestCommits(string accountName, string repository, int pullRequestId,
+            int max = 0)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "pullrequests/" + pullRequestId + "/commits/");
             return GetPaginatedValues<Commit>(overrideUrl, max);
@@ -349,7 +382,8 @@ internal List<Commit> ListPullRequestCommits(string accountName, string reposito
         /// <param name="pullRequestId"></param>
         /// <param name="max"></param>
         /// <returns></returns>
-        internal List<Commit> ListPullRequestCommits(ILogger logger, string accountName, string repository, int pullRequestId, int max = 0)
+        internal List<Commit> ListPullRequestCommits(ILogger logger, string accountName, string repository,
+            int pullRequestId, int max = 0)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "pullrequests/" + pullRequestId + "/commits/");
             return GetPaginatedValues<Commit>(logger, overrideUrl, max);
@@ -370,7 +404,8 @@ internal PullRequestInfo ApprovePullRequest(string accountName, string repositor
         /// <param name="repository"></param>
         /// <param name="pullRequestId"></param>
         /// <returns></returns>
-        internal PullRequestInfo ApprovePullRequest(ILogger logger, string accountName, string repository, int pullRequestId)
+        internal PullRequestInfo ApprovePullRequest(ILogger logger, string accountName, string repository,
+            int pullRequestId)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "pullrequests/" + pullRequestId + "/approve/");
             return _sharpBucketV2.Post(logger, new PullRequestInfo(), overrideUrl);
@@ -381,6 +416,7 @@ internal object RemovePullRequestApproval(string accountName, string repository,
             var overrideUrl = GetRepositoryUrl(accountName, repository, "pullrequests/" + pullRequestId + "/approve/");
             return _sharpBucketV2.Delete(new PullRequestInfo(), overrideUrl);
         }
+
         /// <summary>
         /// With Logging
         /// </summary>
@@ -389,7 +425,8 @@ internal object RemovePullRequestApproval(string accountName, string repository,
         /// <param name="repository"></param>
         /// <param name="pullRequestId"></param>
         /// <returns></returns>
-        internal object RemovePullRequestApproval(ILogger logger, string accountName, string repository, int pullRequestId)
+        internal object RemovePullRequestApproval(ILogger logger, string accountName, string repository,
+            int pullRequestId)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "pullrequests/" + pullRequestId + "/approve/");
             return _sharpBucketV2.Delete(logger, new PullRequestInfo(), overrideUrl);
@@ -400,6 +437,7 @@ internal object GetDiffForPullRequest(string accountName, string repository, int
             var overrideUrl = GetRepositoryUrl(accountName, repository, "pullrequests/" + pullRequestId + "/diff/");
             return _sharpBucketV2.Get(new Object(), overrideUrl);
         }
+
         /// <summary>
         /// With Logging
         /// </summary>
@@ -414,11 +452,13 @@ internal object GetDiffForPullRequest(ILogger logger, string accountName, string
             return _sharpBucketV2.Get(logger, new Object(), overrideUrl);
         }
 
-        internal List<Activity> GetPullRequestActivity(string accountName, string repository, int pullRequestId, int max = 0)
+        internal List<Activity> GetPullRequestActivity(string accountName, string repository, int pullRequestId,
+            int max = 0)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "pullrequests/" + pullRequestId + "/activity/");
             return GetPaginatedValues<Activity>(overrideUrl, max);
         }
+
         /// <summary>
         /// With Logging
         /// </summary>
@@ -428,7 +468,8 @@ internal List<Activity> GetPullRequestActivity(string accountName, string reposi
         /// <param name="pullRequestId"></param>
         /// <param name="max"></param>
         /// <returns></returns>
-        internal List<Activity> GetPullRequestActivity(ILogger logger, string accountName, string repository, int pullRequestId, int max = 0)
+        internal List<Activity> GetPullRequestActivity(ILogger logger, string accountName, string repository,
+            int pullRequestId, int max = 0)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "pullrequests/" + pullRequestId + "/activity/");
             return GetPaginatedValues<Activity>(logger, overrideUrl, max);
@@ -439,6 +480,7 @@ internal Merge AcceptAndMergePullRequest(string accountName, string repository,
             var overrideUrl = GetRepositoryUrl(accountName, repository, "pullrequests/" + pullRequestId + "/merge/");
             return _sharpBucketV2.Post(new Merge(), overrideUrl);
         }
+
         /// <summary>
         /// With Logging
         /// </summary>
@@ -447,7 +489,8 @@ internal Merge AcceptAndMergePullRequest(string accountName, string repository,
         /// <param name="repository"></param>
         /// <param name="pullRequestId"></param>
         /// <returns></returns>
-        internal Merge AcceptAndMergePullRequest(ILogger logger, string accountName, string repository, int pullRequestId)
+        internal Merge AcceptAndMergePullRequest(ILogger logger, string accountName, string repository,
+            int pullRequestId)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "pullrequests/" + pullRequestId + "/merge/");
             return _sharpBucketV2.Post(logger, new Merge(), overrideUrl);
@@ -458,6 +501,7 @@ internal Merge DeclinePullRequest(string accountName, string repository, int pul
             var overrideUrl = GetRepositoryUrl(accountName, repository, "pullrequests/" + pullRequestId + "/decline/");
             return _sharpBucketV2.Get(new Merge(), overrideUrl);
         }
+
         /// <summary>
         /// With Logging
         /// </summary>
@@ -472,7 +516,8 @@ internal Merge DeclinePullRequest(ILogger logger, string accountName, string rep
             return _sharpBucketV2.Get(logger, new Merge(), overrideUrl);
         }
 
-        internal List<Comment> ListPullRequestComments(string accountName, string repository, int pullRequestId, int max = 0)
+        internal List<Comment> ListPullRequestComments(string accountName, string repository, int pullRequestId,
+            int max = 0)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "pullrequests/" + pullRequestId + "/comments/");
             return GetPaginatedValues<Comment>(overrideUrl, max);
@@ -487,7 +532,8 @@ internal List<Comment> ListPullRequestComments(string accountName, string reposi
         /// <param name="pullRequestId"></param>
         /// <param name="max"></param>
         /// <returns></returns>
-        internal List<Comment> ListPullRequestComments(ILogger logger, string accountName, string repository, int pullRequestId, int max = 0)
+        internal List<Comment> ListPullRequestComments(ILogger logger, string accountName, string repository,
+            int pullRequestId, int max = 0)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "pullrequests/" + pullRequestId + "/comments/");
             return GetPaginatedValues<Comment>(logger, overrideUrl, max);
@@ -495,7 +541,8 @@ internal List<Comment> ListPullRequestComments(ILogger logger, string accountNam
 
         internal Comment GetPullRequestComment(string accountName, string repository, int pullRequestId, int commentId)
         {
-            var overrideUrl = GetRepositoryUrl(accountName, repository, "pullrequests/" + pullRequestId + "/comments/" + commentId + "/");
+            var overrideUrl = GetRepositoryUrl(accountName, repository,
+                "pullrequests/" + pullRequestId + "/comments/" + commentId + "/");
             return _sharpBucketV2.Get(new Comment(), overrideUrl);
         }
 
@@ -508,9 +555,11 @@ internal Comment GetPullRequestComment(string accountName, string repository, in
         /// <param name="pullRequestId"></param>
         /// <param name="commentId"></param>
         /// <returns></returns>
-        internal Comment GetPullRequestComment(ILogger logger, string accountName, string repository, int pullRequestId, int commentId)
+        internal Comment GetPullRequestComment(ILogger logger, string accountName, string repository, int pullRequestId,
+            int commentId)
         {
-            var overrideUrl = GetRepositoryUrl(accountName, repository, "pullrequests/" + pullRequestId + "/comments/" + commentId + "/");
+            var overrideUrl = GetRepositoryUrl(accountName, repository,
+                "pullrequests/" + pullRequestId + "/comments/" + commentId + "/");
             return _sharpBucketV2.Get(logger, new Comment(), overrideUrl);
         }
 
@@ -532,13 +581,15 @@ internal List<BranchRestriction> ListBranchRestrictions(string accountName, stri
         /// <param name="repository"></param>
         /// <param name="max"></param>
         /// <returns></returns>
-        internal List<BranchRestriction> ListBranchRestrictions(ILogger logger, string accountName, string repository, int max = 0)
+        internal List<BranchRestriction> ListBranchRestrictions(ILogger logger, string accountName, string repository,
+            int max = 0)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "branch-restrictions/");
             return GetPaginatedValues<BranchRestriction>(logger, overrideUrl, max);
         }
 
-        internal BranchRestriction PostBranchRestriction(string accountName, string repository, BranchRestriction restriction)
+        internal BranchRestriction PostBranchRestriction(string accountName, string repository,
+            BranchRestriction restriction)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "branch-restrictions/");
             return _sharpBucketV2.Post(restriction, overrideUrl);
@@ -552,7 +603,8 @@ internal BranchRestriction PostBranchRestriction(string accountName, string repo
         /// <param name="repository"></param>
         /// <param name="restriction"></param>
         /// <returns></returns>
-        internal BranchRestriction PostBranchRestriction(ILogger logger, string accountName, string repository, BranchRestriction restriction)
+        internal BranchRestriction PostBranchRestriction(ILogger logger, string accountName, string repository,
+            BranchRestriction restriction)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "branch-restrictions/");
             return _sharpBucketV2.Post(logger, restriction, overrideUrl);
@@ -572,13 +624,15 @@ internal BranchRestriction GetBranchRestriction(string accountName, string repos
         /// <param name="repository"></param>
         /// <param name="restrictionId"></param>
         /// <returns></returns>
-        internal BranchRestriction GetBranchRestriction(ILogger logger, string accountName, string repository, int restrictionId)
+        internal BranchRestriction GetBranchRestriction(ILogger logger, string accountName, string repository,
+            int restrictionId)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "branch-restrictions/" + restrictionId);
             return _sharpBucketV2.Get(logger, new BranchRestriction(), overrideUrl);
         }
 
-        internal BranchRestriction PutBranchRestriction(string accountName, string repository, BranchRestriction restriction)
+        internal BranchRestriction PutBranchRestriction(string accountName, string repository,
+            BranchRestriction restriction)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "branch-restrictions/" + restriction.id);
             return _sharpBucketV2.Put(restriction, overrideUrl);
@@ -592,11 +646,13 @@ internal BranchRestriction PutBranchRestriction(string accountName, string repos
         /// <param name="repository"></param>
         /// <param name="restriction"></param>
         /// <returns></returns>
-        internal BranchRestriction PutBranchRestriction(ILogger logger, string accountName, string repository, BranchRestriction restriction)
+        internal BranchRestriction PutBranchRestriction(ILogger logger, string accountName, string repository,
+            BranchRestriction restriction)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "branch-restrictions/" + restriction.id);
             return _sharpBucketV2.Put(logger, restriction, overrideUrl);
         }
+
         internal HttpStatusCode DeleteBranchRestriction(string accountName, string repository, int restrictionId)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "branch-restrictions/" + restrictionId);
@@ -611,7 +667,8 @@ internal HttpStatusCode DeleteBranchRestriction(string accountName, string repos
         /// <param name="repository"></param>
         /// <param name="restrictionId"></param>
         /// <returns></returns>
-        internal HttpStatusCode DeleteBranchRestriction(ILogger logger, string accountName, string repository, int restrictionId)
+        internal HttpStatusCode DeleteBranchRestriction(ILogger logger, string accountName, string repository,
+            int restrictionId)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "branch-restrictions/" + restrictionId);
             return _sharpBucketV2.Delete(logger, new HttpStatusCode(), overrideUrl);
@@ -672,6 +729,7 @@ internal List<Commit> ListCommits(string accountName, string repository, string
             {
                 overrideUrl += branchortag;
             }
+
             return GetPaginatedValues<Commit>(overrideUrl, max);
         }
 
@@ -684,13 +742,15 @@ internal List<Commit> ListCommits(string accountName, string repository, string
         /// <param name="branchortag"></param>
         /// <param name="max"></param>
         /// <returns></returns>
-        internal List<Commit> ListCommits(ILogger logger, string accountName, string repository, string branchortag = null, int max = 0)
+        internal List<Commit> ListCommits(ILogger logger, string accountName, string repository,
+            string branchortag = null, int max = 0)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "commits/");
             if (!string.IsNullOrEmpty(branchortag))
             {
                 overrideUrl += branchortag;
             }
+
             return GetPaginatedValues<Commit>(logger, overrideUrl, max);
         }
 
@@ -729,7 +789,8 @@ internal List<Comment> ListCommitComments(string accountName, string repository,
         /// <param name="revision"></param>
         /// <param name="max"></param>
         /// <returns></returns>
-        internal List<Comment> ListCommitComments(ILogger logger, string accountName, string repository, string revision, int max = 0)
+        internal List<Comment> ListCommitComments(ILogger logger, string accountName, string repository,
+            string revision, int max = 0)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "commits/" + revision + "/comments/");
             return GetPaginatedValues<Comment>(logger, overrideUrl, max);
@@ -737,7 +798,8 @@ internal List<Comment> ListCommitComments(ILogger logger, string accountName, st
 
         internal object GetCommitComment(string accountName, string repository, string revision, int commentId)
         {
-            var overrideUrl = GetRepositoryUrl(accountName, repository, "commits/" + revision + "/comments/" + revision + "/" + commentId + "/");
+            var overrideUrl = GetRepositoryUrl(accountName, repository,
+                "commits/" + revision + "/comments/" + revision + "/" + commentId + "/");
             return _sharpBucketV2.Get(new object(), overrideUrl);
         }
 
@@ -750,9 +812,11 @@ internal object GetCommitComment(string accountName, string repository, string r
         /// <param name="revision"></param>
         /// <param name="commentId"></param>
         /// <returns></returns>
-        internal object GetCommitComment(ILogger logger, string accountName, string repository, string revision, int commentId)
+        internal object GetCommitComment(ILogger logger, string accountName, string repository, string revision,
+            int commentId)
         {
-            var overrideUrl = GetRepositoryUrl(accountName, repository, "commits/" + revision + "/comments/" + revision + "/" + commentId + "/");
+            var overrideUrl = GetRepositoryUrl(accountName, repository,
+                "commits/" + revision + "/comments/" + revision + "/" + commentId + "/");
             return _sharpBucketV2.Get(logger, new object(), overrideUrl);
         }
 
@@ -811,7 +875,8 @@ internal object AddNewBuildStatus(string accountName, string repository, string
         /// <param name="revision"></param>
         /// <param name="buildInfo"></param>
         /// <returns></returns>
-        internal object AddNewBuildStatus(ILogger logger, string accountName, string repository, string revision, BuildInfo buildInfo)
+        internal object AddNewBuildStatus(ILogger logger, string accountName, string repository, string revision,
+            BuildInfo buildInfo)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "commit/" + revision + "/statuses/build/");
             return _sharpBucketV2.Post(logger, buildInfo, overrideUrl);
@@ -819,7 +884,8 @@ internal object AddNewBuildStatus(ILogger logger, string accountName, string rep
 
         internal BuildInfo GetBuildStatusInfo(string accountName, string repository, string revision, string key)
         {
-            var overrideUrl = GetRepositoryUrl(accountName, repository, "commit/" + revision + "/statuses/build/" + key);
+            var overrideUrl =
+                GetRepositoryUrl(accountName, repository, "commit/" + revision + "/statuses/build/" + key);
             return _sharpBucketV2.Get(new BuildInfo(), overrideUrl);
         }
 
@@ -832,15 +898,19 @@ internal BuildInfo GetBuildStatusInfo(string accountName, string repository, str
         /// <param name="revision"></param>
         /// <param name="key"></param>
         /// <returns></returns>
-        internal BuildInfo GetBuildStatusInfo(ILogger logger, string accountName, string repository, string revision, string key)
+        internal BuildInfo GetBuildStatusInfo(ILogger logger, string accountName, string repository, string revision,
+            string key)
         {
-            var overrideUrl = GetRepositoryUrl(accountName, repository, "commit/" + revision + "/statuses/build/" + key);
+            var overrideUrl =
+                GetRepositoryUrl(accountName, repository, "commit/" + revision + "/statuses/build/" + key);
             return _sharpBucketV2.Get(logger, new BuildInfo(), overrideUrl);
         }
 
-        internal object ChangeBuildStatusInfo(string accountName, string repository, string revision, string key, BuildInfo buildInfo)
+        internal object ChangeBuildStatusInfo(string accountName, string repository, string revision, string key,
+            BuildInfo buildInfo)
         {
-            var overrideUrl = GetRepositoryUrl(accountName, repository, "commit/" + revision + "/statuses/build/" + key);
+            var overrideUrl =
+                GetRepositoryUrl(accountName, repository, "commit/" + revision + "/statuses/build/" + key);
             return _sharpBucketV2.Put(buildInfo, overrideUrl);
         }
 
@@ -854,9 +924,11 @@ internal object ChangeBuildStatusInfo(string accountName, string repository, str
         /// <param name="key"></param>
         /// <param name="buildInfo"></param>
         /// <returns></returns>
-        internal object ChangeBuildStatusInfo(ILogger logger, string accountName, string repository, string revision, string key, BuildInfo buildInfo)
+        internal object ChangeBuildStatusInfo(ILogger logger, string accountName, string repository, string revision,
+            string key, BuildInfo buildInfo)
         {
-            var overrideUrl = GetRepositoryUrl(accountName, repository, "commit/" + revision + "/statuses/build/" + key);
+            var overrideUrl =
+                GetRepositoryUrl(accountName, repository, "commit/" + revision + "/statuses/build/" + key);
             return _sharpBucketV2.Put(logger, buildInfo, overrideUrl);
         }
 
@@ -942,5 +1014,6 @@ internal List<Tag> ListTags(ILogger logger, string accountName, string repositor
         }
 
         #endregion
+
     }
 }
\ No newline at end of file
diff --git a/SharpBucket/V2/EndPoints/RepositoryResource.cs b/SharpBucket/V2/EndPoints/RepositoryResource.cs
index 726fd8ba..35af1c77 100644
--- a/SharpBucket/V2/EndPoints/RepositoryResource.cs
+++ b/SharpBucket/V2/EndPoints/RepositoryResource.cs
@@ -82,6 +82,28 @@ public Repository PostRepository(ILogger logger, Repository repository)
             return _repositoriesEndPoint.PostRepository(logger, repository, _accountName);
         }
 
+
+        /// <summary>
+        /// Put a specific repository
+        /// </summary>
+        /// <param name="repository"></param>
+        /// <returns></returns>
+        public Repository PutRepository(Repository repository)
+        {
+            return _repositoriesEndPoint.PutRepository(repository, _accountName, repository.name.ToLowerInvariant());
+        }
+
+        /// <summary>
+        /// Put a specific repository
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="repository"></param>
+        /// <returns></returns>
+        public Repository PutRepository(ILogger logger, Repository repository)
+        {
+            return _repositoriesEndPoint.PutRepository(logger, repository, _accountName, repository.name.ToLowerInvariant());
+        }
+
         /// <summary>
         /// Gets the list of accounts watching a repository. 
         /// </summary>
@@ -551,5 +573,7 @@ public object PutDefaultReviewer(ILogger logger, string targetUsername)
         }
 
         #endregion
+
+
     }
 }
\ No newline at end of file

From 20c058b3d8fd055b1828e291509772a0bc3d86d4 Mon Sep 17 00:00:00 2001
From: Anh-Duc Le <Anh-Duc_Le@outlook.com>
Date: Wed, 25 Apr 2018 16:30:15 +0200
Subject: [PATCH 09/21] TeamEndPoin : Added PostProject/ ListProjects /
 GetProject

UserEndPoint: Added GetEmails

POCOS :

+ Added Email/ Group/ LinkConfigurations/ Member/ Project/ ProjectPostParams
+ Modified Branch / Branchrestriction / Repository / User
---
 SharpBucket/SharpBucket.csproj             |   6 +
 SharpBucket/V2/EndPoints/TeamsEndPoint.cs  | 152 +++++++++++++++++++++
 SharpBucket/V2/EndPoints/UserEndpoint.cs   |  36 ++++-
 SharpBucket/V2/Pocos/Branch.cs             |   8 ++
 SharpBucket/V2/Pocos/BranchRestriction.cs  |   4 +-
 SharpBucket/V2/Pocos/Email.cs              |  17 +++
 SharpBucket/V2/Pocos/Group.cs              |  19 +++
 SharpBucket/V2/Pocos/LinkConfigurations.cs |  15 ++
 SharpBucket/V2/Pocos/Member.cs             |   8 ++
 SharpBucket/V2/Pocos/Project.cs            |  22 +++
 SharpBucket/V2/Pocos/ProjectPostParams.cs  |  16 +++
 SharpBucket/V2/Pocos/Repository.cs         |   2 +-
 SharpBucket/V2/Pocos/User.cs               |   2 +
 13 files changed, 304 insertions(+), 3 deletions(-)
 create mode 100644 SharpBucket/V2/Pocos/Email.cs
 create mode 100644 SharpBucket/V2/Pocos/Group.cs
 create mode 100644 SharpBucket/V2/Pocos/LinkConfigurations.cs
 create mode 100644 SharpBucket/V2/Pocos/Member.cs
 create mode 100644 SharpBucket/V2/Pocos/Project.cs
 create mode 100644 SharpBucket/V2/Pocos/ProjectPostParams.cs

diff --git a/SharpBucket/SharpBucket.csproj b/SharpBucket/SharpBucket.csproj
index 3da67654..760ebbbf 100644
--- a/SharpBucket/SharpBucket.csproj
+++ b/SharpBucket/SharpBucket.csproj
@@ -85,13 +85,19 @@
     <Compile Include="V2\Pocos\BranchRestrictionInfo.cs" />
     <Compile Include="V2\Pocos\BuildInfo.cs" />
     <Compile Include="V2\Pocos\CommitInfo.cs" />
+    <Compile Include="V2\Pocos\Email.cs" />
     <Compile Include="V2\Pocos\Fork.cs" />
     <Compile Include="V2\Pocos\ForkInfo.cs" />
+    <Compile Include="V2\Pocos\Group.cs" />
     <Compile Include="V2\Pocos\IteratorBasedPage.cs" />
+    <Compile Include="V2\Pocos\LinkConfigurations.cs" />
     <Compile Include="V2\Pocos\ListBasedPage.cs" />
+    <Compile Include="V2\Pocos\Member.cs" />
     <Compile Include="V2\Pocos\Parent.cs" />
     <Compile Include="V2\Pocos\Activity.cs" />
     <Compile Include="V2\Pocos\ActivityInfo.cs" />
+    <Compile Include="V2\Pocos\Project.cs" />
+    <Compile Include="V2\Pocos\ProjectPostParams.cs" />
     <Compile Include="V2\Pocos\Tag.cs" />
     <Compile Include="V2\Pocos\Update.cs" />
     <Compile Include="V2\Pocos\Comment.cs" />
diff --git a/SharpBucket/V2/EndPoints/TeamsEndPoint.cs b/SharpBucket/V2/EndPoints/TeamsEndPoint.cs
index 5f3ec930..981cb860 100644
--- a/SharpBucket/V2/EndPoints/TeamsEndPoint.cs
+++ b/SharpBucket/V2/EndPoints/TeamsEndPoint.cs
@@ -1,5 +1,6 @@
 using System.Collections.Generic;
 using System.Dynamic;
+using Serilog;
 using SharpBucket.V2.Pocos;
 
 namespace SharpBucket.V2.EndPoints
@@ -24,6 +25,20 @@ public List<Team> GetUserTeams(int max = 0)
             //return _sharpBucketV2.Get<List<Team>>(null, "teams/", parameters);
         }
 
+        /// <summary>
+        /// With Logging
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="max"></param>
+        /// <returns></returns>
+        public List<Team> GetUserTeams(ILogger logger, int max = 0)
+        {
+            dynamic parameters = new ExpandoObject();
+            parameters.role = "member";
+            return GetPaginatedValues<Team>(logger, "teams/", max, parameters);
+            //return _sharpBucketV2.Get<List<Team>>(null, "teams/", parameters);
+        }
+
         /// <summary>
         /// Gets the public information associated with a team. 
         /// If the team's profile is private, the caller must be authenticated and authorized to view this information. 
@@ -34,6 +49,18 @@ public Team GetProfile()
             return _sharpBucketV2.Get(new Team(), _baseUrl);
         }
 
+        /// <summary>
+        /// Gets the public information associated with a team. 
+        /// If the team's profile is private, the caller must be authenticated and authorized to view this information. 
+        /// With Logging
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <returns></returns>
+        public Team GetProfile(ILogger logger)
+        {
+            return _sharpBucketV2.Get(logger, new Team(), _baseUrl);
+        }
+
         /// <summary>
         /// Gets the team's members.
         /// </summary>
@@ -45,6 +72,19 @@ public List<Team> ListMembers(int max = 0)
             return GetPaginatedValues<Team>(overrideUrl, max);
         }
 
+        /// <summary>
+        /// Gets the team's members.
+        /// With Logging
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="max">The maximum number of items to return. 0 returns all items.</param>
+        /// <returns></returns>
+        public List<Team> ListMembers(ILogger logger, int max = 0)
+        {
+            var overrideUrl = _baseUrl + "members/";
+            return GetPaginatedValues<Team>(logger, overrideUrl, max);
+        }
+
         /// <summary>
         /// Gets the list of accounts following the team.
         /// </summary>
@@ -56,6 +96,19 @@ public List<Team> ListFollowers(int max = 0)
             return GetPaginatedValues<Team>(overrideUrl, max);
         }
 
+        /// <summary>
+        /// Gets the list of accounts following the team.
+        /// With Logging
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="max">The maximum number of items to return. 0 returns all items.</param>
+        /// <returns></returns>
+        public List<Team> ListFollowers(ILogger logger, int max = 0)
+        {
+            var overrideUrl = _baseUrl + "followers/";
+            return GetPaginatedValues<Team>(logger, overrideUrl, max);
+        }
+
         /// <summary>
         /// Gets a list of accounts the team is following.
         /// </summary>
@@ -67,6 +120,19 @@ public List<Team> ListFollowing(int max = 0)
             return GetPaginatedValues<Team>(overrideUrl, max);
         }
 
+        /// <summary>
+        /// Gets a list of accounts the team is following.
+        /// With Logging
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="max">The maximum number of items to return. 0 returns all items.</param>
+        /// <returns></returns>
+        public List<Team> ListFollowing(ILogger logger, int max = 0)
+        {
+            var overrideUrl = _baseUrl + "following/";
+            return GetPaginatedValues<Team>(logger, overrideUrl, max);
+        }
+
         /// <summary>
         /// Gets the list of the team's repositories. 
         /// Private repositories only appear on this list if the caller is authenticated and is authorized to view the repository.
@@ -78,5 +144,91 @@ public List<Repository> ListRepositories(int max = 0)
             var overrideUrl = _baseUrl + "repositories/";
             return GetPaginatedValues<Repository>(overrideUrl, max);
         }
+
+        /// <summary>
+        /// Gets the list of the team's repositories. 
+        /// Private repositories only appear on this list if the caller is authenticated and is authorized to view the repository.
+        /// With Logging
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="max">The maximum number of items to return. 0 returns all items.</param>
+        /// <returns></returns>
+        public List<Repository> ListRepositories(ILogger logger, int max = 0)
+        {
+            var overrideUrl = _baseUrl + "repositories/";
+            return GetPaginatedValues<Repository>(logger, overrideUrl, max);
+        }
+
+        /// <summary>
+        /// Get's the list of the team's projects.
+        /// A paginated list of projects that belong to the specified team.
+        /// </summary>
+        /// <param name="max"></param>
+        /// <returns></returns>
+        public List<Project> ListProjects(int max = 0)
+        {
+            var overrideUrl = _baseUrl + "projects/";
+            return GetPaginatedValues<Project>(overrideUrl, max);
+        }
+
+        /// <summary>
+        /// With Logging
+        /// Get's the list of the team's projects.
+        /// A paginated list of projects that belong to the specified team.
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="max"></param>
+        /// <returns></returns>
+        public List<Project> ListProjects(ILogger logger, int max = 0)
+        {
+            var overrideUrl = _baseUrl + "projects/";
+            return GetPaginatedValues<Project>(logger, overrideUrl, max);
+        }
+        /// <summary>
+        /// ProjectPostParams contains solely the required parameters for posting a project
+        /// </summary>
+        /// <param name="project"></param>
+        /// <returns></returns>
+        public ProjectPostParams PostProject(ProjectPostParams project)
+        {
+            var overrideUrl = _baseUrl + "projects/";
+            return _sharpBucketV2.Post(project, overrideUrl);
+        }
+
+        /// <summary>
+        /// With Logging
+        /// ProjectPostParams contains solely the required parameters for posting a project
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="project"></param>
+        /// <returns></returns>
+        public ProjectPostParams PostProject(ILogger logger, ProjectPostParams project)
+        {
+            var overrideUrl = _baseUrl + "projects/";
+            return _sharpBucketV2.Post(logger, project, overrideUrl);
+        }
+        /// <summary>
+        /// Get a project by using 
+        /// </summary>
+        /// <param name="projectKey"></param>
+        /// <returns></returns>
+        public Project GetProject(string projectKey)
+        {
+            var overrideUrl = _baseUrl + "projects/" + projectKey;
+            return _sharpBucketV2.Get(new Project(), overrideUrl);
+        }
+
+        /// <summary>
+        /// With Logging
+        /// Get a project by using project key
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="projectKey"></param>
+        /// <returns></returns>
+        public Project GetProject(ILogger logger, string projectKey)
+        {
+            var overrideUrl = _baseUrl + "projects/" + projectKey;
+            return _sharpBucketV2.Get(logger, new Project(), overrideUrl);
+        }
     }
 }
\ No newline at end of file
diff --git a/SharpBucket/V2/EndPoints/UserEndpoint.cs b/SharpBucket/V2/EndPoints/UserEndpoint.cs
index 7b1ba8fe..8b7bcae4 100644
--- a/SharpBucket/V2/EndPoints/UserEndpoint.cs
+++ b/SharpBucket/V2/EndPoints/UserEndpoint.cs
@@ -1,4 +1,6 @@
-using SharpBucket.V2.Pocos;
+using System.Collections.Generic;
+using Serilog;
+using SharpBucket.V2.Pocos;
 
 namespace SharpBucket.V2.EndPoints
 {
@@ -13,5 +15,37 @@ public User GetUser()
         {
             return _sharpBucketV2.Get<User>(null, _baseUrl);
         }
+
+        /// <summary>
+        /// Get With Logging
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <returns></returns>
+        public User GetUser(ILogger logger)
+        {
+            return _sharpBucketV2.Get<User>(logger, null, _baseUrl);
+        }
+
+        /// <summary>
+        /// Returns all the authenticated user's email addresses. Both confirmed and unconfirmed.
+        /// </summary>
+        /// <returns></returns>
+        public List<Email> GetEmails(int max = 0)
+        {
+            var overrideUrl = _baseUrl + "emails/";
+            return GetPaginatedValues<Email>(overrideUrl, max);
+        }
+
+        /// <summary>
+        /// With Logging
+        /// </summary>
+        /// <param name="logger"></param>
+        /// <param name="max"></param>
+        /// <returns></returns>
+        public object GetEmails(ILogger logger, int max = 0)
+        {
+            var overrideUrl = _baseUrl + "emails/";
+            return GetPaginatedValues<Email>(logger, overrideUrl, max);
+        }
     }
 }
\ No newline at end of file
diff --git a/SharpBucket/V2/Pocos/Branch.cs b/SharpBucket/V2/Pocos/Branch.cs
index 5f07e6dc..8dfff0df 100644
--- a/SharpBucket/V2/Pocos/Branch.cs
+++ b/SharpBucket/V2/Pocos/Branch.cs
@@ -3,5 +3,13 @@
     public class Branch
     {
         public string name { get; set; }
+        public string message { get; set; }
+        public string author { get; set; }
+        public string parents { get; set; }
+        public string files { get; set; }
+        public string branch { get; set; }
+        public string type { get; set; }
+        public string path { get; set; }
+        public Links links { get; set; }
     }
 }
\ No newline at end of file
diff --git a/SharpBucket/V2/Pocos/BranchRestriction.cs b/SharpBucket/V2/Pocos/BranchRestriction.cs
index f3ad0084..34c3d8c1 100644
--- a/SharpBucket/V2/Pocos/BranchRestriction.cs
+++ b/SharpBucket/V2/Pocos/BranchRestriction.cs
@@ -4,11 +4,13 @@ namespace SharpBucket.V2.Pocos
 {
     public class BranchRestriction
     {
-        public List<object> groups { get; set; }
+        public List<Group> groups { get; set; }
         public int? id { get; set; }
         public string kind { get; set; }
         public List<Link> links { get; set; }
         public string pattern { get; set; }
         public List<User> users { get; set; }
+        public int? value { get; set; }
+        public string type { get; set; }
     }
 }
\ No newline at end of file
diff --git a/SharpBucket/V2/Pocos/Email.cs b/SharpBucket/V2/Pocos/Email.cs
new file mode 100644
index 00000000..80907299
--- /dev/null
+++ b/SharpBucket/V2/Pocos/Email.cs
@@ -0,0 +1,17 @@
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using System.Text;
+using System.Threading.Tasks;
+
+namespace SharpBucket.V2.Pocos
+{
+    public class Email
+    {
+        public bool is_primary { get; set; }
+        public bool is_confirmed { get; set; }
+        public string type { get; set; }
+        public string email { get; set; }
+        public Links links { get; set; }
+    }
+}
diff --git a/SharpBucket/V2/Pocos/Group.cs b/SharpBucket/V2/Pocos/Group.cs
new file mode 100644
index 00000000..2d00e66e
--- /dev/null
+++ b/SharpBucket/V2/Pocos/Group.cs
@@ -0,0 +1,19 @@
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using System.Text;
+using System.Threading.Tasks;
+
+namespace SharpBucket.V2.Pocos
+{
+    public class Group
+    {
+        public string name { get; set; }
+        public Links links { get; set; }
+        public string account_privilege { get; set; }
+        public string full_slug { get; set; }
+        public Owner owner { get; set; }
+        public string type { get; set; }
+        public string slug { get; set; }
+    }
+}
diff --git a/SharpBucket/V2/Pocos/LinkConfigurations.cs b/SharpBucket/V2/Pocos/LinkConfigurations.cs
new file mode 100644
index 00000000..6484a238
--- /dev/null
+++ b/SharpBucket/V2/Pocos/LinkConfigurations.cs
@@ -0,0 +1,15 @@
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using System.Text;
+using System.Threading.Tasks;
+
+namespace SharpBucket.V2.Pocos
+{
+    public class LinkConfigurations : Link
+    {
+        public string handler { get; set; }
+        public string link_url { get; set; }
+        public string link_key { get; set; }
+    }
+}
diff --git a/SharpBucket/V2/Pocos/Member.cs b/SharpBucket/V2/Pocos/Member.cs
new file mode 100644
index 00000000..943b5477
--- /dev/null
+++ b/SharpBucket/V2/Pocos/Member.cs
@@ -0,0 +1,8 @@
+namespace SharpBucket.V2.Pocos
+{
+
+    public class Member : User
+    {
+
+    }
+}
diff --git a/SharpBucket/V2/Pocos/Project.cs b/SharpBucket/V2/Pocos/Project.cs
new file mode 100644
index 00000000..3e69c1cc
--- /dev/null
+++ b/SharpBucket/V2/Pocos/Project.cs
@@ -0,0 +1,22 @@
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using System.Text;
+using System.Threading.Tasks;
+
+namespace SharpBucket.V2.Pocos
+{
+    public class Project
+    {
+        public string uuid { get; set; }
+        public Links links { get; set; }
+        public string description { get; set; }
+        public DateTime? created_on { get; set; }
+        public string key { get; set; }
+        public Owner owner { get; set; }
+        public DateTime? updated_on { get; set; }
+        public string type { get; set; }
+        public bool? is_private { get; set; }
+        public string name { get; set; }
+    }
+}
diff --git a/SharpBucket/V2/Pocos/ProjectPostParams.cs b/SharpBucket/V2/Pocos/ProjectPostParams.cs
new file mode 100644
index 00000000..03a188ba
--- /dev/null
+++ b/SharpBucket/V2/Pocos/ProjectPostParams.cs
@@ -0,0 +1,16 @@
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using System.Text;
+using System.Threading.Tasks;
+
+namespace SharpBucket.V2.Pocos
+{
+    public class ProjectPostParams
+    {
+        public string description { get; set; }
+        public string key { get; set; }
+        public bool? is_private { get; set; }
+        public string name { get; set; }
+    }
+}
diff --git a/SharpBucket/V2/Pocos/Repository.cs b/SharpBucket/V2/Pocos/Repository.cs
index 7db6d339..be915a37 100644
--- a/SharpBucket/V2/Pocos/Repository.cs
+++ b/SharpBucket/V2/Pocos/Repository.cs
@@ -11,7 +11,7 @@ public class Repository
         public string created_on { get; set; }
         public string full_name { get; set; }
         public bool? has_issues { get; set; }
-        public string owner { get; set; }
+        public Owner owner { get; set; }
         public string updated_on { get; set; }
         public ulong? size { get; set; }
         public bool? is_private { get; set; }
diff --git a/SharpBucket/V2/Pocos/User.cs b/SharpBucket/V2/Pocos/User.cs
index 8c39bc5a..66952110 100644
--- a/SharpBucket/V2/Pocos/User.cs
+++ b/SharpBucket/V2/Pocos/User.cs
@@ -9,5 +9,7 @@ public class User
         public Links links { get; set; }
         public string created_on { get; set; }
         public string location { get; set; }
+        public string type { get; set; }
+        public string uuid { get; set; }
     }
 }
\ No newline at end of file

From dfcb474c4608df402328f0839a6d86f0c44c9039 Mon Sep 17 00:00:00 2001
From: Anh-Duc Le <Anh-Duc_Le@outlook.com>
Date: Wed, 25 Apr 2018 17:18:12 +0200
Subject: [PATCH 10/21] endpoint and sharpbucket v2 interfaces

---
 SharpBucket/ISharpBucket.cs          | 66 ++++++++++++++++++++
 SharpBucket/SharpBucket.cs           |  2 +-
 SharpBucket/SharpBucket.csproj       |  2 +
 SharpBucket/V2/EndPoints/EndPoint.cs |  8 +++
 SharpBucket/V2/ISharpBucketV2.cs     | 91 ++++++++++++++++++++++++++++
 SharpBucket/V2/SharpBucketV2.cs      |  2 +-
 6 files changed, 169 insertions(+), 2 deletions(-)
 create mode 100644 SharpBucket/ISharpBucket.cs
 create mode 100644 SharpBucket/V2/ISharpBucketV2.cs

diff --git a/SharpBucket/ISharpBucket.cs b/SharpBucket/ISharpBucket.cs
new file mode 100644
index 00000000..8a7dae1b
--- /dev/null
+++ b/SharpBucket/ISharpBucket.cs
@@ -0,0 +1,66 @@
+using Serilog;
+using SharpBucket.Authentication;
+
+namespace SharpBucket
+{
+    public interface ISharpBucket
+    {
+        /// <summary>   
+        /// Use basic authentication with the BitBucket API. OAuth authentication is preferred over
+        /// basic authentication, due to security reasons.
+        /// </summary>
+        /// <param name="username">Your BitBucket user name.</param>
+        /// <param name="password">Your BitBucket password.</param>
+        void BasicAuthentication(string username, string password);
+
+        /// <summary>
+        /// Use 2 legged OAuth 1.0a authentication. This is similar to basic authentication, since
+        /// it requires the same number of steps. It is still safer to use than basic authentication, 
+        /// since you can revoke the API keys.
+        /// More info:
+        /// https://confluence.atlassian.com/display/BITBUCKET/OAuth+on+Bitbucket
+        /// </summary>
+        /// <param name="consumerKey">Your consumer API key obtained from the BitBucket web page.</param>
+        /// <param name="consumerSecretKey">Your consumer secret API key also obtained from the BitBucket web page.</param>
+        void OAuth2LeggedAuthentication(string consumerKey, string consumerSecretKey);
+
+        /// <summary>
+        /// Use 3 legged OAuth 1.0a authentication. This is the most secure one, but for simple uses it might
+        /// be a bit too complex.
+        /// More info:
+        /// https://confluence.atlassian.com/display/BITBUCKET/OAuth+on+Bitbucket
+        /// </summary>
+        /// <param name="consumerKey">Your consumer API key obtained from the BitBucket web page.</param>
+        /// <param name="consumerSecretKey">Your consumer secret API key also obtained from the BitBucket web page.</param>
+        /// <param name="callback">Callback URL to which BitBucket will send the pin.</param>
+        /// <returns></returns>
+        OAuthentication3Legged OAuth3LeggedAuthentication(
+            string consumerKey,
+            string consumerSecretKey,
+            string callback = "oob");
+
+        /// <summary>
+        /// Use 3 legged OAuth 1.0a authentication. Use this method if you have already obtained the OAuthToken
+        /// and OAuthSecretToken. This method can be used so you do not have to go trough the whole 3 legged
+        /// process every time. You can save the tokens you receive the first time and reuse them in another session.
+        /// </summary>
+        /// <param name="consumerKey">Your consumer API key obtained from the BitBucket web page.</param>
+        /// <param name="consumerSecretKey">Your consumer secret API key also obtained from the BitBucket web page.</param>
+        /// <param name="oauthToken">Your OAuth token that was obtained on a previous session.</param>
+        /// <param name="oauthTokenSecret">Your OAuth secret token thata was obtained on a previous session.</param>
+        /// <returns></returns>
+        OAuthentication3Legged OAuth3LeggedAuthentication(
+            string consumerKey,
+            string consumerSecretKey,
+            string oauthToken,
+            string oauthTokenSecret);
+
+        /// <summary>
+        /// Use Oauth2 authentication. This is the neweset version and is prefered.
+        /// </summary>
+        /// <param name="consumerKey"></param>
+        /// <param name="consumerSecretKey"></param>
+        /// <returns></returns>
+        OAuthentication2 OAuthentication2(string consumerKey, string consumerSecretKey);
+    }
+}
\ No newline at end of file
diff --git a/SharpBucket/SharpBucket.cs b/SharpBucket/SharpBucket.cs
index 6acc8caf..e3b075ce 100644
--- a/SharpBucket/SharpBucket.cs
+++ b/SharpBucket/SharpBucket.cs
@@ -13,7 +13,7 @@ namespace SharpBucket
     /// More info:
     /// https://confluence.atlassian.com/display/BITBUCKET/Use+the+Bitbucket+REST+APIs
     /// </summary>
-    public class SharpBucket
+    public class SharpBucket : ISharpBucket
     {
         private Authenticate authenticator;
 
diff --git a/SharpBucket/SharpBucket.csproj b/SharpBucket/SharpBucket.csproj
index 760ebbbf..314eeb23 100644
--- a/SharpBucket/SharpBucket.csproj
+++ b/SharpBucket/SharpBucket.csproj
@@ -55,6 +55,7 @@
     <Compile Include="Authentication\OAuthentication2.cs" />
     <Compile Include="Authentication\RequestExecutor.cs" />
     <Compile Include="Authentication\Token.cs" />
+    <Compile Include="ISharpBucket.cs" />
     <Compile Include="Utility\ObjectToDictionaryHelper.cs" />
     <Compile Include="V1\EndPoints\GroupsEndPoint.cs" />
     <Compile Include="V1\EndPoints\IssueResource.cs" />
@@ -79,6 +80,7 @@
     <Compile Include="V2\EndPoints\RepositoryResource.cs" />
     <Compile Include="V2\EndPoints\TagResource.cs" />
     <Compile Include="V2\EndPoints\UserEndpoint.cs" />
+    <Compile Include="V2\ISharpBucketV2.cs" />
     <Compile Include="V2\Pocos\Author.cs" />
     <Compile Include="V2\Pocos\Branch.cs" />
     <Compile Include="V2\Pocos\BranchRestriction.cs" />
diff --git a/SharpBucket/V2/EndPoints/EndPoint.cs b/SharpBucket/V2/EndPoints/EndPoint.cs
index 94a1a69a..965709b7 100644
--- a/SharpBucket/V2/EndPoints/EndPoint.cs
+++ b/SharpBucket/V2/EndPoints/EndPoint.cs
@@ -21,6 +21,14 @@ public EndPoint(SharpBucketV2 sharpBucketV2, string resourcePath)
             _baseUrl = resourcePath;
         }
 
+
+        public EndPoint(ISharpBucketV2 sharpBucketV2, string resourcePath)
+        {
+            _sharpBucketV2 = (SharpBucketV2) sharpBucketV2;
+            _baseUrl = resourcePath;
+        }
+
+
         /// <summary>
         /// Generator that allows lazy access to paginated resources.
         /// </summary>
diff --git a/SharpBucket/V2/ISharpBucketV2.cs b/SharpBucket/V2/ISharpBucketV2.cs
new file mode 100644
index 00000000..47430710
--- /dev/null
+++ b/SharpBucket/V2/ISharpBucketV2.cs
@@ -0,0 +1,91 @@
+using System.Collections.Generic;
+using Serilog;
+using SharpBucket.Authentication;
+using SharpBucket.V2.EndPoints;
+using SharpBucket.V2.Pocos;
+
+namespace SharpBucket.V2
+{
+    public interface ISharpBucketV2 : ISharpBucket
+    {
+        /// <summary>
+        /// Get the Teams End Point for a specific team.
+        /// </summary>
+        /// <param name="teamName">The team whose team End Point you wish to get.</param>
+        /// <returns></returns>
+        TeamsEndPoint TeamsEndPoint(string teamName);
+
+        /// <summary>
+        /// Get the Repositories End point.
+        /// </summary>
+        /// <returns></returns>
+        RepositoriesEndPoint RepositoriesEndPoint();
+
+        /// <summary>
+        /// Get the UsersEndPoint End Point.
+        /// </summary>
+        /// <param name="accountName">The account for which you wish to get the UsersEndPoint End Point.</param>
+        /// <returns></returns>
+        UsersEndpoint UsersEndPoint(string accountName);
+
+        UserEndpoint UserEndPoint();
+
+        /// <summary>   
+        /// Use basic authentication with the BitBucket API. OAuth authentication is preferred over
+        /// basic authentication, due to security reasons.
+        /// </summary>
+        /// <param name="username">Your BitBucket user name.</param>
+        /// <param name="password">Your BitBucket password.</param>
+        void BasicAuthentication(string username, string password);
+
+        /// <summary>
+        /// Use 2 legged OAuth 1.0a authentication. This is similar to basic authentication, since
+        /// it requires the same number of steps. It is still safer to use than basic authentication, 
+        /// since you can revoke the API keys.
+        /// More info:
+        /// https://confluence.atlassian.com/display/BITBUCKET/OAuth+on+Bitbucket
+        /// </summary>
+        /// <param name="consumerKey">Your consumer API key obtained from the BitBucket web page.</param>
+        /// <param name="consumerSecretKey">Your consumer secret API key also obtained from the BitBucket web page.</param>
+        void OAuth2LeggedAuthentication(string consumerKey, string consumerSecretKey);
+
+        /// <summary>
+        /// Use 3 legged OAuth 1.0a authentication. This is the most secure one, but for simple uses it might
+        /// be a bit too complex.
+        /// More info:
+        /// https://confluence.atlassian.com/display/BITBUCKET/OAuth+on+Bitbucket
+        /// </summary>
+        /// <param name="consumerKey">Your consumer API key obtained from the BitBucket web page.</param>
+        /// <param name="consumerSecretKey">Your consumer secret API key also obtained from the BitBucket web page.</param>
+        /// <param name="callback">Callback URL to which BitBucket will send the pin.</param>
+        /// <returns></returns>
+        OAuthentication3Legged OAuth3LeggedAuthentication(
+            string consumerKey,
+            string consumerSecretKey,
+            string callback = "oob");
+
+        /// <summary>
+        /// Use 3 legged OAuth 1.0a authentication. Use this method if you have already obtained the OAuthToken
+        /// and OAuthSecretToken. This method can be used so you do not have to go trough the whole 3 legged
+        /// process every time. You can save the tokens you receive the first time and reuse them in another session.
+        /// </summary>
+        /// <param name="consumerKey">Your consumer API key obtained from the BitBucket web page.</param>
+        /// <param name="consumerSecretKey">Your consumer secret API key also obtained from the BitBucket web page.</param>
+        /// <param name="oauthToken">Your OAuth token that was obtained on a previous session.</param>
+        /// <param name="oauthTokenSecret">Your OAuth secret token thata was obtained on a previous session.</param>
+        /// <returns></returns>
+        OAuthentication3Legged OAuth3LeggedAuthentication(
+            string consumerKey,
+            string consumerSecretKey,
+            string oauthToken,
+            string oauthTokenSecret);
+
+        /// <summary>
+        /// Use Oauth2 authentication. This is the neweset version and is prefered.
+        /// </summary>
+        /// <param name="consumerKey"></param>
+        /// <param name="consumerSecretKey"></param>
+        /// <returns></returns>
+        OAuthentication2 OAuthentication2(string consumerKey, string consumerSecretKey);
+    }
+}
\ No newline at end of file
diff --git a/SharpBucket/V2/SharpBucketV2.cs b/SharpBucket/V2/SharpBucketV2.cs
index eba6b0fb..5c1d4c9a 100644
--- a/SharpBucket/V2/SharpBucketV2.cs
+++ b/SharpBucket/V2/SharpBucketV2.cs
@@ -7,7 +7,7 @@ namespace SharpBucket.V2
     /// You can read more about the V2 of the API here:
     /// https://confluence.atlassian.com/display/BITBUCKET/Version+2
     /// </summary>
-    public sealed class SharpBucketV2 : SharpBucket
+    public sealed class SharpBucketV2 : SharpBucket, ISharpBucketV2
     {
         internal const string BITBUCKET_URL = "https://api.bitbucket.org/2.0";
 

From 313491ee0516c2048e4cc5f4adc9b36765060196 Mon Sep 17 00:00:00 2001
From: Anh-Duc Le <Anh-Duc_Le@outlook.com>
Date: Thu, 26 Apr 2018 09:44:57 +0200
Subject: [PATCH 11/21] Added ISharpBucket and ISharpBucketV2 interfaces to
 enable mocking for testing

---
 SharpBucket/ISharpBucket.cs                      | 13 +++++++++++++
 SharpBucket/SharpBucket.cs                       |  2 +-
 SharpBucket/SharpBucket.csproj                   |  2 ++
 SharpBucket/V2/EndPoints/EndPoint.cs             |  6 ++++++
 SharpBucket/V2/EndPoints/RepositoriesEndPoint.cs |  5 +++++
 SharpBucket/V2/EndPoints/TeamsEndPoint.cs        |  5 +++++
 SharpBucket/V2/EndPoints/UserEndpoint.cs         |  4 ++++
 SharpBucket/V2/EndPoints/UsersEndpoint.cs        |  6 ++++++
 SharpBucket/V2/ISharpBucketV2.cs                 | 12 ++++++++++++
 SharpBucket/V2/SharpBucketV2.cs                  |  2 +-
 10 files changed, 55 insertions(+), 2 deletions(-)
 create mode 100644 SharpBucket/ISharpBucket.cs
 create mode 100644 SharpBucket/V2/ISharpBucketV2.cs

diff --git a/SharpBucket/ISharpBucket.cs b/SharpBucket/ISharpBucket.cs
new file mode 100644
index 00000000..dfa7b5ee
--- /dev/null
+++ b/SharpBucket/ISharpBucket.cs
@@ -0,0 +1,13 @@
+using SharpBucket.Authentication;
+
+namespace SharpBucket
+{
+    public interface ISharpBucket
+    {
+        void BasicAuthentication(string username, string password);
+        void OAuth2LeggedAuthentication(string consumerKey, string consumerSecretKey);
+        OAuthentication3Legged OAuth3LeggedAuthentication(string consumerKey, string consumerSecretKey, string callback = "oob");
+        OAuthentication3Legged OAuth3LeggedAuthentication(string consumerKey, string consumerSecretKey, string oauthToken, string oauthTokenSecret);
+        OAuthentication2 OAuthentication2(string consumerKey, string consumerSecretKey);
+    }
+}
\ No newline at end of file
diff --git a/SharpBucket/SharpBucket.cs b/SharpBucket/SharpBucket.cs
index 7b5fbab4..bb351678 100644
--- a/SharpBucket/SharpBucket.cs
+++ b/SharpBucket/SharpBucket.cs
@@ -12,7 +12,7 @@ namespace SharpBucket
     /// More info:
     /// https://confluence.atlassian.com/display/BITBUCKET/Use+the+Bitbucket+REST+APIs
     /// </summary>
-    public class SharpBucket
+    public class SharpBucket : ISharpBucket
     {
         private Authenticate authenticator;
 
diff --git a/SharpBucket/SharpBucket.csproj b/SharpBucket/SharpBucket.csproj
index c550d455..599981c2 100644
--- a/SharpBucket/SharpBucket.csproj
+++ b/SharpBucket/SharpBucket.csproj
@@ -51,6 +51,7 @@
     <Compile Include="Authentication\OAuthentication2.cs" />
     <Compile Include="Authentication\RequestExecutor.cs" />
     <Compile Include="Authentication\Token.cs" />
+    <Compile Include="ISharpBucket.cs" />
     <Compile Include="Utility\ObjectToDictionaryHelper.cs" />
     <Compile Include="V1\EndPoints\GroupsEndPoint.cs" />
     <Compile Include="V1\EndPoints\IssueResource.cs" />
@@ -75,6 +76,7 @@
     <Compile Include="V2\EndPoints\RepositoryResource.cs" />
     <Compile Include="V2\EndPoints\TagResource.cs" />
     <Compile Include="V2\EndPoints\UserEndpoint.cs" />
+    <Compile Include="V2\ISharpBucketV2.cs" />
     <Compile Include="V2\Pocos\Author.cs" />
     <Compile Include="V2\Pocos\Branch.cs" />
     <Compile Include="V2\Pocos\BranchRestriction.cs" />
diff --git a/SharpBucket/V2/EndPoints/EndPoint.cs b/SharpBucket/V2/EndPoints/EndPoint.cs
index a088c107..7fc772ea 100644
--- a/SharpBucket/V2/EndPoints/EndPoint.cs
+++ b/SharpBucket/V2/EndPoints/EndPoint.cs
@@ -20,6 +20,12 @@ public EndPoint(SharpBucketV2 sharpBucketV2, string resourcePath)
             _baseUrl = resourcePath;
         }
 
+        public EndPoint(ISharpBucketV2 sharpBucketV2, string resourcePath)
+        {
+            _sharpBucketV2 = (SharpBucketV2)sharpBucketV2;
+            _baseUrl = resourcePath;
+        }
+
         /// <summary>
         /// Generator that allows lazy access to paginated resources.
         /// </summary>
diff --git a/SharpBucket/V2/EndPoints/RepositoriesEndPoint.cs b/SharpBucket/V2/EndPoints/RepositoriesEndPoint.cs
index 681462a4..c901dff9 100644
--- a/SharpBucket/V2/EndPoints/RepositoriesEndPoint.cs
+++ b/SharpBucket/V2/EndPoints/RepositoriesEndPoint.cs
@@ -21,6 +21,11 @@ public RepositoriesEndPoint(SharpBucketV2 sharpBucketV2)
         {
         }
 
+        public RepositoriesEndPoint(ISharpBucketV2 sharpBucketV2)
+            : base(sharpBucketV2, "repositories/")
+        {
+        }
+
         /// <summary>
         /// List of repositories associated with an account. If the caller is properly authenticated and authorized, 
         /// this method returns a collection containing public and private repositories. 
diff --git a/SharpBucket/V2/EndPoints/TeamsEndPoint.cs b/SharpBucket/V2/EndPoints/TeamsEndPoint.cs
index 5f3ec930..93ab4dd8 100644
--- a/SharpBucket/V2/EndPoints/TeamsEndPoint.cs
+++ b/SharpBucket/V2/EndPoints/TeamsEndPoint.cs
@@ -15,6 +15,11 @@ public TeamsEndPoint(SharpBucketV2 sharpBucketV2, string teamName)
             : base(sharpBucketV2, "teams/" + teamName + "/")
         {
         }
+        public TeamsEndPoint(ISharpBucketV2 sharpBucketV2, string teamName)
+            : base(sharpBucketV2, "teams/" + teamName + "/")
+        {
+        }
+
 
         public List<Team> GetUserTeams(int max = 0)
         {
diff --git a/SharpBucket/V2/EndPoints/UserEndpoint.cs b/SharpBucket/V2/EndPoints/UserEndpoint.cs
index 7b1ba8fe..cbd66fff 100644
--- a/SharpBucket/V2/EndPoints/UserEndpoint.cs
+++ b/SharpBucket/V2/EndPoints/UserEndpoint.cs
@@ -8,6 +8,10 @@ public UserEndpoint(SharpBucketV2 sharpBucketV2)
             : base(sharpBucketV2, "user/")
         {
         }
+        public UserEndpoint(ISharpBucketV2 sharpBucketV2)
+            : base(sharpBucketV2, "user/")
+        {
+        }
 
         public User GetUser()
         {
diff --git a/SharpBucket/V2/EndPoints/UsersEndpoint.cs b/SharpBucket/V2/EndPoints/UsersEndpoint.cs
index 27846268..58d79d6d 100644
--- a/SharpBucket/V2/EndPoints/UsersEndpoint.cs
+++ b/SharpBucket/V2/EndPoints/UsersEndpoint.cs
@@ -18,6 +18,12 @@ public UsersEndpoint(string accountName, SharpBucketV2 sharpBucketV2) :
             _repositoriesUrl = "repositories/" + accountName + "/";
         }
 
+        public UsersEndpoint(string accountName, ISharpBucketV2 sharpBucketV2) :
+            base(sharpBucketV2, "users/" + accountName + "/")
+        {
+            _repositoriesUrl = "repositories/" + accountName + "/";
+        }
+
         /// <summary>
         /// Gets the public information associated with a user. 
         /// If the user's profile is private, the caller must be authenticated as the account holder to view this information.  
diff --git a/SharpBucket/V2/ISharpBucketV2.cs b/SharpBucket/V2/ISharpBucketV2.cs
new file mode 100644
index 00000000..7b685d02
--- /dev/null
+++ b/SharpBucket/V2/ISharpBucketV2.cs
@@ -0,0 +1,12 @@
+using SharpBucket.V2.EndPoints;
+
+namespace SharpBucket.V2
+{
+    public interface ISharpBucketV2 :ISharpBucket
+    {
+        RepositoriesEndPoint RepositoriesEndPoint();
+        TeamsEndPoint TeamsEndPoint(string teamName);
+        UserEndpoint UserEndPoint();
+        UsersEndpoint UsersEndPoint(string accountName);
+    }
+}
\ No newline at end of file
diff --git a/SharpBucket/V2/SharpBucketV2.cs b/SharpBucket/V2/SharpBucketV2.cs
index eba6b0fb..5c1d4c9a 100644
--- a/SharpBucket/V2/SharpBucketV2.cs
+++ b/SharpBucket/V2/SharpBucketV2.cs
@@ -7,7 +7,7 @@ namespace SharpBucket.V2
     /// You can read more about the V2 of the API here:
     /// https://confluence.atlassian.com/display/BITBUCKET/Version+2
     /// </summary>
-    public sealed class SharpBucketV2 : SharpBucket
+    public sealed class SharpBucketV2 : SharpBucket, ISharpBucketV2
     {
         internal const string BITBUCKET_URL = "https://api.bitbucket.org/2.0";
 

From 64d8c435729697d61d99f43ba6157e4cd6cce3a9 Mon Sep 17 00:00:00 2001
From: Anh-Duc Le <Anh-Duc_Le@outlook.com>
Date: Thu, 26 Apr 2018 09:48:52 +0200
Subject: [PATCH 12/21] Revert "endpoint and sharpbucket v2 interfaces"

This reverts commit dfcb474c4608df402328f0839a6d86f0c44c9039.
---
 SharpBucket/ISharpBucket.cs          | 66 --------------------
 SharpBucket/SharpBucket.cs           |  2 +-
 SharpBucket/SharpBucket.csproj       |  2 -
 SharpBucket/V2/EndPoints/EndPoint.cs |  8 ---
 SharpBucket/V2/ISharpBucketV2.cs     | 91 ----------------------------
 SharpBucket/V2/SharpBucketV2.cs      |  2 +-
 6 files changed, 2 insertions(+), 169 deletions(-)
 delete mode 100644 SharpBucket/ISharpBucket.cs
 delete mode 100644 SharpBucket/V2/ISharpBucketV2.cs

diff --git a/SharpBucket/ISharpBucket.cs b/SharpBucket/ISharpBucket.cs
deleted file mode 100644
index 8a7dae1b..00000000
--- a/SharpBucket/ISharpBucket.cs
+++ /dev/null
@@ -1,66 +0,0 @@
-using Serilog;
-using SharpBucket.Authentication;
-
-namespace SharpBucket
-{
-    public interface ISharpBucket
-    {
-        /// <summary>   
-        /// Use basic authentication with the BitBucket API. OAuth authentication is preferred over
-        /// basic authentication, due to security reasons.
-        /// </summary>
-        /// <param name="username">Your BitBucket user name.</param>
-        /// <param name="password">Your BitBucket password.</param>
-        void BasicAuthentication(string username, string password);
-
-        /// <summary>
-        /// Use 2 legged OAuth 1.0a authentication. This is similar to basic authentication, since
-        /// it requires the same number of steps. It is still safer to use than basic authentication, 
-        /// since you can revoke the API keys.
-        /// More info:
-        /// https://confluence.atlassian.com/display/BITBUCKET/OAuth+on+Bitbucket
-        /// </summary>
-        /// <param name="consumerKey">Your consumer API key obtained from the BitBucket web page.</param>
-        /// <param name="consumerSecretKey">Your consumer secret API key also obtained from the BitBucket web page.</param>
-        void OAuth2LeggedAuthentication(string consumerKey, string consumerSecretKey);
-
-        /// <summary>
-        /// Use 3 legged OAuth 1.0a authentication. This is the most secure one, but for simple uses it might
-        /// be a bit too complex.
-        /// More info:
-        /// https://confluence.atlassian.com/display/BITBUCKET/OAuth+on+Bitbucket
-        /// </summary>
-        /// <param name="consumerKey">Your consumer API key obtained from the BitBucket web page.</param>
-        /// <param name="consumerSecretKey">Your consumer secret API key also obtained from the BitBucket web page.</param>
-        /// <param name="callback">Callback URL to which BitBucket will send the pin.</param>
-        /// <returns></returns>
-        OAuthentication3Legged OAuth3LeggedAuthentication(
-            string consumerKey,
-            string consumerSecretKey,
-            string callback = "oob");
-
-        /// <summary>
-        /// Use 3 legged OAuth 1.0a authentication. Use this method if you have already obtained the OAuthToken
-        /// and OAuthSecretToken. This method can be used so you do not have to go trough the whole 3 legged
-        /// process every time. You can save the tokens you receive the first time and reuse them in another session.
-        /// </summary>
-        /// <param name="consumerKey">Your consumer API key obtained from the BitBucket web page.</param>
-        /// <param name="consumerSecretKey">Your consumer secret API key also obtained from the BitBucket web page.</param>
-        /// <param name="oauthToken">Your OAuth token that was obtained on a previous session.</param>
-        /// <param name="oauthTokenSecret">Your OAuth secret token thata was obtained on a previous session.</param>
-        /// <returns></returns>
-        OAuthentication3Legged OAuth3LeggedAuthentication(
-            string consumerKey,
-            string consumerSecretKey,
-            string oauthToken,
-            string oauthTokenSecret);
-
-        /// <summary>
-        /// Use Oauth2 authentication. This is the neweset version and is prefered.
-        /// </summary>
-        /// <param name="consumerKey"></param>
-        /// <param name="consumerSecretKey"></param>
-        /// <returns></returns>
-        OAuthentication2 OAuthentication2(string consumerKey, string consumerSecretKey);
-    }
-}
\ No newline at end of file
diff --git a/SharpBucket/SharpBucket.cs b/SharpBucket/SharpBucket.cs
index e3b075ce..6acc8caf 100644
--- a/SharpBucket/SharpBucket.cs
+++ b/SharpBucket/SharpBucket.cs
@@ -13,7 +13,7 @@ namespace SharpBucket
     /// More info:
     /// https://confluence.atlassian.com/display/BITBUCKET/Use+the+Bitbucket+REST+APIs
     /// </summary>
-    public class SharpBucket : ISharpBucket
+    public class SharpBucket
     {
         private Authenticate authenticator;
 
diff --git a/SharpBucket/SharpBucket.csproj b/SharpBucket/SharpBucket.csproj
index 314eeb23..760ebbbf 100644
--- a/SharpBucket/SharpBucket.csproj
+++ b/SharpBucket/SharpBucket.csproj
@@ -55,7 +55,6 @@
     <Compile Include="Authentication\OAuthentication2.cs" />
     <Compile Include="Authentication\RequestExecutor.cs" />
     <Compile Include="Authentication\Token.cs" />
-    <Compile Include="ISharpBucket.cs" />
     <Compile Include="Utility\ObjectToDictionaryHelper.cs" />
     <Compile Include="V1\EndPoints\GroupsEndPoint.cs" />
     <Compile Include="V1\EndPoints\IssueResource.cs" />
@@ -80,7 +79,6 @@
     <Compile Include="V2\EndPoints\RepositoryResource.cs" />
     <Compile Include="V2\EndPoints\TagResource.cs" />
     <Compile Include="V2\EndPoints\UserEndpoint.cs" />
-    <Compile Include="V2\ISharpBucketV2.cs" />
     <Compile Include="V2\Pocos\Author.cs" />
     <Compile Include="V2\Pocos\Branch.cs" />
     <Compile Include="V2\Pocos\BranchRestriction.cs" />
diff --git a/SharpBucket/V2/EndPoints/EndPoint.cs b/SharpBucket/V2/EndPoints/EndPoint.cs
index 965709b7..94a1a69a 100644
--- a/SharpBucket/V2/EndPoints/EndPoint.cs
+++ b/SharpBucket/V2/EndPoints/EndPoint.cs
@@ -21,14 +21,6 @@ public EndPoint(SharpBucketV2 sharpBucketV2, string resourcePath)
             _baseUrl = resourcePath;
         }
 
-
-        public EndPoint(ISharpBucketV2 sharpBucketV2, string resourcePath)
-        {
-            _sharpBucketV2 = (SharpBucketV2) sharpBucketV2;
-            _baseUrl = resourcePath;
-        }
-
-
         /// <summary>
         /// Generator that allows lazy access to paginated resources.
         /// </summary>
diff --git a/SharpBucket/V2/ISharpBucketV2.cs b/SharpBucket/V2/ISharpBucketV2.cs
deleted file mode 100644
index 47430710..00000000
--- a/SharpBucket/V2/ISharpBucketV2.cs
+++ /dev/null
@@ -1,91 +0,0 @@
-using System.Collections.Generic;
-using Serilog;
-using SharpBucket.Authentication;
-using SharpBucket.V2.EndPoints;
-using SharpBucket.V2.Pocos;
-
-namespace SharpBucket.V2
-{
-    public interface ISharpBucketV2 : ISharpBucket
-    {
-        /// <summary>
-        /// Get the Teams End Point for a specific team.
-        /// </summary>
-        /// <param name="teamName">The team whose team End Point you wish to get.</param>
-        /// <returns></returns>
-        TeamsEndPoint TeamsEndPoint(string teamName);
-
-        /// <summary>
-        /// Get the Repositories End point.
-        /// </summary>
-        /// <returns></returns>
-        RepositoriesEndPoint RepositoriesEndPoint();
-
-        /// <summary>
-        /// Get the UsersEndPoint End Point.
-        /// </summary>
-        /// <param name="accountName">The account for which you wish to get the UsersEndPoint End Point.</param>
-        /// <returns></returns>
-        UsersEndpoint UsersEndPoint(string accountName);
-
-        UserEndpoint UserEndPoint();
-
-        /// <summary>   
-        /// Use basic authentication with the BitBucket API. OAuth authentication is preferred over
-        /// basic authentication, due to security reasons.
-        /// </summary>
-        /// <param name="username">Your BitBucket user name.</param>
-        /// <param name="password">Your BitBucket password.</param>
-        void BasicAuthentication(string username, string password);
-
-        /// <summary>
-        /// Use 2 legged OAuth 1.0a authentication. This is similar to basic authentication, since
-        /// it requires the same number of steps. It is still safer to use than basic authentication, 
-        /// since you can revoke the API keys.
-        /// More info:
-        /// https://confluence.atlassian.com/display/BITBUCKET/OAuth+on+Bitbucket
-        /// </summary>
-        /// <param name="consumerKey">Your consumer API key obtained from the BitBucket web page.</param>
-        /// <param name="consumerSecretKey">Your consumer secret API key also obtained from the BitBucket web page.</param>
-        void OAuth2LeggedAuthentication(string consumerKey, string consumerSecretKey);
-
-        /// <summary>
-        /// Use 3 legged OAuth 1.0a authentication. This is the most secure one, but for simple uses it might
-        /// be a bit too complex.
-        /// More info:
-        /// https://confluence.atlassian.com/display/BITBUCKET/OAuth+on+Bitbucket
-        /// </summary>
-        /// <param name="consumerKey">Your consumer API key obtained from the BitBucket web page.</param>
-        /// <param name="consumerSecretKey">Your consumer secret API key also obtained from the BitBucket web page.</param>
-        /// <param name="callback">Callback URL to which BitBucket will send the pin.</param>
-        /// <returns></returns>
-        OAuthentication3Legged OAuth3LeggedAuthentication(
-            string consumerKey,
-            string consumerSecretKey,
-            string callback = "oob");
-
-        /// <summary>
-        /// Use 3 legged OAuth 1.0a authentication. Use this method if you have already obtained the OAuthToken
-        /// and OAuthSecretToken. This method can be used so you do not have to go trough the whole 3 legged
-        /// process every time. You can save the tokens you receive the first time and reuse them in another session.
-        /// </summary>
-        /// <param name="consumerKey">Your consumer API key obtained from the BitBucket web page.</param>
-        /// <param name="consumerSecretKey">Your consumer secret API key also obtained from the BitBucket web page.</param>
-        /// <param name="oauthToken">Your OAuth token that was obtained on a previous session.</param>
-        /// <param name="oauthTokenSecret">Your OAuth secret token thata was obtained on a previous session.</param>
-        /// <returns></returns>
-        OAuthentication3Legged OAuth3LeggedAuthentication(
-            string consumerKey,
-            string consumerSecretKey,
-            string oauthToken,
-            string oauthTokenSecret);
-
-        /// <summary>
-        /// Use Oauth2 authentication. This is the neweset version and is prefered.
-        /// </summary>
-        /// <param name="consumerKey"></param>
-        /// <param name="consumerSecretKey"></param>
-        /// <returns></returns>
-        OAuthentication2 OAuthentication2(string consumerKey, string consumerSecretKey);
-    }
-}
\ No newline at end of file
diff --git a/SharpBucket/V2/SharpBucketV2.cs b/SharpBucket/V2/SharpBucketV2.cs
index 5c1d4c9a..eba6b0fb 100644
--- a/SharpBucket/V2/SharpBucketV2.cs
+++ b/SharpBucket/V2/SharpBucketV2.cs
@@ -7,7 +7,7 @@ namespace SharpBucket.V2
     /// You can read more about the V2 of the API here:
     /// https://confluence.atlassian.com/display/BITBUCKET/Version+2
     /// </summary>
-    public sealed class SharpBucketV2 : SharpBucket, ISharpBucketV2
+    public sealed class SharpBucketV2 : SharpBucket
     {
         internal const string BITBUCKET_URL = "https://api.bitbucket.org/2.0";
 

From 4f83d6eb8c293434b77aa76cabb2af95310b6ada Mon Sep 17 00:00:00 2001
From: Anh-Duc Le <Anh-Duc_Le@outlook.com>
Date: Thu, 26 Apr 2018 09:59:45 +0200
Subject: [PATCH 13/21] SharpBucket interfacing

---
 SharpBucket/ISharpBucket.cs      | 56 +++++++++++++++++++-
 SharpBucket/SharpBucket.cs       | 48 +++--------------
 SharpBucket/SharpBucket.csproj   |  2 +-
 SharpBucket/V2/ISharpBucketV2.cs | 88 ++++++++++++++++++++++++++++++--
 SharpBucket/V2/SharpBucketV2.cs  | 18 ++-----
 5 files changed, 149 insertions(+), 63 deletions(-)

diff --git a/SharpBucket/ISharpBucket.cs b/SharpBucket/ISharpBucket.cs
index dfa7b5ee..d182ae3b 100644
--- a/SharpBucket/ISharpBucket.cs
+++ b/SharpBucket/ISharpBucket.cs
@@ -4,10 +4,62 @@ namespace SharpBucket
 {
     public interface ISharpBucket
     {
+        /// <summary>   
+        /// Use basic authentication with the BitBucket API. OAuth authentication is preferred over
+        /// basic authentication, due to security reasons.
+        /// </summary>
+        /// <param name="username">Your BitBucket user name.</param>
+        /// <param name="password">Your BitBucket password.</param>
         void BasicAuthentication(string username, string password);
+
+        /// <summary>
+        /// Use 2 legged OAuth 1.0a authentication. This is similar to basic authentication, since
+        /// it requires the same number of steps. It is still safer to use than basic authentication, 
+        /// since you can revoke the API keys.
+        /// More info:
+        /// https://confluence.atlassian.com/display/BITBUCKET/OAuth+on+Bitbucket
+        /// </summary>
+        /// <param name="consumerKey">Your consumer API key obtained from the BitBucket web page.</param>
+        /// <param name="consumerSecretKey">Your consumer secret API key also obtained from the BitBucket web page.</param>
         void OAuth2LeggedAuthentication(string consumerKey, string consumerSecretKey);
-        OAuthentication3Legged OAuth3LeggedAuthentication(string consumerKey, string consumerSecretKey, string callback = "oob");
-        OAuthentication3Legged OAuth3LeggedAuthentication(string consumerKey, string consumerSecretKey, string oauthToken, string oauthTokenSecret);
+
+        /// <summary>
+        /// Use 3 legged OAuth 1.0a authentication. This is the most secure one, but for simple uses it might
+        /// be a bit too complex.
+        /// More info:
+        /// https://confluence.atlassian.com/display/BITBUCKET/OAuth+on+Bitbucket
+        /// </summary>
+        /// <param name="consumerKey">Your consumer API key obtained from the BitBucket web page.</param>
+        /// <param name="consumerSecretKey">Your consumer secret API key also obtained from the BitBucket web page.</param>
+        /// <param name="callback">Callback URL to which BitBucket will send the pin.</param>
+        /// <returns></returns>
+        OAuthentication3Legged OAuth3LeggedAuthentication(
+            string consumerKey,
+            string consumerSecretKey,
+            string callback = "oob");
+
+        /// <summary>
+        /// Use 3 legged OAuth 1.0a authentication. Use this method if you have already obtained the OAuthToken
+        /// and OAuthSecretToken. This method can be used so you do not have to go trough the whole 3 legged
+        /// process every time. You can save the tokens you receive the first time and reuse them in another session.
+        /// </summary>
+        /// <param name="consumerKey">Your consumer API key obtained from the BitBucket web page.</param>
+        /// <param name="consumerSecretKey">Your consumer secret API key also obtained from the BitBucket web page.</param>
+        /// <param name="oauthToken">Your OAuth token that was obtained on a previous session.</param>
+        /// <param name="oauthTokenSecret">Your OAuth secret token thata was obtained on a previous session.</param>
+        /// <returns></returns>
+        OAuthentication3Legged OAuth3LeggedAuthentication(
+            string consumerKey,
+            string consumerSecretKey,
+            string oauthToken,
+            string oauthTokenSecret);
+
+        /// <summary>
+        /// Use Oauth2 authentication. This is the neweset version and is prefered.
+        /// </summary>
+        /// <param name="consumerKey"></param>
+        /// <param name="consumerSecretKey"></param>
+        /// <returns></returns>
         OAuthentication2 OAuthentication2(string consumerKey, string consumerSecretKey);
     }
 }
\ No newline at end of file
diff --git a/SharpBucket/SharpBucket.cs b/SharpBucket/SharpBucket.cs
index bb351678..6b0c5795 100644
--- a/SharpBucket/SharpBucket.cs
+++ b/SharpBucket/SharpBucket.cs
@@ -21,41 +21,19 @@ public class SharpBucket : ISharpBucket
         /// </summary>
         protected string _baseUrl;
 
-        /// <summary>   
-        /// Use basic authentication with the BitBucket API. OAuth authentication is preferred over
-        /// basic authentication, due to security reasons.
-        /// </summary>
-        /// <param name="username">Your BitBucket user name.</param>
-        /// <param name="password">Your BitBucket password.</param>
+        /// <inheritdoc />
         public void BasicAuthentication(string username, string password)
         {
             authenticator = new BasicAuthentication(username, password, _baseUrl);
         }
 
-        /// <summary>
-        /// Use 2 legged OAuth 1.0a authentication. This is similar to basic authentication, since
-        /// it requires the same number of steps. It is still safer to use than basic authentication, 
-        /// since you can revoke the API keys.
-        /// More info:
-        /// https://confluence.atlassian.com/display/BITBUCKET/OAuth+on+Bitbucket
-        /// </summary>
-        /// <param name="consumerKey">Your consumer API key obtained from the BitBucket web page.</param>
-        /// <param name="consumerSecretKey">Your consumer secret API key also obtained from the BitBucket web page.</param>
+        /// <inheritdoc />
         public void OAuth2LeggedAuthentication(string consumerKey, string consumerSecretKey)
         {
             authenticator = new OAuthentication2Legged(consumerKey, consumerSecretKey, _baseUrl);
         }
 
-        /// <summary>
-        /// Use 3 legged OAuth 1.0a authentication. This is the most secure one, but for simple uses it might
-        /// be a bit too complex.
-        /// More info:
-        /// https://confluence.atlassian.com/display/BITBUCKET/OAuth+on+Bitbucket
-        /// </summary>
-        /// <param name="consumerKey">Your consumer API key obtained from the BitBucket web page.</param>
-        /// <param name="consumerSecretKey">Your consumer secret API key also obtained from the BitBucket web page.</param>
-        /// <param name="callback">Callback URL to which BitBucket will send the pin.</param>
-        /// <returns></returns>
+        /// <inheritdoc />
         public OAuthentication3Legged OAuth3LeggedAuthentication(
             string consumerKey,
             string consumerSecretKey,
@@ -64,17 +42,8 @@ public OAuthentication3Legged OAuth3LeggedAuthentication(
             authenticator = new OAuthentication3Legged(consumerKey, consumerSecretKey, callback, _baseUrl);
             return (OAuthentication3Legged)authenticator;
         }
-
-        /// <summary>
-        /// Use 3 legged OAuth 1.0a authentication. Use this method if you have already obtained the OAuthToken
-        /// and OAuthSecretToken. This method can be used so you do not have to go trough the whole 3 legged
-        /// process every time. You can save the tokens you receive the first time and reuse them in another session.
-        /// </summary>
-        /// <param name="consumerKey">Your consumer API key obtained from the BitBucket web page.</param>
-        /// <param name="consumerSecretKey">Your consumer secret API key also obtained from the BitBucket web page.</param>
-        /// <param name="oauthToken">Your OAuth token that was obtained on a previous session.</param>
-        /// <param name="oauthTokenSecret">Your OAuth secret token thata was obtained on a previous session.</param>
-        /// <returns></returns>
+        
+        /// <inheritdoc />
         public OAuthentication3Legged OAuth3LeggedAuthentication(
             string consumerKey,
             string consumerSecretKey,
@@ -90,12 +59,7 @@ public OAuthentication3Legged OAuth3LeggedAuthentication(
             return (OAuthentication3Legged)authenticator;
         }
 
-        /// <summary>
-        /// Use Oauth2 authentication. This is the neweset version and is prefered.
-        /// </summary>
-        /// <param name="consumerKey"></param>
-        /// <param name="consumerSecretKey"></param>
-        /// <returns></returns>
+        /// <inheritdoc />
         public OAuthentication2 OAuthentication2(string consumerKey, string consumerSecretKey)
         {
             authenticator = new OAuthentication2(consumerKey, consumerSecretKey, _baseUrl);
diff --git a/SharpBucket/SharpBucket.csproj b/SharpBucket/SharpBucket.csproj
index 599981c2..0ca6f607 100644
--- a/SharpBucket/SharpBucket.csproj
+++ b/SharpBucket/SharpBucket.csproj
@@ -113,7 +113,6 @@
     <Compile Include="V2\Pocos\Team.cs" />
     <Compile Include="V2\Pocos\Watcher.cs" />
     <Compile Include="V2\Pocos\WatcherInfo.cs" />
-    <Compile Include="V2\SharpBucketV2.cs" />
     <Compile Include="V1\Pocos\IssueFollowers.cs" />
     <Compile Include="SharpBucket.cs" />
     <Compile Include="V2\EndPoints\TeamsEndPoint.cs" />
@@ -154,6 +153,7 @@
     <Compile Include="V1\Pocos\Wiki.cs" />
     <Compile Include="Properties\AssemblyInfo.cs" />
     <Compile Include="V1\SharpBucketV1.cs" />
+    <Compile Include="V2\SharpBucketV2.cs" />
   </ItemGroup>
   <ItemGroup>
     <None Include="packages.config" />
diff --git a/SharpBucket/V2/ISharpBucketV2.cs b/SharpBucket/V2/ISharpBucketV2.cs
index 7b685d02..ee207bc4 100644
--- a/SharpBucket/V2/ISharpBucketV2.cs
+++ b/SharpBucket/V2/ISharpBucketV2.cs
@@ -1,12 +1,92 @@
-using SharpBucket.V2.EndPoints;
+using SharpBucket.Authentication;
+using SharpBucket.V2.EndPoints;
 
 namespace SharpBucket.V2
 {
-    public interface ISharpBucketV2 :ISharpBucket
+    public interface ISharpBucketV2
     {
-        RepositoriesEndPoint RepositoriesEndPoint();
+        /// <summary>
+        /// Get the Teams End Point for a specific team.
+        /// </summary>
+        /// <param name="teamName">The team whose team End Point you wish to get.</param>
+        /// <returns></returns>
         TeamsEndPoint TeamsEndPoint(string teamName);
-        UserEndpoint UserEndPoint();
+
+        /// <summary>
+        /// Get the Repositories End point.
+        /// </summary>
+        /// <returns></returns>
+        RepositoriesEndPoint RepositoriesEndPoint();
+
+        /// <summary>
+        /// Get the UsersEndPoint End Point.
+        /// </summary>
+        /// <param name="accountName">The account for which you wish to get the UsersEndPoint End Point.</param>
+        /// <returns></returns>
         UsersEndpoint UsersEndPoint(string accountName);
+
+        /// <summary>
+        /// Get the UserEndpoint End Point
+        /// </summary>
+        /// <returns></returns>
+        UserEndpoint UserEndPoint();
+
+        /// <summary>   
+        /// Use basic authentication with the BitBucket API. OAuth authentication is preferred over
+        /// basic authentication, due to security reasons.
+        /// </summary>
+        /// <param name="username">Your BitBucket user name.</param>
+        /// <param name="password">Your BitBucket password.</param>
+        void BasicAuthentication(string username, string password);
+
+        /// <summary>
+        /// Use 2 legged OAuth 1.0a authentication. This is similar to basic authentication, since
+        /// it requires the same number of steps. It is still safer to use than basic authentication, 
+        /// since you can revoke the API keys.
+        /// More info:
+        /// https://confluence.atlassian.com/display/BITBUCKET/OAuth+on+Bitbucket
+        /// </summary>
+        /// <param name="consumerKey">Your consumer API key obtained from the BitBucket web page.</param>
+        /// <param name="consumerSecretKey">Your consumer secret API key also obtained from the BitBucket web page.</param>
+        void OAuth2LeggedAuthentication(string consumerKey, string consumerSecretKey);
+
+        /// <summary>
+        /// Use 3 legged OAuth 1.0a authentication. This is the most secure one, but for simple uses it might
+        /// be a bit too complex.
+        /// More info:
+        /// https://confluence.atlassian.com/display/BITBUCKET/OAuth+on+Bitbucket
+        /// </summary>
+        /// <param name="consumerKey">Your consumer API key obtained from the BitBucket web page.</param>
+        /// <param name="consumerSecretKey">Your consumer secret API key also obtained from the BitBucket web page.</param>
+        /// <param name="callback">Callback URL to which BitBucket will send the pin.</param>
+        /// <returns></returns>
+        OAuthentication3Legged OAuth3LeggedAuthentication(
+           string consumerKey,
+           string consumerSecretKey,
+           string callback = "oob");
+
+        /// <summary>
+        /// Use 3 legged OAuth 1.0a authentication. Use this method if you have already obtained the OAuthToken
+        /// and OAuthSecretToken. This method can be used so you do not have to go trough the whole 3 legged
+        /// process every time. You can save the tokens you receive the first time and reuse them in another session.
+        /// </summary>
+        /// <param name="consumerKey">Your consumer API key obtained from the BitBucket web page.</param>
+        /// <param name="consumerSecretKey">Your consumer secret API key also obtained from the BitBucket web page.</param>
+        /// <param name="oauthToken">Your OAuth token that was obtained on a previous session.</param>
+        /// <param name="oauthTokenSecret">Your OAuth secret token thata was obtained on a previous session.</param>
+        /// <returns></returns>
+        OAuthentication3Legged OAuth3LeggedAuthentication(
+           string consumerKey,
+           string consumerSecretKey,
+           string oauthToken,
+           string oauthTokenSecret);
+
+        /// <summary>
+        /// Use Oauth2 authentication. This is the neweset version and is prefered.
+        /// </summary>
+        /// <param name="consumerKey"></param>
+        /// <param name="consumerSecretKey"></param>
+        /// <returns></returns>
+        OAuthentication2 OAuthentication2(string consumerKey, string consumerSecretKey);
     }
 }
\ No newline at end of file
diff --git a/SharpBucket/V2/SharpBucketV2.cs b/SharpBucket/V2/SharpBucketV2.cs
index 5c1d4c9a..2df3b85a 100644
--- a/SharpBucket/V2/SharpBucketV2.cs
+++ b/SharpBucket/V2/SharpBucketV2.cs
@@ -21,35 +21,25 @@ public SharpBucketV2(string baseUrl)
             _baseUrl = baseUrl;
         }
 
-        /// <summary>
-        /// Get the Teams End Point for a specific team.
-        /// </summary>
-        /// <param name="teamName">The team whose team End Point you wish to get.</param>
-        /// <returns></returns>
+        /// <inheritdoc />
         public TeamsEndPoint TeamsEndPoint(string teamName)
         {
             return new TeamsEndPoint(this, teamName);
         }
 
-        /// <summary>
-        /// Get the Repositories End point.
-        /// </summary>
-        /// <returns></returns>
+        /// <inheritdoc />
         public RepositoriesEndPoint RepositoriesEndPoint()
         {
             return new RepositoriesEndPoint(this);
         }
 
-        /// <summary>
-        /// Get the UsersEndPoint End Point.
-        /// </summary>
-        /// <param name="accountName">The account for which you wish to get the UsersEndPoint End Point.</param>
-        /// <returns></returns>
+        /// <inheritdoc />
         public UsersEndpoint UsersEndPoint(string accountName)
         {
             return new UsersEndpoint(accountName, this);
         }
 
+        /// <inheritdoc />
         public UserEndpoint UserEndPoint()
         {
             return new UserEndpoint(this);

From c6b02225e477cefaf705495553039f0282307618 Mon Sep 17 00:00:00 2001
From: Anh-Duc Le <Anh-Duc_Le@outlook.com>
Date: Thu, 26 Apr 2018 10:29:26 +0200
Subject: [PATCH 14/21] repo slug to ToLowerInvariant, uppercase characters
 will cause issues in the API.

---
 SharpBucket/V2/EndPoints/RepositoriesEndPoint.cs | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/SharpBucket/V2/EndPoints/RepositoriesEndPoint.cs b/SharpBucket/V2/EndPoints/RepositoriesEndPoint.cs
index 681462a4..b98afa92 100644
--- a/SharpBucket/V2/EndPoints/RepositoriesEndPoint.cs
+++ b/SharpBucket/V2/EndPoints/RepositoriesEndPoint.cs
@@ -79,7 +79,7 @@ internal Repository PutRepository(Repository repo, string accountName, string re
 
         internal Repository PostRepository(Repository repo, string accountName)
         {
-            var overrideUrl = GetRepositoryUrl(accountName, repo.name, null);
+            var overrideUrl = GetRepositoryUrl(accountName, repo.name.ToLowerInvariant(), null);
             return _sharpBucketV2.Post(repo, overrideUrl);
         }
 

From 580e73e234d71bbbda4486c1729001f7f1991cd0 Mon Sep 17 00:00:00 2001
From: Anh-Duc Le <Anh-Duc_Le@outlook.com>
Date: Thu, 26 Apr 2018 10:33:14 +0200
Subject: [PATCH 15/21] removed wrong files

---
 ConsoleTests/ConsoleTests.csproj | 3 ---
 ConsoleTests/Program.cs          | 2 --
 ConsoleTests/README.md           | 9 ---------
 3 files changed, 14 deletions(-)
 delete mode 100644 ConsoleTests/README.md

diff --git a/ConsoleTests/ConsoleTests.csproj b/ConsoleTests/ConsoleTests.csproj
index bc64a762..5e715486 100644
--- a/ConsoleTests/ConsoleTests.csproj
+++ b/ConsoleTests/ConsoleTests.csproj
@@ -45,9 +45,6 @@
       <Name>SharpBucket</Name>
     </ProjectReference>
   </ItemGroup>
-  <ItemGroup>
-    <None Include="README.md" />
-  </ItemGroup>
   <Import Project="$(MSBuildToolsPath)\Microsoft.CSharp.targets" />
   <!-- To modify your build process, add your task inside one of the targets below and uncomment it. 
        Other similar extension points exist, see Microsoft.Common.targets.
diff --git a/ConsoleTests/Program.cs b/ConsoleTests/Program.cs
index a0ba0359..0aabb717 100644
--- a/ConsoleTests/Program.cs
+++ b/ConsoleTests/Program.cs
@@ -116,7 +116,6 @@ private static void TestIssuesEndPoint(SharpBucketV1 sharpBucket)
         {
             var issuesResource = sharpBucket.RepositoriesEndPoint(accountName, repository).IssuesResource();
 
-            // TODO: You need at least 5 issues in the test repository.
             int ISSUE_ID = 5;
             // Issues
             var issues = issuesResource.ListIssues();
@@ -181,7 +180,6 @@ private static void TestRepositoriesEndPoint(SharpBucketV1 sharpBucket)
             var newPage = new Wiki { data = "Hello to my new page" };
             var newWiki = repositoriesEndPoint.PostWiki(newPage, "NewPage");
             var changeSet = repositoriesEndPoint.ListChangeset();
-            // TODO: You need at least 5 changesets in your repository.
             var change = changeSet.changesets[4];
             var getChange = repositoriesEndPoint.GetChangeset(change.node);
             var diffStats = repositoriesEndPoint.GetChangesetDiffstat(change.node);
diff --git a/ConsoleTests/README.md b/ConsoleTests/README.md
deleted file mode 100644
index 7c726704..00000000
--- a/ConsoleTests/README.md
+++ /dev/null
@@ -1,9 +0,0 @@
-# ConsoleTests
-
-## Prerequisites
-
-In order to run the ConsoleTests you'll first need the following:
-
-* You need at least 5 changesets in your repository.
-* You need at least 5 issues in your repository.
-* You need at least 1 SSH Key on your user

From 983ba34de9b9ba221eec71850e45f5feb259dddd Mon Sep 17 00:00:00 2001
From: Anh-Duc Le <Anh-Duc_Le@outlook.com>
Date: Thu, 26 Apr 2018 10:37:17 +0200
Subject: [PATCH 16/21] added ToLowerInvariant

---
 SharpBucket/V2/EndPoints/RepositoriesEndPoint.cs | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/SharpBucket/V2/EndPoints/RepositoriesEndPoint.cs b/SharpBucket/V2/EndPoints/RepositoriesEndPoint.cs
index f2a712f0..2fd26eaa 100644
--- a/SharpBucket/V2/EndPoints/RepositoriesEndPoint.cs
+++ b/SharpBucket/V2/EndPoints/RepositoriesEndPoint.cs
@@ -167,7 +167,7 @@ internal Repository PutRepository(ILogger logger, Repository repo, string accoun
 
         internal Repository PostRepository(Repository repo, string accountName)
         {
-            var overrideUrl = GetRepositoryUrl(accountName, repo.name, null);
+            var overrideUrl = GetRepositoryUrl(accountName, repo.name.ToLowerInvariant(), null);
             return _sharpBucketV2.Post(repo, overrideUrl);
         }
 
@@ -180,7 +180,7 @@ internal Repository PostRepository(Repository repo, string accountName)
         /// <returns></returns>
         internal Repository PostRepository(ILogger logger, Repository repo, string accountName)
         {
-            var overrideUrl = GetRepositoryUrl(accountName, repo.name, null);
+            var overrideUrl = GetRepositoryUrl(accountName, repo.name.ToLowerInvariant(), null);
             return _sharpBucketV2.Post(logger, repo, overrideUrl);
         }
 

From fe6f8f3d50ceaf6a8ea6af4324387e3d447c496a Mon Sep 17 00:00:00 2001
From: Anh-Duc Le <Anh-Duc_Le@outlook.com>
Date: Thu, 26 Apr 2018 10:39:05 +0200
Subject: [PATCH 17/21] removed unnecessary files

---
 ConsoleTests/ConsoleTests.csproj | 3 ---
 ConsoleTests/Program.cs          | 2 --
 ConsoleTests/README.md           | 9 ---------
 3 files changed, 14 deletions(-)
 delete mode 100644 ConsoleTests/README.md

diff --git a/ConsoleTests/ConsoleTests.csproj b/ConsoleTests/ConsoleTests.csproj
index bc64a762..5e715486 100644
--- a/ConsoleTests/ConsoleTests.csproj
+++ b/ConsoleTests/ConsoleTests.csproj
@@ -45,9 +45,6 @@
       <Name>SharpBucket</Name>
     </ProjectReference>
   </ItemGroup>
-  <ItemGroup>
-    <None Include="README.md" />
-  </ItemGroup>
   <Import Project="$(MSBuildToolsPath)\Microsoft.CSharp.targets" />
   <!-- To modify your build process, add your task inside one of the targets below and uncomment it. 
        Other similar extension points exist, see Microsoft.Common.targets.
diff --git a/ConsoleTests/Program.cs b/ConsoleTests/Program.cs
index a0ba0359..0aabb717 100644
--- a/ConsoleTests/Program.cs
+++ b/ConsoleTests/Program.cs
@@ -116,7 +116,6 @@ private static void TestIssuesEndPoint(SharpBucketV1 sharpBucket)
         {
             var issuesResource = sharpBucket.RepositoriesEndPoint(accountName, repository).IssuesResource();
 
-            // TODO: You need at least 5 issues in the test repository.
             int ISSUE_ID = 5;
             // Issues
             var issues = issuesResource.ListIssues();
@@ -181,7 +180,6 @@ private static void TestRepositoriesEndPoint(SharpBucketV1 sharpBucket)
             var newPage = new Wiki { data = "Hello to my new page" };
             var newWiki = repositoriesEndPoint.PostWiki(newPage, "NewPage");
             var changeSet = repositoriesEndPoint.ListChangeset();
-            // TODO: You need at least 5 changesets in your repository.
             var change = changeSet.changesets[4];
             var getChange = repositoriesEndPoint.GetChangeset(change.node);
             var diffStats = repositoriesEndPoint.GetChangesetDiffstat(change.node);
diff --git a/ConsoleTests/README.md b/ConsoleTests/README.md
deleted file mode 100644
index 7c726704..00000000
--- a/ConsoleTests/README.md
+++ /dev/null
@@ -1,9 +0,0 @@
-# ConsoleTests
-
-## Prerequisites
-
-In order to run the ConsoleTests you'll first need the following:
-
-* You need at least 5 changesets in your repository.
-* You need at least 5 issues in your repository.
-* You need at least 1 SSH Key on your user

From d9b54a64c161505a354a882798148e696c48c010 Mon Sep 17 00:00:00 2001
From: Anh-Duc Le <Anh-Duc_Le@outlook.com>
Date: Thu, 26 Apr 2018 10:53:04 +0200
Subject: [PATCH 18/21] added public Project project { get; set; }     public
 string slug { get; set; }

---
 .gitignore                         | 1 +
 SharpBucket/V2/Pocos/Repository.cs | 3 +++
 2 files changed, 4 insertions(+)

diff --git a/.gitignore b/.gitignore
index 9404e7fa..7ed7f1b3 100644
--- a/.gitignore
+++ b/.gitignore
@@ -23,3 +23,4 @@ _ReSharper*/
 /packages/
 !packages/repositories.config
 /.vs/SharpBucket/v15/Server/sqlite3
+*.nupkg
diff --git a/SharpBucket/V2/Pocos/Repository.cs b/SharpBucket/V2/Pocos/Repository.cs
index be915a37..e9ea9011 100644
--- a/SharpBucket/V2/Pocos/Repository.cs
+++ b/SharpBucket/V2/Pocos/Repository.cs
@@ -17,5 +17,8 @@ public class Repository
         public bool? is_private { get; set; }
         public string name { get; set; }
         public string uuid { get; set; }
+        public Project project { get; set; }
+        public string slug { get; set; }
+
     }
 }
\ No newline at end of file

From 7dedca2d832ae8b72cc25a06c0dc437fb61383ea Mon Sep 17 00:00:00 2001
From: Anh-Duc Le <Anh-Duc_Le@outlook.com>
Date: Thu, 26 Apr 2018 11:37:52 +0200
Subject: [PATCH 19/21] endpoint using interface

---
 .gitignore                                    |  1 +
 SharpBucket/SharpBucket.nuspec                |  2 +-
 SharpBucket/V2/EndPoints/EndPoint.cs          | 29 ++++++++-
 .../V2/EndPoints/RepositoriesEndPoint.cs      | 54 ++++++++---------
 SharpBucket/V2/EndPoints/TeamsEndPoint.cs     |  2 +-
 SharpBucket/V2/EndPoints/UserEndpoint.cs      |  2 +-
 SharpBucket/V2/EndPoints/UsersEndpoint.cs     |  2 +-
 SharpBucket/V2/ISharpBucketV2.cs              | 60 +------------------
 8 files changed, 59 insertions(+), 93 deletions(-)

diff --git a/.gitignore b/.gitignore
index 9404e7fa..7ed7f1b3 100644
--- a/.gitignore
+++ b/.gitignore
@@ -23,3 +23,4 @@ _ReSharper*/
 /packages/
 !packages/repositories.config
 /.vs/SharpBucket/v15/Server/sqlite3
+*.nupkg
diff --git a/SharpBucket/SharpBucket.nuspec b/SharpBucket/SharpBucket.nuspec
index f4560270..133a5f99 100644
--- a/SharpBucket/SharpBucket.nuspec
+++ b/SharpBucket/SharpBucket.nuspec
@@ -2,7 +2,7 @@
 <package >
   <metadata>
     <id>$id$</id>
-    <version>$version$</version>
+    <version>0.7.2.10</version>
     <title>SharpBucket</title>
     <authors>Mitja Bezenšek</authors>
     <owners>Mitja Bezenšek</owners>
diff --git a/SharpBucket/V2/EndPoints/EndPoint.cs b/SharpBucket/V2/EndPoints/EndPoint.cs
index 7fc772ea..735ae353 100644
--- a/SharpBucket/V2/EndPoints/EndPoint.cs
+++ b/SharpBucket/V2/EndPoints/EndPoint.cs
@@ -3,6 +3,9 @@
 using System.Collections.Generic;
 using System.Diagnostics;
 using System.Dynamic;
+using System.Net;
+using RestSharp;
+using SharpBucket.Utility;
 
 namespace SharpBucket.V2.EndPoints
 {
@@ -11,7 +14,7 @@ public class EndPoint
         // vanilla page length in many cases is 10, requiring lots of requests for larger collections
         private const int DEFAULT_PAGE_LEN = 50;
 
-        protected readonly SharpBucketV2 _sharpBucketV2;
+        protected readonly ISharpBucketV2 _sharpBucketV2;
         protected readonly string _baseUrl;
 
         public EndPoint(SharpBucketV2 sharpBucketV2, string resourcePath)
@@ -22,7 +25,7 @@ public EndPoint(SharpBucketV2 sharpBucketV2, string resourcePath)
 
         public EndPoint(ISharpBucketV2 sharpBucketV2, string resourcePath)
         {
-            _sharpBucketV2 = (SharpBucketV2)sharpBucketV2;
+            _sharpBucketV2 = sharpBucketV2;
             _baseUrl = resourcePath;
         }
 
@@ -50,7 +53,7 @@ private IEnumerable<List<TValue>> IteratePages<TValue>(string overrideUrl, int p
             int page = 1;
             do
             {
-                response = _sharpBucketV2.Get(new IteratorBasedPage<TValue>(), overrideUrl.Replace(SharpBucketV2.BITBUCKET_URL, ""), requestParameters);
+                response = Get(new IteratorBasedPage<TValue>(), overrideUrl.Replace(SharpBucketV2.BITBUCKET_URL, ""), requestParameters);
                 if (response == null)
                 {
                     break;
@@ -100,5 +103,25 @@ protected List<TValue> GetPaginatedValues<TValue>(string overrideUrl, int max =
 
             return values;
         }
+
+        protected T Get<T>(T body, string overrideUrl, object requestParameters = null)
+        {
+            return ((SharpBucketV2)_sharpBucketV2).Get(body, overrideUrl, requestParameters);
+        }
+
+        protected T Post<T>(T body, string overrideUrl)
+        {
+            return ((SharpBucketV2) _sharpBucketV2).Post(body, overrideUrl);
+        }
+
+        protected T Put<T>(T body, string overrideUrl)
+        {
+            return ((SharpBucketV2)_sharpBucketV2).Put(body, overrideUrl);
+        }
+
+        protected T Delete<T>(T body, string overrideUrl)
+        {
+            return ((SharpBucketV2)_sharpBucketV2).Delete(body, overrideUrl);
+        }
     }
 }
\ No newline at end of file
diff --git a/SharpBucket/V2/EndPoints/RepositoriesEndPoint.cs b/SharpBucket/V2/EndPoints/RepositoriesEndPoint.cs
index c901dff9..e132d0f8 100644
--- a/SharpBucket/V2/EndPoints/RepositoriesEndPoint.cs
+++ b/SharpBucket/V2/EndPoints/RepositoriesEndPoint.cs
@@ -73,25 +73,25 @@ public RepositoryResource RepositoryResource(string accountName, string reposito
         internal Repository GetRepository(string accountName, string repository)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, null);
-            return _sharpBucketV2.Get(new Repository(), overrideUrl);
+            return Get(new Repository(), overrideUrl);
         }
 
         internal Repository PutRepository(Repository repo, string accountName, string repository)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, null);
-            return _sharpBucketV2.Put(repo, overrideUrl);
+            return Put(repo, overrideUrl);
         }
 
         internal Repository PostRepository(Repository repo, string accountName)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repo.name, null);
-            return _sharpBucketV2.Post(repo, overrideUrl);
+            return Post(repo, overrideUrl);
         }
 
         internal Repository DeleteRepository(string accountName, string repository)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, null);
-            return _sharpBucketV2.Delete(new Repository(), overrideUrl);
+            return Delete(new Repository(), overrideUrl);
         }
 
         private string GetRepositoryUrl(string accountName, string repository, string append)
@@ -130,13 +130,13 @@ internal List<PullRequest> ListPullRequests(string accountName, string repositor
         internal PullRequest PostPullRequest(string accountName, string repository, PullRequest pullRequest)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "pullrequests/");
-            return _sharpBucketV2.Post(pullRequest, overrideUrl);
+            return Post(pullRequest, overrideUrl);
         }
 
         internal PullRequest PutPullRequest(string accountName, string repository, PullRequest pullRequest)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "pullrequests/");
-            return _sharpBucketV2.Put(pullRequest, overrideUrl);
+            return Put(pullRequest, overrideUrl);
         }
 
         internal List<Activity> GetPullRequestLog(string accountName, string repository, int max = 0)
@@ -152,7 +152,7 @@ internal List<Activity> GetPullRequestLog(string accountName, string repository,
         internal PullRequest GetPullRequest(string accountName, string repository, int pullRequestId)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "pullrequests/" + pullRequestId + "/");
-            return _sharpBucketV2.Get(new PullRequest(), overrideUrl);
+            return Get(new PullRequest(), overrideUrl);
         }
 
         internal List<Commit> ListPullRequestCommits(string accountName, string repository, int pullRequestId, int max = 0)
@@ -164,19 +164,19 @@ internal List<Commit> ListPullRequestCommits(string accountName, string reposito
         internal PullRequestInfo ApprovePullRequest(string accountName, string repository, int pullRequestId)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "pullrequests/" + pullRequestId + "/approve/");
-            return _sharpBucketV2.Post(new PullRequestInfo(), overrideUrl);
+            return Post(new PullRequestInfo(), overrideUrl);
         }
 
         internal object RemovePullRequestApproval(string accountName, string repository, int pullRequestId)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "pullrequests/" + pullRequestId + "/approve/");
-            return _sharpBucketV2.Delete(new PullRequestInfo(), overrideUrl);
+            return Delete(new PullRequestInfo(), overrideUrl);
         }
 
         internal object GetDiffForPullRequest(string accountName, string repository, int pullRequestId)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "pullrequests/" + pullRequestId + "/diff/");
-            return _sharpBucketV2.Get(new Object(), overrideUrl);
+            return Get(new Object(), overrideUrl);
         }
 
         internal List<Activity> GetPullRequestActivity(string accountName, string repository, int pullRequestId, int max = 0)
@@ -188,13 +188,13 @@ internal List<Activity> GetPullRequestActivity(string accountName, string reposi
         internal Merge AcceptAndMergePullRequest(string accountName, string repository, int pullRequestId)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "pullrequests/" + pullRequestId + "/merge/");
-            return _sharpBucketV2.Post(new Merge(), overrideUrl);
+            return Post(new Merge(), overrideUrl);
         }
 
         internal Merge DeclinePullRequest(string accountName, string repository, int pullRequestId)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "pullrequests/" + pullRequestId + "/decline/");
-            return _sharpBucketV2.Get(new Merge(), overrideUrl);
+            return Get(new Merge(), overrideUrl);
         }
 
         internal List<Comment> ListPullRequestComments(string accountName, string repository, int pullRequestId, int max = 0)
@@ -206,7 +206,7 @@ internal List<Comment> ListPullRequestComments(string accountName, string reposi
         internal Comment GetPullRequestComment(string accountName, string repository, int pullRequestId, int commentId)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "pullrequests/" + pullRequestId + "/comments/" + commentId + "/");
-            return _sharpBucketV2.Get(new Comment(), overrideUrl);
+            return Get(new Comment(), overrideUrl);
         }
 
         #endregion
@@ -222,25 +222,25 @@ internal List<BranchRestriction> ListBranchRestrictions(string accountName, stri
         internal BranchRestriction PostBranchRestriction(string accountName, string repository, BranchRestriction restriction)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "branch-restrictions/");
-            return _sharpBucketV2.Post(restriction, overrideUrl);
+            return Post(restriction, overrideUrl);
         }
 
         internal BranchRestriction GetBranchRestriction(string accountName, string repository, int restrictionId)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "branch-restrictions/" + restrictionId);
-            return _sharpBucketV2.Get(new BranchRestriction(), overrideUrl);
+            return Get(new BranchRestriction(), overrideUrl);
         }
 
         internal BranchRestriction PutBranchRestriction(string accountName, string repository, BranchRestriction restriction)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "branch-restrictions/" + restriction.id);
-            return _sharpBucketV2.Put(restriction, overrideUrl);
+            return Put(restriction, overrideUrl);
         }
 
         internal BranchRestriction DeleteBranchRestriction(string accountName, string repository, int restrictionId)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "branch-restrictions/" + restrictionId);
-            return _sharpBucketV2.Delete(new BranchRestriction(), overrideUrl);
+            return Delete(new BranchRestriction(), overrideUrl);
         }
 
         #endregion
@@ -250,13 +250,13 @@ internal BranchRestriction DeleteBranchRestriction(string accountName, string re
         internal object GetDiff(string accountName, string repository, object options)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "diff/" + options);
-            return _sharpBucketV2.Get(new object(), overrideUrl);
+            return Get(new object(), overrideUrl);
         }
 
         internal object GetPatch(string accountName, string repository, object options)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "patch/" + options);
-            return _sharpBucketV2.Get(new object(), overrideUrl);
+            return Get(new object(), overrideUrl);
         }
 
         #endregion
@@ -276,7 +276,7 @@ internal List<Commit> ListCommits(string accountName, string repository, string
         internal Commit GetCommit(string accountName, string repository, string revision)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "commit/" + revision);
-            return _sharpBucketV2.Get(new Commit(), overrideUrl);
+            return Get(new Commit(), overrideUrl);
         }
 
         internal List<Comment> ListCommitComments(string accountName, string repository, string revision, int max = 0)
@@ -288,37 +288,37 @@ internal List<Comment> ListCommitComments(string accountName, string repository,
         internal object GetCommitComment(string accountName, string repository, string revision, int commentId)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "commits/" + revision + "/comments/" + revision + "/" + commentId + "/");
-            return _sharpBucketV2.Get(new object(), overrideUrl);
+            return Get(new object(), overrideUrl);
         }
 
         internal object ApproveCommit(string accountName, string repository, string revision)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "commits/" + revision + "/approve/");
-            return _sharpBucketV2.Post(new object(), overrideUrl);
+            return Post(new object(), overrideUrl);
         }
 
         internal object DeleteCommitApproval(string accountName, string repository, string revision)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "commits/" + revision + "/approve/");
-            return _sharpBucketV2.Delete(new object(), overrideUrl);
+            return Delete(new object(), overrideUrl);
         }
 
         internal object AddNewBuildStatus(string accountName, string repository, string revision, BuildInfo buildInfo)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "commit/" + revision + "/statuses/build/");
-            return _sharpBucketV2.Post(buildInfo, overrideUrl);
+            return Post(buildInfo, overrideUrl);
         }
 
         internal BuildInfo GetBuildStatusInfo(string accountName, string repository, string revision, string key)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "commit/" + revision + "/statuses/build/" + key);
-            return _sharpBucketV2.Get(new BuildInfo(), overrideUrl);
+            return Get(new BuildInfo(), overrideUrl);
         }
 
         internal object ChangeBuildStatusInfo(string accountName, string repository, string revision, string key, BuildInfo buildInfo)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "commit/" + revision + "/statuses/build/" + key);
-            return _sharpBucketV2.Put(buildInfo, overrideUrl);
+            return Put(buildInfo, overrideUrl);
         }
 
         #endregion
@@ -328,7 +328,7 @@ internal object ChangeBuildStatusInfo(string accountName, string repository, str
         internal object PutDefaultReviewer(string accountName, string repository, string targetUsername)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "default-reviewers/" + targetUsername);
-            return _sharpBucketV2.Put(new object(), overrideUrl);
+            return Put(new object(), overrideUrl);
         }
 
         #endregion
diff --git a/SharpBucket/V2/EndPoints/TeamsEndPoint.cs b/SharpBucket/V2/EndPoints/TeamsEndPoint.cs
index 93ab4dd8..945016e9 100644
--- a/SharpBucket/V2/EndPoints/TeamsEndPoint.cs
+++ b/SharpBucket/V2/EndPoints/TeamsEndPoint.cs
@@ -36,7 +36,7 @@ public List<Team> GetUserTeams(int max = 0)
         /// <returns></returns>
         public Team GetProfile()
         {
-            return _sharpBucketV2.Get(new Team(), _baseUrl);
+            return Get(new Team(), _baseUrl);
         }
 
         /// <summary>
diff --git a/SharpBucket/V2/EndPoints/UserEndpoint.cs b/SharpBucket/V2/EndPoints/UserEndpoint.cs
index cbd66fff..43d39128 100644
--- a/SharpBucket/V2/EndPoints/UserEndpoint.cs
+++ b/SharpBucket/V2/EndPoints/UserEndpoint.cs
@@ -15,7 +15,7 @@ public UserEndpoint(ISharpBucketV2 sharpBucketV2)
 
         public User GetUser()
         {
-            return _sharpBucketV2.Get<User>(null, _baseUrl);
+            return Get<User>(null, _baseUrl);
         }
     }
 }
\ No newline at end of file
diff --git a/SharpBucket/V2/EndPoints/UsersEndpoint.cs b/SharpBucket/V2/EndPoints/UsersEndpoint.cs
index 58d79d6d..e96e107a 100644
--- a/SharpBucket/V2/EndPoints/UsersEndpoint.cs
+++ b/SharpBucket/V2/EndPoints/UsersEndpoint.cs
@@ -31,7 +31,7 @@ public UsersEndpoint(string accountName, ISharpBucketV2 sharpBucketV2) :
         /// <returns></returns>
         public User GetProfile()
         {
-            return _sharpBucketV2.Get(new User(), _baseUrl);
+            return Get(new User(), _baseUrl);
         }
 
         /// <summary>
diff --git a/SharpBucket/V2/ISharpBucketV2.cs b/SharpBucket/V2/ISharpBucketV2.cs
index ee207bc4..ad1297e9 100644
--- a/SharpBucket/V2/ISharpBucketV2.cs
+++ b/SharpBucket/V2/ISharpBucketV2.cs
@@ -3,7 +3,7 @@
 
 namespace SharpBucket.V2
 {
-    public interface ISharpBucketV2
+    public interface ISharpBucketV2 : ISharpBucket
     {
         /// <summary>
         /// Get the Teams End Point for a specific team.
@@ -30,63 +30,5 @@ public interface ISharpBucketV2
         /// </summary>
         /// <returns></returns>
         UserEndpoint UserEndPoint();
-
-        /// <summary>   
-        /// Use basic authentication with the BitBucket API. OAuth authentication is preferred over
-        /// basic authentication, due to security reasons.
-        /// </summary>
-        /// <param name="username">Your BitBucket user name.</param>
-        /// <param name="password">Your BitBucket password.</param>
-        void BasicAuthentication(string username, string password);
-
-        /// <summary>
-        /// Use 2 legged OAuth 1.0a authentication. This is similar to basic authentication, since
-        /// it requires the same number of steps. It is still safer to use than basic authentication, 
-        /// since you can revoke the API keys.
-        /// More info:
-        /// https://confluence.atlassian.com/display/BITBUCKET/OAuth+on+Bitbucket
-        /// </summary>
-        /// <param name="consumerKey">Your consumer API key obtained from the BitBucket web page.</param>
-        /// <param name="consumerSecretKey">Your consumer secret API key also obtained from the BitBucket web page.</param>
-        void OAuth2LeggedAuthentication(string consumerKey, string consumerSecretKey);
-
-        /// <summary>
-        /// Use 3 legged OAuth 1.0a authentication. This is the most secure one, but for simple uses it might
-        /// be a bit too complex.
-        /// More info:
-        /// https://confluence.atlassian.com/display/BITBUCKET/OAuth+on+Bitbucket
-        /// </summary>
-        /// <param name="consumerKey">Your consumer API key obtained from the BitBucket web page.</param>
-        /// <param name="consumerSecretKey">Your consumer secret API key also obtained from the BitBucket web page.</param>
-        /// <param name="callback">Callback URL to which BitBucket will send the pin.</param>
-        /// <returns></returns>
-        OAuthentication3Legged OAuth3LeggedAuthentication(
-           string consumerKey,
-           string consumerSecretKey,
-           string callback = "oob");
-
-        /// <summary>
-        /// Use 3 legged OAuth 1.0a authentication. Use this method if you have already obtained the OAuthToken
-        /// and OAuthSecretToken. This method can be used so you do not have to go trough the whole 3 legged
-        /// process every time. You can save the tokens you receive the first time and reuse them in another session.
-        /// </summary>
-        /// <param name="consumerKey">Your consumer API key obtained from the BitBucket web page.</param>
-        /// <param name="consumerSecretKey">Your consumer secret API key also obtained from the BitBucket web page.</param>
-        /// <param name="oauthToken">Your OAuth token that was obtained on a previous session.</param>
-        /// <param name="oauthTokenSecret">Your OAuth secret token thata was obtained on a previous session.</param>
-        /// <returns></returns>
-        OAuthentication3Legged OAuth3LeggedAuthentication(
-           string consumerKey,
-           string consumerSecretKey,
-           string oauthToken,
-           string oauthTokenSecret);
-
-        /// <summary>
-        /// Use Oauth2 authentication. This is the neweset version and is prefered.
-        /// </summary>
-        /// <param name="consumerKey"></param>
-        /// <param name="consumerSecretKey"></param>
-        /// <returns></returns>
-        OAuthentication2 OAuthentication2(string consumerKey, string consumerSecretKey);
     }
 }
\ No newline at end of file

From b9079a09794085ea8d9dbb6d9a078b4896651c76 Mon Sep 17 00:00:00 2001
From: Anh-Duc Le <Anh-Duc_Le@outlook.com>
Date: Thu, 26 Apr 2018 11:57:48 +0200
Subject: [PATCH 20/21] CRUD methods public

---
 SharpBucket/ISharpBucket.cs                   |  5 ++
 SharpBucket/SharpBucket.cs                    |  8 +--
 SharpBucket/V2/EndPoints/EndPoint.cs          | 22 +-------
 .../V2/EndPoints/RepositoriesEndPoint.cs      | 54 +++++++++----------
 SharpBucket/V2/EndPoints/TeamsEndPoint.cs     |  2 +-
 SharpBucket/V2/EndPoints/UserEndpoint.cs      |  2 +-
 SharpBucket/V2/EndPoints/UsersEndpoint.cs     |  2 +-
 7 files changed, 40 insertions(+), 55 deletions(-)

diff --git a/SharpBucket/ISharpBucket.cs b/SharpBucket/ISharpBucket.cs
index d182ae3b..455b0e85 100644
--- a/SharpBucket/ISharpBucket.cs
+++ b/SharpBucket/ISharpBucket.cs
@@ -61,5 +61,10 @@ OAuthentication3Legged OAuth3LeggedAuthentication(
         /// <param name="consumerSecretKey"></param>
         /// <returns></returns>
         OAuthentication2 OAuthentication2(string consumerKey, string consumerSecretKey);
+
+        T Get<T>(T body, string overrideUrl, object requestParameters = null);
+        T Post<T>(T body, string overrideUrl);
+        T Put<T>(T body, string overrideUrl);
+        T Delete<T>(T body, string overrideUrl);
     }
 }
\ No newline at end of file
diff --git a/SharpBucket/SharpBucket.cs b/SharpBucket/SharpBucket.cs
index 6b0c5795..f64531a8 100644
--- a/SharpBucket/SharpBucket.cs
+++ b/SharpBucket/SharpBucket.cs
@@ -83,24 +83,24 @@ private T Send<T>(T body, Method method, string overrideUrl = null, IDictionary<
             return response;
         }
 
-        internal T Get<T>(T body, string overrideUrl, object requestParameters = null)
+        public T Get<T>(T body, string overrideUrl, object requestParameters = null)
         {
             //Convert to dictionary to avoid refactoring the Send method.
             var parameterDictionary = requestParameters.ToDictionary();
             return Send(body, Method.GET, overrideUrl, parameterDictionary);
         }
 
-        internal T Post<T>(T body, string overrideUrl)
+        public T Post<T>(T body, string overrideUrl)
         {
             return Send(body, Method.POST, overrideUrl);
         }
 
-        internal T Put<T>(T body, string overrideUrl)
+        public T Put<T>(T body, string overrideUrl)
         {
             return Send(body, Method.PUT, overrideUrl);
         }
 
-        internal T Delete<T>(T body, string overrideUrl)
+        public T Delete<T>(T body, string overrideUrl)
         {
             return Send(body, Method.DELETE, overrideUrl);
         }
diff --git a/SharpBucket/V2/EndPoints/EndPoint.cs b/SharpBucket/V2/EndPoints/EndPoint.cs
index 735ae353..4b286e9c 100644
--- a/SharpBucket/V2/EndPoints/EndPoint.cs
+++ b/SharpBucket/V2/EndPoints/EndPoint.cs
@@ -53,7 +53,7 @@ private IEnumerable<List<TValue>> IteratePages<TValue>(string overrideUrl, int p
             int page = 1;
             do
             {
-                response = Get(new IteratorBasedPage<TValue>(), overrideUrl.Replace(SharpBucketV2.BITBUCKET_URL, ""), requestParameters);
+                response = _sharpBucketV2.Get(new IteratorBasedPage<TValue>(), overrideUrl.Replace(SharpBucketV2.BITBUCKET_URL, ""), requestParameters);
                 if (response == null)
                 {
                     break;
@@ -103,25 +103,5 @@ protected List<TValue> GetPaginatedValues<TValue>(string overrideUrl, int max =
 
             return values;
         }
-
-        protected T Get<T>(T body, string overrideUrl, object requestParameters = null)
-        {
-            return ((SharpBucketV2)_sharpBucketV2).Get(body, overrideUrl, requestParameters);
-        }
-
-        protected T Post<T>(T body, string overrideUrl)
-        {
-            return ((SharpBucketV2) _sharpBucketV2).Post(body, overrideUrl);
-        }
-
-        protected T Put<T>(T body, string overrideUrl)
-        {
-            return ((SharpBucketV2)_sharpBucketV2).Put(body, overrideUrl);
-        }
-
-        protected T Delete<T>(T body, string overrideUrl)
-        {
-            return ((SharpBucketV2)_sharpBucketV2).Delete(body, overrideUrl);
         }
-    }
 }
\ No newline at end of file
diff --git a/SharpBucket/V2/EndPoints/RepositoriesEndPoint.cs b/SharpBucket/V2/EndPoints/RepositoriesEndPoint.cs
index e132d0f8..c901dff9 100644
--- a/SharpBucket/V2/EndPoints/RepositoriesEndPoint.cs
+++ b/SharpBucket/V2/EndPoints/RepositoriesEndPoint.cs
@@ -73,25 +73,25 @@ public RepositoryResource RepositoryResource(string accountName, string reposito
         internal Repository GetRepository(string accountName, string repository)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, null);
-            return Get(new Repository(), overrideUrl);
+            return _sharpBucketV2.Get(new Repository(), overrideUrl);
         }
 
         internal Repository PutRepository(Repository repo, string accountName, string repository)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, null);
-            return Put(repo, overrideUrl);
+            return _sharpBucketV2.Put(repo, overrideUrl);
         }
 
         internal Repository PostRepository(Repository repo, string accountName)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repo.name, null);
-            return Post(repo, overrideUrl);
+            return _sharpBucketV2.Post(repo, overrideUrl);
         }
 
         internal Repository DeleteRepository(string accountName, string repository)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, null);
-            return Delete(new Repository(), overrideUrl);
+            return _sharpBucketV2.Delete(new Repository(), overrideUrl);
         }
 
         private string GetRepositoryUrl(string accountName, string repository, string append)
@@ -130,13 +130,13 @@ internal List<PullRequest> ListPullRequests(string accountName, string repositor
         internal PullRequest PostPullRequest(string accountName, string repository, PullRequest pullRequest)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "pullrequests/");
-            return Post(pullRequest, overrideUrl);
+            return _sharpBucketV2.Post(pullRequest, overrideUrl);
         }
 
         internal PullRequest PutPullRequest(string accountName, string repository, PullRequest pullRequest)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "pullrequests/");
-            return Put(pullRequest, overrideUrl);
+            return _sharpBucketV2.Put(pullRequest, overrideUrl);
         }
 
         internal List<Activity> GetPullRequestLog(string accountName, string repository, int max = 0)
@@ -152,7 +152,7 @@ internal List<Activity> GetPullRequestLog(string accountName, string repository,
         internal PullRequest GetPullRequest(string accountName, string repository, int pullRequestId)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "pullrequests/" + pullRequestId + "/");
-            return Get(new PullRequest(), overrideUrl);
+            return _sharpBucketV2.Get(new PullRequest(), overrideUrl);
         }
 
         internal List<Commit> ListPullRequestCommits(string accountName, string repository, int pullRequestId, int max = 0)
@@ -164,19 +164,19 @@ internal List<Commit> ListPullRequestCommits(string accountName, string reposito
         internal PullRequestInfo ApprovePullRequest(string accountName, string repository, int pullRequestId)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "pullrequests/" + pullRequestId + "/approve/");
-            return Post(new PullRequestInfo(), overrideUrl);
+            return _sharpBucketV2.Post(new PullRequestInfo(), overrideUrl);
         }
 
         internal object RemovePullRequestApproval(string accountName, string repository, int pullRequestId)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "pullrequests/" + pullRequestId + "/approve/");
-            return Delete(new PullRequestInfo(), overrideUrl);
+            return _sharpBucketV2.Delete(new PullRequestInfo(), overrideUrl);
         }
 
         internal object GetDiffForPullRequest(string accountName, string repository, int pullRequestId)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "pullrequests/" + pullRequestId + "/diff/");
-            return Get(new Object(), overrideUrl);
+            return _sharpBucketV2.Get(new Object(), overrideUrl);
         }
 
         internal List<Activity> GetPullRequestActivity(string accountName, string repository, int pullRequestId, int max = 0)
@@ -188,13 +188,13 @@ internal List<Activity> GetPullRequestActivity(string accountName, string reposi
         internal Merge AcceptAndMergePullRequest(string accountName, string repository, int pullRequestId)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "pullrequests/" + pullRequestId + "/merge/");
-            return Post(new Merge(), overrideUrl);
+            return _sharpBucketV2.Post(new Merge(), overrideUrl);
         }
 
         internal Merge DeclinePullRequest(string accountName, string repository, int pullRequestId)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "pullrequests/" + pullRequestId + "/decline/");
-            return Get(new Merge(), overrideUrl);
+            return _sharpBucketV2.Get(new Merge(), overrideUrl);
         }
 
         internal List<Comment> ListPullRequestComments(string accountName, string repository, int pullRequestId, int max = 0)
@@ -206,7 +206,7 @@ internal List<Comment> ListPullRequestComments(string accountName, string reposi
         internal Comment GetPullRequestComment(string accountName, string repository, int pullRequestId, int commentId)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "pullrequests/" + pullRequestId + "/comments/" + commentId + "/");
-            return Get(new Comment(), overrideUrl);
+            return _sharpBucketV2.Get(new Comment(), overrideUrl);
         }
 
         #endregion
@@ -222,25 +222,25 @@ internal List<BranchRestriction> ListBranchRestrictions(string accountName, stri
         internal BranchRestriction PostBranchRestriction(string accountName, string repository, BranchRestriction restriction)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "branch-restrictions/");
-            return Post(restriction, overrideUrl);
+            return _sharpBucketV2.Post(restriction, overrideUrl);
         }
 
         internal BranchRestriction GetBranchRestriction(string accountName, string repository, int restrictionId)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "branch-restrictions/" + restrictionId);
-            return Get(new BranchRestriction(), overrideUrl);
+            return _sharpBucketV2.Get(new BranchRestriction(), overrideUrl);
         }
 
         internal BranchRestriction PutBranchRestriction(string accountName, string repository, BranchRestriction restriction)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "branch-restrictions/" + restriction.id);
-            return Put(restriction, overrideUrl);
+            return _sharpBucketV2.Put(restriction, overrideUrl);
         }
 
         internal BranchRestriction DeleteBranchRestriction(string accountName, string repository, int restrictionId)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "branch-restrictions/" + restrictionId);
-            return Delete(new BranchRestriction(), overrideUrl);
+            return _sharpBucketV2.Delete(new BranchRestriction(), overrideUrl);
         }
 
         #endregion
@@ -250,13 +250,13 @@ internal BranchRestriction DeleteBranchRestriction(string accountName, string re
         internal object GetDiff(string accountName, string repository, object options)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "diff/" + options);
-            return Get(new object(), overrideUrl);
+            return _sharpBucketV2.Get(new object(), overrideUrl);
         }
 
         internal object GetPatch(string accountName, string repository, object options)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "patch/" + options);
-            return Get(new object(), overrideUrl);
+            return _sharpBucketV2.Get(new object(), overrideUrl);
         }
 
         #endregion
@@ -276,7 +276,7 @@ internal List<Commit> ListCommits(string accountName, string repository, string
         internal Commit GetCommit(string accountName, string repository, string revision)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "commit/" + revision);
-            return Get(new Commit(), overrideUrl);
+            return _sharpBucketV2.Get(new Commit(), overrideUrl);
         }
 
         internal List<Comment> ListCommitComments(string accountName, string repository, string revision, int max = 0)
@@ -288,37 +288,37 @@ internal List<Comment> ListCommitComments(string accountName, string repository,
         internal object GetCommitComment(string accountName, string repository, string revision, int commentId)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "commits/" + revision + "/comments/" + revision + "/" + commentId + "/");
-            return Get(new object(), overrideUrl);
+            return _sharpBucketV2.Get(new object(), overrideUrl);
         }
 
         internal object ApproveCommit(string accountName, string repository, string revision)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "commits/" + revision + "/approve/");
-            return Post(new object(), overrideUrl);
+            return _sharpBucketV2.Post(new object(), overrideUrl);
         }
 
         internal object DeleteCommitApproval(string accountName, string repository, string revision)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "commits/" + revision + "/approve/");
-            return Delete(new object(), overrideUrl);
+            return _sharpBucketV2.Delete(new object(), overrideUrl);
         }
 
         internal object AddNewBuildStatus(string accountName, string repository, string revision, BuildInfo buildInfo)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "commit/" + revision + "/statuses/build/");
-            return Post(buildInfo, overrideUrl);
+            return _sharpBucketV2.Post(buildInfo, overrideUrl);
         }
 
         internal BuildInfo GetBuildStatusInfo(string accountName, string repository, string revision, string key)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "commit/" + revision + "/statuses/build/" + key);
-            return Get(new BuildInfo(), overrideUrl);
+            return _sharpBucketV2.Get(new BuildInfo(), overrideUrl);
         }
 
         internal object ChangeBuildStatusInfo(string accountName, string repository, string revision, string key, BuildInfo buildInfo)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "commit/" + revision + "/statuses/build/" + key);
-            return Put(buildInfo, overrideUrl);
+            return _sharpBucketV2.Put(buildInfo, overrideUrl);
         }
 
         #endregion
@@ -328,7 +328,7 @@ internal object ChangeBuildStatusInfo(string accountName, string repository, str
         internal object PutDefaultReviewer(string accountName, string repository, string targetUsername)
         {
             var overrideUrl = GetRepositoryUrl(accountName, repository, "default-reviewers/" + targetUsername);
-            return Put(new object(), overrideUrl);
+            return _sharpBucketV2.Put(new object(), overrideUrl);
         }
 
         #endregion
diff --git a/SharpBucket/V2/EndPoints/TeamsEndPoint.cs b/SharpBucket/V2/EndPoints/TeamsEndPoint.cs
index 945016e9..93ab4dd8 100644
--- a/SharpBucket/V2/EndPoints/TeamsEndPoint.cs
+++ b/SharpBucket/V2/EndPoints/TeamsEndPoint.cs
@@ -36,7 +36,7 @@ public List<Team> GetUserTeams(int max = 0)
         /// <returns></returns>
         public Team GetProfile()
         {
-            return Get(new Team(), _baseUrl);
+            return _sharpBucketV2.Get(new Team(), _baseUrl);
         }
 
         /// <summary>
diff --git a/SharpBucket/V2/EndPoints/UserEndpoint.cs b/SharpBucket/V2/EndPoints/UserEndpoint.cs
index 43d39128..cbd66fff 100644
--- a/SharpBucket/V2/EndPoints/UserEndpoint.cs
+++ b/SharpBucket/V2/EndPoints/UserEndpoint.cs
@@ -15,7 +15,7 @@ public UserEndpoint(ISharpBucketV2 sharpBucketV2)
 
         public User GetUser()
         {
-            return Get<User>(null, _baseUrl);
+            return _sharpBucketV2.Get<User>(null, _baseUrl);
         }
     }
 }
\ No newline at end of file
diff --git a/SharpBucket/V2/EndPoints/UsersEndpoint.cs b/SharpBucket/V2/EndPoints/UsersEndpoint.cs
index e96e107a..58d79d6d 100644
--- a/SharpBucket/V2/EndPoints/UsersEndpoint.cs
+++ b/SharpBucket/V2/EndPoints/UsersEndpoint.cs
@@ -31,7 +31,7 @@ public UsersEndpoint(string accountName, ISharpBucketV2 sharpBucketV2) :
         /// <returns></returns>
         public User GetProfile()
         {
-            return Get(new User(), _baseUrl);
+            return _sharpBucketV2.Get(new User(), _baseUrl);
         }
 
         /// <summary>

From 957d8a105a07ec32f59a877630ea8528f6c89a88 Mon Sep 17 00:00:00 2001
From: Anh-Duc Le <Anh-Duc_Le@outlook.com>
Date: Thu, 26 Apr 2018 15:01:21 +0200
Subject: [PATCH 21/21] slug parser for repository endpoint and added missing
 links

---
 SharpBucket/Authentication/RequestExecutor.cs    |  3 ++-
 SharpBucket/ISharpBucket.cs                      |  2 +-
 SharpBucket/SharpBucket.nuspec                   |  2 +-
 SharpBucket/V2/EndPoints/RepositoriesEndPoint.cs | 14 ++++++++++----
 SharpBucket/V2/Pocos/Links.cs                    | 15 +++++++++++++--
 5 files changed, 27 insertions(+), 9 deletions(-)

diff --git a/SharpBucket/Authentication/RequestExecutor.cs b/SharpBucket/Authentication/RequestExecutor.cs
index 9cc81e90..4e29e833 100644
--- a/SharpBucket/Authentication/RequestExecutor.cs
+++ b/SharpBucket/Authentication/RequestExecutor.cs
@@ -148,7 +148,8 @@ public static T ExecuteRequestWithLogging<T>(ILogger logger, string url, Method
                 return result.Content as dynamic;
             }
 
-            logger.Debug($"{result.StatusCode}: {result.StatusDescription}");
+            logger.Debug($"{result.StatusCode}");
+            logger.Verbose($"{result.Content}");
             return result.Data;
         }
     }
diff --git a/SharpBucket/ISharpBucket.cs b/SharpBucket/ISharpBucket.cs
index 586be5de..50ac9549 100644
--- a/SharpBucket/ISharpBucket.cs
+++ b/SharpBucket/ISharpBucket.cs
@@ -68,7 +68,7 @@ OAuthentication3Legged OAuth3LeggedAuthentication(
         T Put<T>(T body, string overrideUrl);
         T Delete<T>(T body, string overrideUrl);
 
-        T Get<T>(ILogger logger, T body, string overrideUrl, object requestParameters = null);
+        T Get<T>(ILogger logger,T body, string overrideUrl, object requestParameters = null);
         T Post<T>(ILogger logger, T body, string overrideUrl);
         T Put<T>(ILogger logger, T body, string overrideUrl);
         T Delete<T>(ILogger logger, T body, string overrideUrl);
diff --git a/SharpBucket/SharpBucket.nuspec b/SharpBucket/SharpBucket.nuspec
index 791a0e11..92e76767 100644
--- a/SharpBucket/SharpBucket.nuspec
+++ b/SharpBucket/SharpBucket.nuspec
@@ -2,7 +2,7 @@
 <package >
   <metadata>
     <id>$id$</id>
-    <version>0.7.2.12</version>
+    <version>0.7.2.14</version>
     <title>SharpBucket</title>
     <authors>Mitja Bezenšek</authors>
     <owners>Mitja Bezenšek</owners>
diff --git a/SharpBucket/V2/EndPoints/RepositoriesEndPoint.cs b/SharpBucket/V2/EndPoints/RepositoriesEndPoint.cs
index 74a5e49a..e68f1618 100644
--- a/SharpBucket/V2/EndPoints/RepositoriesEndPoint.cs
+++ b/SharpBucket/V2/EndPoints/RepositoriesEndPoint.cs
@@ -1,6 +1,7 @@
 using System;
 using System.Collections.Generic;
 using System.Net;
+using System.Text.RegularExpressions;
 using Serilog;
 using SharpBucket.V2.Pocos;
 using Comment = SharpBucket.V2.Pocos.Comment;
@@ -172,7 +173,7 @@ internal Repository PutRepository(ILogger logger, Repository repo, string accoun
 
         internal Repository PostRepository(Repository repo, string accountName)
         {
-            var overrideUrl = GetRepositoryUrl(accountName, repo.name.ToLowerInvariant(), null);
+            var overrideUrl = GetRepositoryUrl(accountName, repo.name, null);
             return _sharpBucketV2.Post(repo, overrideUrl);
         }
 
@@ -185,7 +186,7 @@ internal Repository PostRepository(Repository repo, string accountName)
         /// <returns></returns>
         internal Repository PostRepository(ILogger logger, Repository repo, string accountName)
         {
-            var overrideUrl = GetRepositoryUrl(accountName, repo.name.ToLowerInvariant(), null);
+            var overrideUrl = GetRepositoryUrl(accountName, repo.name, null);
             return _sharpBucketV2.Post(logger, repo, overrideUrl);
         }
 
@@ -207,12 +208,17 @@ internal HttpStatusCode DeleteRepository(ILogger logger, string accountName, str
             var overrideUrl = GetRepositoryUrl(accountName, repository, null);
             return _sharpBucketV2.Delete(logger, new HttpStatusCode(), overrideUrl);
         }
-
+        
+        private string ParseSlug(string repositoryName)
+        {
+            var slugRegex = new Regex(@"[^a-zA-Z\.\-_0-9]+");
+            return slugRegex.Replace(repositoryName, "-").ToLowerInvariant();
+        }
 
         private string GetRepositoryUrl(string accountName, string repository, string append)
         {
             var format = _baseUrl + "{0}/{1}/{2}";
-            return string.Format(format, accountName, repository, append);
+            return string.Format(format, accountName, ParseSlug(repository), append);
         }
 
         internal List<Watcher> ListWatchers(string accountName, string repository, int max = 0)
diff --git a/SharpBucket/V2/Pocos/Links.cs b/SharpBucket/V2/Pocos/Links.cs
index 1a17abc7..e1c50343 100644
--- a/SharpBucket/V2/Pocos/Links.cs
+++ b/SharpBucket/V2/Pocos/Links.cs
@@ -4,12 +4,23 @@ namespace SharpBucket.V2.Pocos
 {
     public class Links
     {
+        public Link watchers { get; set; }
+        public Link branches { get; set; }
+        public Link tags { get; set; }
+        public Link commits { get; set; }
+        public List<Link> clone { get; set; }
         public Link self { get; set; }
+        public Link source { get; set; }
+        public Link html { get; set; }
+        public Link avatar { get; set; }
+        public Link hooks { get; set; }
+        public Link forks { get; set; }
+        public Link downloads { get; set; }
+        public Link isues { get; set; }
+        public Link pullrequests { get; set; }
         public Link repositories { get; set; }
         public Link link { get; set; }
         public Link followers { get; set; }
-        public Link avatar { get; set; }
         public Link following { get; set; }
-        public List<Link> clone { get; set; }
     }
 }
\ No newline at end of file