Sign upLogin

RoushTech/RollbarDotNet

View on GitHub
Star

Showing 10 of 10 total issues

Method BuildFrames has 33 lines of code (exceeds 25 allowed). Consider refactoring.
Open

        protected List<Frame> BuildFrames(Exception exception)
        {
            var frames = new List<Frame>();
            var stacktrace = new System.Diagnostics.StackTrace(exception, true);
            var stackTraceFrames = stacktrace.GetFrames();
Severity: Minor
Found in src/RollbarDotNet/Builder/ExceptionBuilder.cs - About 1 hr to fix

    Similar blocks of code found in 3 locations. Consider refactoring.
    Open

            protected Dictionary<string, string> QueryToDictionary(IQueryCollection queryCollection)
            {
                var dictionary = new Dictionary<string, string>();
                foreach (var query in queryCollection)
                {
    Severity: Major
    Found in src/RollbarDotNet/Builder/RequestBuilder.cs and 2 other locations - About 1 hr to fix
    src/RollbarDotNet/Builder/RequestBuilder.cs on lines 102..111
    src/RollbarDotNet/Builder/RequestBuilder.cs on lines 125..135

    Similar blocks of code found in 3 locations. Consider refactoring.
    Open

            protected Dictionary<string, string> FormToDictionary(IFormCollection formCollection)
            {
                var dictionary = new Dictionary<string, string>();
                foreach (var form in formCollection)
                {
    Severity: Major
    Found in src/RollbarDotNet/Builder/RequestBuilder.cs and 2 other locations - About 1 hr to fix
    src/RollbarDotNet/Builder/RequestBuilder.cs on lines 113..123
    src/RollbarDotNet/Builder/RequestBuilder.cs on lines 125..135

    Similar blocks of code found in 3 locations. Consider refactoring.
    Open

            protected Dictionary<string, string> HeadersToDictionary(IHeaderDictionary headerDictionary)
            {
                var dictionary = new Dictionary<string, string>();
                foreach (var header in headerDictionary)
                {
    Severity: Major
    Found in src/RollbarDotNet/Builder/RequestBuilder.cs and 2 other locations - About 1 hr to fix
    src/RollbarDotNet/Builder/RequestBuilder.cs on lines 102..111
    src/RollbarDotNet/Builder/RequestBuilder.cs on lines 113..123

    Method QueryStringBreakdown has 26 lines of code (exceeds 25 allowed). Consider refactoring.
    Open

            protected string QueryStringBreakdown(string queryString)
            {
                if (queryString.Length == 0)
                {
                    return queryString;
    Severity: Minor
    Found in src/RollbarDotNet/Builder/RequestBuilder.cs - About 1 hr to fix

      Similar blocks of code found in 2 locations. Consider refactoring.
      Open

      namespace RollbarDotNet.Payloads
      {
          using Newtonsoft.Json;
       
          [JsonObject(MemberSerialization.OptIn)]
      Severity: Minor
      Found in src/RollbarDotNet/Payloads/Person.cs and 1 other location - About 45 mins to fix
      src/RollbarDotNet/Payloads/Exception.cs on lines 1..17

      Similar blocks of code found in 2 locations. Consider refactoring.
      Open

      namespace RollbarDotNet.Payloads
      {
          using Newtonsoft.Json;
       
          [JsonObject(MemberSerialization.OptIn)]
      Severity: Minor
      Found in src/RollbarDotNet/Payloads/Exception.cs and 1 other location - About 45 mins to fix
      src/RollbarDotNet/Payloads/Person.cs on lines 1..17

      Method Log has 5 arguments (exceeds 4 allowed). Consider refactoring.
      Open

              public void Log<TState>(LogLevel logLevel, EventId eventId, TState state, Exception exception,
                  Func<TState, Exception, string> formatter)
      Severity: Minor
      Found in src/RollbarDotNet/Logger/RollbarDotNetLogger.cs - About 35 mins to fix

        Multiple consecutive blank lines
        Open

         
         
        Severity: Info
        Found in Readme.md by markdownlint

        Bare URL used
        Open

        The UUID can be looked up directly via https://rollbar.com/occurrence/uuid/?uuid=[UUID HERE]. This may be really useful if you want to let your users report errors to you, you can include this UUID automatically in the report.
        Severity: Info
        Found in Readme.md by markdownlint
        Severity
        Category
        Status
        Source
        Language