EHD-1663: Reduce errors in Sentry: Check if Return URLs are valid loc… #135
jamesgriff
This run and associated checks have been archived and are scheduled for deletion.
Learn more about checks retention
Build
1m 42s
Deploy (dev)
/
deploy
Deploy (prod)
/
deploy
Annotations
1 error and 30 warnings
|
Pause the workflow here
The run was canceled by @github-actions[bot].
|
|
Build:
GenderPayGap.UnitTests/GenderPayGap.WebUI.Tests/TestHelpers/MockRequestCookieCollection.cs#L12
'MockRequestCookieCollection.Keys' hides inherited member 'Dictionary<string, string>.Keys'. Use the new keyword if hiding was intended.
|
|
Build:
GenderPayGap.UnitTests/GenderPayGap.WebUI.Tests/ExtensionsAspNetCoreTests/ConfigTests.cs#L20
Consider using the constraint model, Assert.That(actual, Is.EqualTo(expected)), instead of the classic model, Assert.AreEqual(expected, actual) (https://github.com/nunit/nunit.analyzers/tree/master/documentation/NUnit2005.md)
|
|
Build:
GenderPayGap.UnitTests/GenderPayGap.WebUI.Tests/BusinessLogicTests/Repositories/UserRepository/UpdatePasswordTests.cs#L57
Consider using the constraint model, Assert.That(actual, Is.EqualTo(expected)), instead of the classic model, Assert.AreEqual(expected, actual) (https://github.com/nunit/nunit.analyzers/tree/master/documentation/NUnit2005.md)
|
|
Build:
GenderPayGap.UnitTests/GenderPayGap.WebUI.Tests/BusinessLogicTests/Repositories/UserRepository/UpdatePasswordTests.cs#L58
Consider using the constraint model, Assert.That(actual, Is.EqualTo(expected)), instead of the classic model, Assert.AreEqual(expected, actual) (https://github.com/nunit/nunit.analyzers/tree/master/documentation/NUnit2005.md)
|
|
Build:
GenderPayGap.UnitTests/GenderPayGap.WebUI.Tests/BusinessLogicTests/Repositories/UserRepository/UpdatePasswordTests.cs#L79
Consider using the constraint model, Assert.That(actual, Is.EqualTo(expected)), instead of the classic model, Assert.AreEqual(expected, actual) (https://github.com/nunit/nunit.analyzers/tree/master/documentation/NUnit2005.md)
|
|
Build:
GenderPayGap.UnitTests/GenderPayGap.WebUI.Tests/BusinessLogicTests/Repositories/UserRepository/UpdatePasswordTests.cs#L83
Consider using the constraint model, Assert.That(actual, Is.EqualTo(expected)), instead of the classic model, Assert.AreEqual(expected, actual) (https://github.com/nunit/nunit.analyzers/tree/master/documentation/NUnit2005.md)
|
|
Build:
GenderPayGap.UnitTests/GenderPayGap.WebUI.Tests/BusinessLogicTests/Repositories/UserRepository/UpdatePasswordTests.cs#L84
Consider using the constraint model, Assert.That(actual, Is.EqualTo(expected)), instead of the classic model, Assert.AreEqual(expected, actual) (https://github.com/nunit/nunit.analyzers/tree/master/documentation/NUnit2005.md)
|
|
Build:
GenderPayGap.UnitTests/GenderPayGap.WebUI.Tests/BusinessLogicTests/Repositories/UserRepository/UpdatePasswordTests.cs#L102
Consider using the constraint model, Assert.That(actual, Is.EqualTo(expected)), instead of the classic model, Assert.AreEqual(expected, actual) (https://github.com/nunit/nunit.analyzers/tree/master/documentation/NUnit2005.md)
|
|
Build:
GenderPayGap.UnitTests/GenderPayGap.WebUI.Tests/BusinessLogicTests/Repositories/UserRepository/UpdateEmailTests.cs#L58
Consider using the constraint model, Assert.That(actual, Is.EqualTo(expected)), instead of the classic model, Assert.AreEqual(expected, actual) (https://github.com/nunit/nunit.analyzers/tree/master/documentation/NUnit2005.md)
|
|
Build:
GenderPayGap.UnitTests/GenderPayGap.WebUI.Tests/BusinessLogicTests/Repositories/UserRepository/UpdateEmailTests.cs#L81
Consider using the constraint model, Assert.That(actual, Is.EqualTo(expected)), instead of the classic model, Assert.AreEqual(expected, actual) (https://github.com/nunit/nunit.analyzers/tree/master/documentation/NUnit2005.md)
|
|
Build:
GenderPayGap.Core/Extensions/Encryption.cs#L178
'RijndaelManaged' is obsolete: 'The Rijndael and RijndaelManaged types are obsolete. Use Aes instead.' (https://aka.ms/dotnet-warnings/SYSLIB0022)
|
|
Build:
GenderPayGap.Core/Extensions/Encryption.cs#L183
'Rfc2898DeriveBytes.Rfc2898DeriveBytes(string, byte[], int)' is obsolete: 'The default hash algorithm and iteration counts in Rfc2898DeriveBytes constructors are outdated and insecure. Use a constructor that accepts the hash algorithm and the number of iterations.' (https://aka.ms/dotnet-warnings/SYSLIB0041)
|
|
Build:
GenderPayGap.Core/Extensions/AspNetCore/Config.cs#L11
The field 'Config._EnvironmentName' is never used
|
|
Build:
GenderPayGap.Database/Models/PublicSectorType.cs#L7
'PublicSectorType' overrides Object.Equals(object o) but does not override Object.GetHashCode()
|
|
Build:
GenderPayGap.Database/Models/SicCode.cs#L6
'SicCode' overrides Object.Equals(object o) but does not override Object.GetHashCode()
|
|
Build:
GenderPayGap.Database/Models/SicSection.cs#L6
'SicSection' overrides Object.Equals(object o) but does not override Object.GetHashCode()
|
|
Build:
GenderPayGap.Database/Models/Extensions/OrganisationAddress.cs#L58
'OrganisationAddress' overrides Object.Equals(object o) but does not override Object.GetHashCode()
|
|
Build:
GenderPayGap.Database/GpgDatabaseContext.cs#L98
'GpgDatabaseContext.SaveChanges()' hides inherited member 'DbContext.SaveChanges()'. To make the current member override that implementation, add the override keyword. Otherwise add the new keyword.
|
|
Build:
GenderPayGap.Database/Models/Extensions/User.cs#L12
'User' overrides Object.Equals(object o) but does not override Object.GetHashCode()
|
|
Build:
GenderPayGap.Database/Models/Extensions/User.cs#L35
'Encryption.DecryptData(string)' is obsolete
|
|
Build:
GenderPayGap.Core/Extensions/Encryption.cs#L178
'RijndaelManaged' is obsolete: 'The Rijndael and RijndaelManaged types are obsolete. Use Aes instead.' (https://aka.ms/dotnet-warnings/SYSLIB0022)
|
|
Build:
GenderPayGap.Core/Extensions/Encryption.cs#L183
'Rfc2898DeriveBytes.Rfc2898DeriveBytes(string, byte[], int)' is obsolete: 'The default hash algorithm and iteration counts in Rfc2898DeriveBytes constructors are outdated and insecure. Use a constructor that accepts the hash algorithm and the number of iterations.' (https://aka.ms/dotnet-warnings/SYSLIB0041)
|
|
Build:
GenderPayGap.Core/Extensions/AspNetCore/Config.cs#L11
The field 'Config._EnvironmentName' is never used
|
|
Build:
GenderPayGap.Database/Models/Extensions/OrganisationAddress.cs#L58
'OrganisationAddress' overrides Object.Equals(object o) but does not override Object.GetHashCode()
|
|
Build:
GenderPayGap.Database/Models/PublicSectorType.cs#L7
'PublicSectorType' overrides Object.Equals(object o) but does not override Object.GetHashCode()
|
|
Build:
GenderPayGap.Database/Models/SicCode.cs#L6
'SicCode' overrides Object.Equals(object o) but does not override Object.GetHashCode()
|
|
Build:
GenderPayGap.Database/Models/SicSection.cs#L6
'SicSection' overrides Object.Equals(object o) but does not override Object.GetHashCode()
|
|
Build:
GenderPayGap.Database/Models/Extensions/User.cs#L12
'User' overrides Object.Equals(object o) but does not override Object.GetHashCode()
|
|
Build:
GenderPayGap.Database/GpgDatabaseContext.cs#L98
'GpgDatabaseContext.SaveChanges()' hides inherited member 'DbContext.SaveChanges()'. To make the current member override that implementation, add the override keyword. Otherwise add the new keyword.
|
|
Build:
GenderPayGap.Database/Models/Extensions/User.cs#L35
'Encryption.DecryptData(string)' is obsolete
|
Artifacts
Produced during runtime
| Name | Size | Digest | |
|---|---|---|---|
|
build-zip
Expired
|
20.5 MB |
sha256:ed33a16c50003f091a04084c30d71913cbd90f0eabd4463e609ee45997ff3c81
|
|