[AC-1698] Check if a user has 2FA enabled more efficiently #737
r-tometest-database.yml
on: pull_request
Run tests
2m 48s
Run validation
1m 47s
Annotations
31 warnings
|
Run validation
The following actions uses Node.js version which is deprecated and will be forced to run on node20: actions/setup-dotnet@3447fd6a9f9e57506b15f895c5b76d3b197dc7c2, actions/upload-artifact@a8a3f3ad30e3422c9c7b888a15615d19a852ae32. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
|
Run validation:
src/Core/Models/Mail/Provider/ProviderInitiateDeleteModel.cs#L11
'ProviderInitiateDeleteModel.WebVaultUrl' hides inherited member 'BaseMailModel.WebVaultUrl'. Use the new keyword if hiding was intended.
|
|
Run validation:
src/Core/Models/Mail/Provider/ProviderInitiateDeleteModel.cs#L14
'ProviderInitiateDeleteModel.SiteName' hides inherited member 'BaseMailModel.SiteName'. Use the new keyword if hiding was intended.
|
|
Run validation:
src/Core/Auth/Models/Api/Request/Accounts/RegisterVerificationEmailClickedRequestModel.cs#L12
Non-nullable property 'Email' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
|
|
Run validation:
src/Core/Auth/Models/Api/Request/Accounts/RegisterVerificationEmailClickedRequestModel.cs#L15
Non-nullable property 'EmailVerificationToken' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
|
|
Run validation:
src/Core/Auth/Models/Api/Request/Accounts/RegisterSendVerificationEmailRequestModel.cs#L13
Non-nullable property 'Email' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
|
|
Run validation:
src/Core/Auth/Services/Implementations/AuthRequestService.cs#L126
Dereference of a possibly null reference.
|
|
Run validation:
src/Core/Auth/Services/Implementations/AuthRequestService.cs#L139
Possible null reference argument for parameter 'user' in 'Task<AuthRequest> AuthRequestService.CreateAuthRequestAsync(AuthRequestCreateRequestModel model, User user, Guid? organizationId)'.
|
|
Run validation:
src/Core/Billing/Services/IProviderBillingService.cs#L20
XML comment has cref attribute 'seats' that could not be resolved
|
|
Run validation:
src/Core/Billing/Services/ISubscriberService.cs#L93
XML comment has cref attribute 'subscriber' that could not be resolved
|
|
Run tests
The following actions uses Node.js version which is deprecated and will be forced to run on node20: actions/setup-dotnet@3447fd6a9f9e57506b15f895c5b76d3b197dc7c2. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
|
Run tests:
test/Infrastructure.IntegrationTest/Tools/SendRepositoryTests.cs#L30
Dereference of a possibly null reference.
|
|
Run tests:
test/Infrastructure.IntegrationTest/AdminConsole/Repositories/OrganizationUserRepositoryTests.cs#L41
Dereference of a possibly null reference.
|
|
Run tests:
test/Infrastructure.IntegrationTest/Vault/Repositories/CipherRepositoryTests.cs#L43
Dereference of a possibly null reference.
|
|
Run tests:
test/Infrastructure.IntegrationTest/DistributedCacheTests.cs#L39
Dereference of a possibly null reference.
|
|
Run tests:
test/Infrastructure.IntegrationTest/Vault/Repositories/CollectionRepositoryTests.cs#L71
Dereference of a possibly null reference.
|
|
Run tests:
test/Infrastructure.IntegrationTest/Auth/Repositories/EmergencyAccessRepositoryTests.cs#L42
Dereference of a possibly null reference.
|
|
Run tests:
test/Infrastructure.IntegrationTest/AdminConsole/Repositories/OrganizationUserRepositoryTests.cs#L95
Dereference of a possibly null reference.
|
|
Run tests:
test/Infrastructure.IntegrationTest/AdminConsole/Repositories/OrganizationUserRepositoryTests.cs#L96
Dereference of a possibly null reference.
|
|
Run tests:
test/Infrastructure.IntegrationTest/Vault/Repositories/CipherRepositoryTests.cs#L68
Dereference of a possibly null reference.
|
|
Run tests:
test/Infrastructure.IntegrationTest/Vault/Repositories/CipherRepositoryTests.cs#L113
Dereference of a possibly null reference.
|
|
Run tests:
src/Core/Models/Mail/Provider/ProviderInitiateDeleteModel.cs#L11
'ProviderInitiateDeleteModel.WebVaultUrl' hides inherited member 'BaseMailModel.WebVaultUrl'. Use the new keyword if hiding was intended.
|
|
Run tests:
src/Core/Models/Mail/Provider/ProviderInitiateDeleteModel.cs#L14
'ProviderInitiateDeleteModel.SiteName' hides inherited member 'BaseMailModel.SiteName'. Use the new keyword if hiding was intended.
|
|
Run tests:
src/Core/Auth/Models/Api/Request/Accounts/RegisterVerificationEmailClickedRequestModel.cs#L12
Non-nullable property 'Email' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
|
|
Run tests:
src/Core/Auth/Models/Api/Request/Accounts/RegisterVerificationEmailClickedRequestModel.cs#L15
Non-nullable property 'EmailVerificationToken' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
|
|
Run tests:
src/Core/Auth/Models/Api/Request/Accounts/RegisterSendVerificationEmailRequestModel.cs#L13
Non-nullable property 'Email' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
|
|
Run tests:
src/Core/Auth/Services/Implementations/AuthRequestService.cs#L126
Dereference of a possibly null reference.
|
|
Run tests:
src/Core/Auth/Services/Implementations/AuthRequestService.cs#L139
Possible null reference argument for parameter 'user' in 'Task<AuthRequest> AuthRequestService.CreateAuthRequestAsync(AuthRequestCreateRequestModel model, User user, Guid? organizationId)'.
|
|
Run tests:
src/Core/Billing/Services/IProviderBillingService.cs#L20
XML comment has cref attribute 'seats' that could not be resolved
|
|
Run tests:
src/Core/Billing/Services/ISubscriberService.cs#L93
XML comment has cref attribute 'subscriber' that could not be resolved
|
|
Deprecation notice: v1, v2, and v3 of the artifact actions
The following artifacts were uploaded using a version of actions/upload-artifact that is scheduled for deprecation: "report.xml", "sql.dacpac".
Please update your workflow to use v4 of the artifact actions.
Learn more: https://github.blog/changelog/2024-04-16-deprecation-notice-v3-of-the-artifact-actions/
|
Artifacts
Produced during runtime
| Name | Size | |
|---|---|---|
|
report.xml
|
2.66 KB |
|
|
sql.dacpac
|
109 KB |
|