yavsccli target

Makefile

@@ -7,6 +7,7 @@ CONFIG=Debug
 
 git_status := $(shell git status -s --porcelain |wc -l)
 yavscd=binaries/$(CONFIG)/yavscd
+yavsccli=binaries/$(CONFIG)/yavsccli
 
 all: $(yavscd)
 
@@ -71,8 +72,12 @@ $(yavscd): src/Yavsc/bin/$(CONFIG)/dnx451/Yavsc.dll src/Yavsc.Server/bin/$(CONFI
 	mkbundle --static $(DNXLIBS) src/Yavsc/bin/$(CONFIG)/dnx451/Yavsc.dll src/Yavsc/bin/$(CONFIG)/dnx451/pt/Yavsc.resources.dll src/Yavsc/bin/$(CONFIG)/dnx451/en/Yavsc.resources.dll src/Yavsc.Server/bin/$(CONFIG)/dnx451/Yavsc.Server.dll src/Yavsc.Server/bin/$(CONFIG)/dnx451/en/Yavsc.Server.resources.dll src/Yavsc.Abstract/bin/$(CONFIG)/dnx451/Yavsc.Abstract.dll src/Yavsc.Abstract/bin/$(CONFIG)/dnx451/en/Yavsc.Abstract.resources.dll src/Yavsc.Abstract/bin/$(CONFIG)/dnx451/pt/Yavsc.Abstract.resources.dll src/OAuth.AspNet.AuthServer/bin/$(CONFIG)/dnx451/OAuth.AspNet.AuthServer.dll src/OAuth.AspNet.Token/bin/$(CONFIG)/dnx451/OAuth.AspNet.Token.dll  $(LIBS) -L $(DNX_USER_HOME)/runtimes/dnx-mono.1.0.0-rc1-update2/bin --machine-config $(MONO_PREFIX)/etc/mono/4.5/machine.config -o $(yavscd)
 	strip $(yavscd)
 
+yavsccli: src/cli/bin/$(CONFIG)/dnx451/cli.dll src/cli/bin/$(CONFIG)/dnx451/fr/cli.resources.dll src/Yavsc/bin/$(CONFIG)/dnx451/Yavsc.dll src/Yavsc/bin/$(CONFIG)/dnx451/pt/Yavsc.resources.dll src/Yavsc.Server/bin/$(CONFIG)/dnx451/Yavsc.Server.dll src/Yavsc.Abstract/bin/$(CONFIG)/dnx451/Yavsc.Abstract.dll src/OAuth.AspNet.AuthServer/bin/$(CONFIG)/dnx451/OAuth.AspNet.AuthServer.dll src/OAuth.AspNet.Token/bin/$(CONFIG)/dnx451/OAuth.AspNet.Token.dll
+	mkdir -p binaries/$(CONFIG)
+	mkbundle --static $(DNXLIBS) src/cli/bin/$(CONFIG)/dnx451/cli.dll src/cli/bin/$(CONFIG)/dnx451/fr/cli.resources.dll src/Yavsc/bin/$(CONFIG)/dnx451/Yavsc.dll src/Yavsc/bin/$(CONFIG)/dnx451/pt/Yavsc.resources.dll src/Yavsc/bin/$(CONFIG)/dnx451/en/Yavsc.resources.dll src/Yavsc.Server/bin/$(CONFIG)/dnx451/Yavsc.Server.dll src/Yavsc.Server/bin/$(CONFIG)/dnx451/en/Yavsc.Server.resources.dll src/Yavsc.Abstract/bin/$(CONFIG)/dnx451/Yavsc.Abstract.dll src/Yavsc.Abstract/bin/$(CONFIG)/dnx451/en/Yavsc.Abstract.resources.dll src/Yavsc.Abstract/bin/$(CONFIG)/dnx451/pt/Yavsc.Abstract.resources.dll src/OAuth.AspNet.AuthServer/bin/$(CONFIG)/dnx451/OAuth.AspNet.AuthServer.dll src/OAuth.AspNet.Token/bin/$(CONFIG)/dnx451/OAuth.AspNet.Token.dll  $(LIBS) -L $(DNX_USER_HOME)/runtimes/dnx-mono.1.0.0-rc1-update2/bin --machine-config $(MONO_PREFIX)/etc/mono/4.5/machine.config -o $(yavsccli)
+	strip $(yavsccli)
+
 version-increment-patch:
 	scripts/version.sh $$(cat version.txt) patch > version.txt
 
-
 .PHONY: packages

src/Yavsc/Helpers/TeXHelpers.cs

@@ -143,11 +143,13 @@ namespace Yavsc.Helpers
                 using (Process p = new Process())
                 {
                     p.StartInfo.WorkingDirectory = tempdir;
-                    p.StartInfo = new ProcessStartInfo();
-                    p.StartInfo.UseShellExecute = false;
-                    p.StartInfo.WorkingDirectory = tempdir;
-                    p.StartInfo.FileName = "/usr/bin/texi2pdf";
-                    p.StartInfo.Arguments = $"--batch --build-dir=. -o {fo.FullName} {fi.FullName}";
+                    p.StartInfo = new ProcessStartInfo
+                    {
+                        UseShellExecute = false,
+                        WorkingDirectory = tempdir,
+                        FileName = "/usr/bin/texi2pdf",
+                        Arguments = $"--batch --build-dir=. -o {fo.FullName} {fi.FullName}"
+                    };
                     p.Start();
                     p.WaitForExit();
                     if (p.ExitCode != 0)

src/Yavsc/Hubs/ChatHub.cs

@@ -37,10 +37,10 @@ namespace Yavsc
     using Yavsc.Services;
     public partial class ChatHub : Hub, IDisposable
     {
-        ApplicationDbContext _dbContext;
-        private IConnexionManager _cxManager;
-        private IStringLocalizer _localizer;
-        ILogger _logger;
+        private readonly ApplicationDbContext _dbContext;
+        private readonly IConnexionManager _cxManager;
+        private readonly IStringLocalizer _localizer;
+        private readonly ILogger _logger;
 
         public HubInputValidator InputValidator { get; }
 

src/Yavsc/Services/ChatHubConnexionManager.cs

@@ -20,9 +20,9 @@ namespace Yavsc.Services
     /// </summary>
     public class HubConnectionManager : IConnexionManager
     {
-        ILogger _logger;
+        private readonly ILogger _logger;
 
-        Action<string, string> _errorHandler;
+        private Action<string, string> _errorHandler;
 
         /// <summary>
         /// by cx id

src/Yavsc/Services/FileSystemAuthManager.cs

@@ -28,23 +28,21 @@ namespace Yavsc.Services
                 return false;
             }
         }
-        UserMatch Out = new OutOfCircle();
-        UserMatch In = new BelongsToCircle();
 
-        readonly ApplicationDbContext _dbContext;
-        readonly ILogger _logger;
+        private readonly UserMatch Out = new OutOfCircle();
+        private readonly UserMatch In = new BelongsToCircle();
 
-        readonly SiteSettings SiteSettings;
+        private readonly ApplicationDbContext _dbContext;
 
-        readonly string aclfileName;
+        private readonly SiteSettings SiteSettings;
+
+        private readonly string aclfileName;
 
         readonly RuleSetParser ruleSetParser;
 
-        public FileSystemAuthManager(ApplicationDbContext dbContext, ILoggerFactory loggerFactory,
-            IOptions<SiteSettings> sitesOptions)
+        public FileSystemAuthManager(ApplicationDbContext dbContext, IOptions<SiteSettings> sitesOptions)
         {
             _dbContext = dbContext;
-            _logger = loggerFactory.CreateLogger<FileSystemAuthManager>();
             SiteSettings = sitesOptions.Value;
             aclfileName = SiteSettings.AccessListFileName;
             ruleSetParser = new RuleSetParser(false);

src/Yavsc/Services/LiveProcessor.cs

@@ -23,17 +23,12 @@ namespace Yavsc.Services
 
     public class LiveProcessor : ILiveProcessor
     {
-        readonly IHubContext _hubContext;
         private readonly ILogger _logger;
-        readonly ApplicationDbContext _dbContext;
-
 
         public ConcurrentDictionary<string, LiveCastHandler> Casters { get; } = new ConcurrentDictionary<string, LiveCastHandler>();
 
-        public LiveProcessor(ApplicationDbContext dbContext, ILoggerFactory loggerFactory)
+        public LiveProcessor(ILoggerFactory loggerFactory)
         {
-            _dbContext = dbContext;
-            _hubContext = GlobalHost.ConnectionManager.GetHubContext<ChatHub>();
             _logger = loggerFactory.CreateLogger<LiveProcessor>();
         }
 

src/Yavsc/Startup/SendFileWrapper.cs

@@ -8,59 +8,57 @@ using Microsoft.Extensions.Logging;
 namespace Yavsc
 {
 
-        class YaSendFileWrapper : IHttpSendFileFeature
-        {
-            private readonly Stream _output;
+    class YaSendFileWrapper : IHttpSendFileFeature
+    {
+        private readonly Stream _output;
+
+        private readonly ILogger _logger;
 
-            private readonly ILogger _logger;
+        internal YaSendFileWrapper(Stream output, ILogger logger)
+        {
+            _output = output;
+            _logger = logger;
+        }
 
-            internal YaSendFileWrapper(Stream output, ILogger logger)
+        public async Task SendFileAsync(string fileName, long offset, long? length, CancellationToken cancel)
+        {
+            cancel.ThrowIfCancellationRequested();
+            if (string.IsNullOrWhiteSpace(fileName))
             {
-                _output = output;
-                _logger = logger;
+                throw new ArgumentNullException("fileName");
             }
-
-            public async Task SendFileAsync(string fileName, long offset, long? length, CancellationToken cancel)
+            if (!File.Exists(fileName))
+            {
+                throw new FileNotFoundException(string.Empty, fileName);
+            }
+            FileInfo fileInfo = new FileInfo(fileName);
+            if (offset < 0 || offset > fileInfo.Length)
             {
-                cancel.ThrowIfCancellationRequested();
-                if (string.IsNullOrWhiteSpace(fileName))
-                {
-                    throw new ArgumentNullException("fileName");
-                }
-                if (!File.Exists(fileName))
-                {
-                    throw new FileNotFoundException(string.Empty, fileName);
-                }
-                FileInfo fileInfo = new FileInfo(fileName);
-                if (offset < 0 || offset > fileInfo.Length)
-                {
-                    throw new ArgumentOutOfRangeException("offset", offset, string.Empty);
-                }
-                if (length.HasValue && (length.Value < 0 || length.Value > fileInfo.Length - offset))
-                {
-                    throw new ArgumentOutOfRangeException("length", length, string.Empty);
-                }
-                FileStream fileStream = new FileStream(fileName, FileMode.Open, FileAccess.Read, FileShare.ReadWrite, maxbufferlen, FileOptions.Asynchronous | FileOptions.SequentialScan);
-                try
-                {
-                    fileStream.Seek(offset, SeekOrigin.Begin);
-                    _logger.LogInformation(string.Format("Copying bytes range:{0},{1} filename:{2} ", offset, (!length.HasValue) ? null : (offset + length), fileName));
-                    // await CopyToAsync(fileStream, _output, length, cancel);
-                    await CopyToAsync(fileStream, _output);
-                }
-                finally
-                {
-                    fileStream.Dispose();
-                }
+                throw new ArgumentOutOfRangeException("offset", offset, string.Empty);
             }
+            if (length.HasValue && (length.Value < 0 || length.Value > fileInfo.Length - offset))
+            {
+                throw new ArgumentOutOfRangeException("length", length, string.Empty);
+            }
+            FileStream fileStream = new FileStream(fileName, FileMode.Open, FileAccess.Read, FileShare.ReadWrite, maxbufferlen, FileOptions.Asynchronous | FileOptions.SequentialScan);
+            try
+            {
+                fileStream.Seek(offset, SeekOrigin.Begin);
+                _logger.LogInformation(string.Format("Copying bytes range:{0},{1} filename:{2} ", offset, (!length.HasValue) ? null : (offset + length), fileName));
+                // await CopyToAsync(fileStream, _output, length, cancel);
+                await CopyToAsync(fileStream, _output);
+            }
+            finally
+            {
+                fileStream.Dispose();
+            }
+        }
 
         private const int maxbufferlen = 65536;
 
-        private byte[] buffer = new byte[maxbufferlen];
-
         private async Task CopyToAsync(FileStream fileStream, Stream output)
         {
-            await Task.Run(()=> fileStream.CopyTo(output, maxbufferlen));
+            await Task.Run(() => fileStream.CopyTo(output, maxbufferlen));
         }
     }
 }

src/Yavsc/Startup/Startup.WebSockets.cs

@@ -24,18 +24,5 @@ namespace Yavsc
             app.UseSignalR(PathString.FromUriComponent(Constants.SignalRPath));
         }
 
-        private async Task Echo(WebSocket webSocket)
-        {
-            var buffer = new byte[1024 * 4];
-            WebSocketReceiveResult result = await webSocket.ReceiveAsync(new ArraySegment<byte>(buffer), CancellationToken.None);
-            while (!result.CloseStatus.HasValue)
-            {
-                await webSocket.SendAsync(new ArraySegment<byte>(buffer, 0, result.Count), result.MessageType, result.EndOfMessage, CancellationToken.None);
-
-                result = await webSocket.ReceiveAsync(new ArraySegment<byte>(buffer), CancellationToken.None);
-            }
-            await webSocket.CloseAsync(result.CloseStatus.Value, result.CloseStatusDescription, CancellationToken.None);
-        }
-
     }
 }

src/Yavsc/ViewComponents/CalendarViewComponent.cs

@@ -8,15 +8,12 @@ namespace Yavsc.ViewComponents
 {
     public class CalendarViewComponent : ViewComponent
     {
-        readonly ApplicationDbContext _dbContext;
         readonly ICalendarManager _manager;
 
         public CalendarViewComponent (
-            ApplicationDbContext dbContext,
             ICalendarManager manager)
         {
             _manager = manager;
-            _dbContext = dbContext;
         }
 
         public async Task<IViewComponentResult> InvokeAsync (

src/Yavsc/ViewComponents/CirclesControlViewComponent.cs

@@ -11,7 +11,7 @@ namespace Yavsc.ViewComponents
 
     public class CirclesControlViewComponent : ViewComponent
     {
-        ApplicationDbContext dbContext;
+        private readonly ApplicationDbContext dbContext;
         public CirclesControlViewComponent(ApplicationDbContext dbContext)
         {
             this.dbContext = dbContext;

src/Yavsc/ViewComponents/CommentViewComponent.cs

@@ -6,11 +6,10 @@ namespace Yavsc.ViewComponents
 {
     public partial class CommentViewComponent : ViewComponent
     {
-        IStringLocalizer<CommentViewComponent> localizer;
+        private readonly IStringLocalizer<CommentViewComponent> localizer;
         public CommentViewComponent(IStringLocalizer<CommentViewComponent> localizer)
         {
-           this.localizer = localizer;
-           
+            this.localizer = localizer;
         }
         public IViewComponentResult Invoke(IIdentified<long> longCommentable)
         {

src/Yavsc/ViewComponents/TaggerComponent.cs

@@ -7,15 +7,11 @@ using Yavsc.Models;
 namespace Yavsc.ViewComponents
 {
     public class TaggerViewComponent : ViewComponent
-    
     {     
-        readonly ApplicationDbContext dbContext;
         readonly IStringLocalizer<TaggerViewComponent> localizer;
         public TaggerViewComponent(
-            ApplicationDbContext pdbContext,
             IStringLocalizer<TaggerViewComponent> pLocalizer)
         {
-            dbContext = pdbContext;
             this.localizer = pLocalizer;
         }
         public IViewComponentResult Invoke(ITaggable<long> longTaggable)

src/test/Mandatory/Remoting.cs

@@ -22,7 +22,7 @@ namespace test
     [Trait("regression", "oui")]
     public class Remoting : BaseTestContext, IClassFixture<ServerSideFixture>
     {
-        RegiserAPI r;
+        readonly RegiserAPI r;
         public Remoting(ServerSideFixture serverFixture, ITestOutputHelper output)
         : base(output, serverFixture)
         {

src/test/Mandatory/ServerSideFixture.cs

@@ -22,8 +22,8 @@ namespace test
         SiteSettings _siteSetup;
         ILogger _logger;
         IApplication _app;
-        EMailer _mailer;
-        ILoggerFactory _loggerFactory;
+        readonly EMailer _mailer;
+        readonly ILoggerFactory _loggerFactory;
         IEmailSender _mailSender;
 
         public static string ApiKey => "53f4d5da-93a9-4584-82f9-b8fdf243b002";
@@ -95,8 +95,8 @@ namespace test
             }
         }
         bool dbCreated;
-        private WebHostBuilder host;
-        private IHostingEngine hostengnine;
+        private readonly WebHostBuilder host;
+        private readonly IHostingEngine hostengnine;
 
 
 
@@ -145,7 +145,7 @@ namespace test
         }
 
 
-        private ModelBuilder modelBuilder;
+        private readonly ModelBuilder modelBuilder;
 
         public string TestingDatabase { get; private set; }
 

src/test/WIP/NotWorking.cs

@@ -19,11 +19,12 @@ namespace test
     {
         readonly SourceInformationProvider _sourceInfoProvider;
         readonly IOptions<LocalizationOptions> _localizationOptions;
-        public NotWorking( IHostingEnvironment env , IOptions<SiteSettings> siteSettings, 
+        public NotWorking(
         SourceInformationProvider sourceInfoProvider,
         IOptions<LocalizationOptions> localizationOptions,
         ServerSideFixture serverFixture, ITestOutputHelper output
-          ) : base (output, serverFixture) {
+          ) : base(output, serverFixture)
+        {
             _sourceInfoProvider = sourceInfoProvider;
             _localizationOptions = localizationOptions;
         }