aws / porting-assistant-dotnet-datastore Goto Github PK

Add the following fields to configurable features in the config file:

• FeatureCategory
• Description
• IsLinuxCompatible

When porting, add the Antlr3.Runtime package when any of the following namespaces are used:

• Antlr.Runtime
• Antlr.Runtime.Misc
• Antlr.Runtime.Tree

Remove OAuth directive and redundant actions.

Update rules and recommendation files to include .NET 6 support

Port the solution, migrate IIS web server settings to ,net core middleware

Config attributes include:
HttpModules, HttpHandlers, HttpsRedirection and added forwarded header kestrel

Solutions containing cyrillic (non-latin?) characters do not appear in the assessed solutions list. Re-adding solution is not possible, "Solution file already exists".

Steps to reproduce:

1. Download solution below:
   Кириллица.zip
2. Add a solution.

When running an assessment I don't get any compatibility score and almost all of my Nuget dependencies are marked with a status of "Failed" (not "Incompatible").

I have pre-built the solution successfully and I don't get any build issues with the tool. Some of the nuget packages are from a private repo but the majority are public and have newer, compatible versions (for instance CSVHelper and FakeItEasy).

In my assessment log I have a number of errors similar to the following:

[2020-08-25 10:49:15 INF] EncoreCache.ExternalPackagesCompatibilityChecker: Downloading fakeiteasy.json.gz from aws.portingassistant.dotnet.datastore
[2020-08-25 10:49:16 ERR] EncoreCache.ExternalPackagesCompatibilityChecker: Failed when download and parsing fakeiteasy.json.gz from aws.portingassistant.dotnet.datastore, Amazon.S3.AmazonS3Exception: Access Denied
 ---> Amazon.Runtime.Internal.HttpErrorResponseException: Exception of type 'Amazon.Runtime.Internal.HttpErrorResponseException' was thrown.
   at Amazon.Runtime.HttpWebRequestMessage.GetResponseAsync(CancellationToken cancellationToken)
....
[2020-08-25 10:49:33 INF] EncoreCache.ExternalPackagesCompatibilityChecker: Downloading system.json.gz from aws.portingassistant.dotnet.datastore
[2020-08-25 10:49:33 ERR] EncoreCache.ExternalPackagesCompatibilityChecker: Failed when download and parsing system.json.gz from aws.portingassistant.dotnet.datastore, Amazon.S3.AmazonS3Exception: Access Denied
 ---> Amazon.Runtime.Internal.HttpErrorResponseException: Exception of type 'Amazon.Runtime.Internal.HttpErrorResponseException' was thrown.
   at Amazon.Runtime.HttpWebRequestMessage.GetResponseAsync(CancellationToken cancellationToken)

The first line of the stack trace for these errors is EncoreCache.ExternalPackagesCompatibilityChecker.ProcessCompatibility(List'1 packageVersions, Dictionary'2 resultsDict)

Additionally I have no results in the API calls tab either.

The build fails for most of the projects in the solution I am trying to analyze. The build works fine when running in Visual Studio so I can't determine why the build is failing in the porting assistant tool.

Is there a log file of the build that I can look at to see why it is failing?

We need to implement rules for owin logging.

We need to add a new rule based on a customer contribution to cover porting from NPOI.Excel to NPOI package.

Add support for .netstandard2.1 to any rules that support it

Currently we do not support rules for OwinRequest and OwinResponse we need to add new rules to change them into .net core supported objects.

Add new rules file to support Microsoft.Owin.Security namespace.

Create rules for Microsoft.Owin.Security.DataHandler namespace

Could you point me how to trace why porting assistant says that Nunit 3.12 is deprecated? It is latest version available and it supports .NET Core.

We need new porting rules to cover the Microsoft.Owin.Security.DataProtection namespace.

Problem

• CTA pulls down resources.zip from S3 which contains templates and config files. These files are not version controlled.

Solution

• Add files to source control in their own directory
• Update workflow to automatically zip the files to resources.zip and upload to s3 bucket under ctafiles directory

We need to make the action validations more readable by adding spaces where appropriate. The validation process already trims the spaces in the action validations.

Currently Microsoft.Owin.Hosting.WebApp.Start and System.Diagnostics.Process.Start both match against an invocation expression rule for only the former.

The rule should only match against the first method from the Owin namespace.

When updating the code that retrieves data from the config files, we are no longer dependent on the System.Configuration and System.Web.Configuration namespaces. We need to remove them from the code files.

Microsoft.Owin new query string rule

Add the DotNetZip package when any of the following namespaces are found:

• Ionic
• Ionic.BZip2
• Ionic.Crc
• Ionic.Zip
• Ionic.Zlib

Add rules for Microsoft.Owin.Security.OpenIdConnect

Microsoft.Owin.Security.Authorization rules

Hi!

I tried to use this application, however when I choose a solution to assess it seems it starts doing something because I see some processes start and they are using the computers resources.

However, the list of assessed solutions stays empty. If I try to assess the same solution again the application says the solution is already added.

I have tried multiple solutions with the same result, sometimes I get a notification about not all nuget packages can be assessed but the list stays empty.

We need anew action to replace only the method we matched against.

Add a new owin namespace rules file to cover all eventualities for PhysicalFileSystems rule so it also applies outside object creation nodes.

Currently the owin static files rules are limited in scope and only add comments for the user to complete the changes. These rules will automate much of this for the user instead of relying on comments.

Some projects in the soloution I'm trying trying to analyze fail to build, and I get a notification "Failed to build 14 projects in x.sln" (in total the solution has 64 projects). The solution builds fine in Visual studio (and on our build servers etc, using MSBuild). The projects that fail seem to be the ones that have dependencies on nuget packages in our own feed (not on nuget.org)

How do I configure the available nuget feeds for the porting assistant?

I have tried to add our feed to the solution level .nuget/nuget.config file but that doesn't make any difference. In VS the feeds are configured in the package manager settings so it has no problem locating the feed when building/restoring. Building in VS first to get all packages restored does not help, I get the same error in the porting assistant even though everything is already restored.

All the projects that are failing are using the project SDK format with PackageReference elements.

We need to make sure the recommended owin version for netcore3.1 is not the latest version, since it's only net5.0 compatible

We need to implement rules to cover google authentication.

Add Rules to Support Migration of System.ServiceModel and related Namespaces to CoreWCF counterpart

We need to create new rules to port IHeaderDictionary and CreatePerOwinContext.

The code used to read web.config settings is not getting migrated when porting to .net core. We need to create rules to migrate this code to use the appsettings.json configuration.

We need a new rule to remove the override modifier from Invoke method declaration inside of classes implementing the OwinMiddleware base class.

Add a new owin rule to remove the override modifier form owinMiddleware base class type of classes.

Currently we check all comments and other actions the same way, we should add some granularity to reduce the amount of errors happening during validation due to comments impact.

Microsoft.AspNet.Identity.Owin rules

Currently we do not have rules to port RequestCookieCollection, ResponseCookieCollection and IReadableStringCollection from the Microsoft.Owin namespace.

Add removal of old namespaces

Hi,

I'm trying to use this application but am getting stuck when selecting a named profile. I have followed the pre-requisites here and have added an inline policy to the role my profile is associated with but when clicking Next I just get the error:

PortingRole does not have the correct IAM policies. If you need help setting up your profile see Learn more above.

I have also tried creating a Role with the required permission only and then creating a Named Profile which assumes this role through the use of the source_profile and role_arn options in the credentials file. This does not work either.

Other potentially useful information:

The credentials are temporary credentials for my Federated User.
I am behind an authenticated proxy (however my AWS CLI works fine)
I have tried without the proxy and it does not work (although if the tool is a .net app and is configured to pick up the system proxy this might explain the issue)

Many thanks :)

Create 0 build error owin solution

Create rules for Microsoft.Owin.Security.DataHandler.Encoder namespace

Develop rules for Owin PathString and OpenIdConnect.

We need to implement rules for Microsoft.Owin.Infrastructure.

Need an Owin rule to add a global variable within an owin middleware object to hold the next pipeline element.
Need an Owin rule to remove the constructor initializer within an owin middleware object.
Need an Owin rule to add an expression to initialize the next pipeline object within an owin middleware constructor.
Need an Owin rule to remove reference to the OwinMiddleware Next variable and replace it with a newly declared pipeline object.

Rules for the Microsoft.Owin.Security.Infrastructure namespace

Add rules for microsoft.owin.security.notifications

A new version of Microsoft.AspNetCore.OData was released with updated namespaces. We need to update our rules to match the new namespace

Add 3 rule files for Owin. Owin, Microsoft.Owin and Microsoft.Owin.Hosting.