Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve performance #137

Merged
merged 1 commit into from
Aug 1, 2024
Merged

Improve performance #137

merged 1 commit into from
Aug 1, 2024

Conversation

Fosol
Copy link
Contributor

@Fosol Fosol commented Aug 1, 2024

Reviewed and improved performance of the history endpoint

Summary

  • Fixed local build/debug configuration
  • Add DB migration 1.0.4
  • Upgraded dependencies to resolve security issue
  • Updated history API endpoints to dramatically reduce size of download

@Fosol Fosol added bug Something isn't working enhancement New feature or request labels Aug 1, 2024
@Fosol Fosol self-assigned this Aug 1, 2024
@@ -13,9 +13,9 @@
<PackageReference Include="Microsoft.AspNetCore.Http" Version="2.2.2" />
<PackageReference Include="Microsoft.Extensions.Http" Version="8.0.0" />
<PackageReference Include="Microsoft.Extensions.Options" Version="8.0.2" />
<PackageReference Include="System.IdentityModel.Tokens.Jwt" Version="7.6.0" />
<PackageReference Include="System.IdentityModel.Tokens.Jwt" Version="8.0.1" />
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Security bulletin upgrade

<PackageReference Include="Microsoft.AspNetCore.Http" Version="2.2.2" />
<PackageReference Include="Microsoft.Extensions.Http" Version="8.0.0" />
<PackageReference Include="Microsoft.Extensions.Options" Version="8.0.2" />
<PackageReference Include="System.Net.Http" Version="4.3.4" />
<PackageReference Include="System.Text.Json" Version="8.0.3" />
<PackageReference Include="System.Text.Json" Version="8.0.4" />
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Security bulletin upgrade

[typeof(int), typeof(DateTime), typeof(DateTime?), typeof(int?), typeof(int?), typeof(int?), typeof(string)])!
).HasName(nameof(FindFileSystemHistoryItemsByMonthForUser));

// modelBuilder.Entity<ServerHistoryItemsByMonth>().ToTable((string?)null);
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Various attempts to get Entity Framework to allow mapping the stored procedures to other objects. I was unsuccessful, but hope to figure this out still.

@@ -10,7 +10,7 @@
"request": "launch",
"preLaunchTask": "build",
// If you have changed target frameworks, make sure to update the program path.
"program": "${workspaceFolder}/src/api/bin/Debug/net7.0/HSB.API.dll",
"program": "${workspaceFolder}/src/api/bin/Debug/net8.0/HSB.API.dll",
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed local build/debug

<PackageReference Include="Microsoft.AspNetCore.Http" Version="2.2.2" />
<PackageReference Include="Microsoft.Extensions.Http" Version="8.0.0" />
<PackageReference Include="Microsoft.Extensions.Options" Version="8.0.2" />
<PackageReference Include="System.Net.Http" Version="4.3.4" />
<PackageReference Include="System.Text.Json" Version="8.0.3" />
<PackageReference Include="System.Text.Json" Version="8.0.4" />
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Security bulletin upgrade

@Fosol Fosol merged commit 59c54ce into bcgov:main Aug 1, 2024
1 check passed
@Fosol Fosol deleted the zoom branch August 1, 2024 16:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant