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.

Sign up for GitHub

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

Jump to bottom

Refactoring/project setup change #320

Merged
merged 40 commits into from
Aug 10, 2023
Merged

Refactoring/project setup change #320

merged 40 commits into from
Aug 10, 2023

Conversation

dukeofharen
Copy link
Owner

@dukeofharen dukeofharen commented Aug 10, 2023 •
edited
Loading

Pull request type

Please check the type of change your PR introduces:

  • Bugfix
  • Feature
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Documentation content changes
  • Other (please describe):

What is the new behavior?

  • Made several infrastructure changes to the code.
  • Added AuditBehavior which logs all the interactions between the web API and the application layer using MediatR (only in verbose logging mode).

Does this introduce a breaking change?

  • Yes
  • No

Duco added 30 commits July 30, 2023 20:28
Moved stubcontext from persistence to application project
410fc41
Deleted ApiHeadersMiddleware.cs
67016d4
Moved cookie stuff from LoginService to HttpContextService.cs
b572a62
Added basic web infrastructure module
12127d8
Moved UserContext and LoginService to web infrastructure project
66d1015
Introduced test project for web infrastructure
385f169
Moved more stuff over to web infra
967c12f
Introduced StubHandler
cf13095
Split off tests from StubHandlingMiddlewareFacts
a458c60
Fix
a0a7612
Renamed LoginService to LoginCookieService
8374f8c
Introduced ApiAuthorizationService
35259d6
Moved more logic from StubHandlingMiddleware.cs to SubHandler
4d733a6
Created test factory method for creating IHttpContextAccessor mock
2dba312
Added unit test for UserContext
dadd38b
Moved ICustomHostedService.cs interface to application project
2f3ecde
Moved scheduled job logic to application project
3673979
Added unit tests
aa7065c
Added command for executing stub from web project
e8b6a3f
Moved some stuff around
c85a8f0
README fix
5278c1c
Nuget upgrade
385b571
Updated MediatR to 12
fdc042f
Fixed MediatR
1dd9235
Separated check for verbosity
3681054
Added verbose logging to config parsing
01d4b85
Re-added Unit to commands without response
dcf3bcc
Fixed unit tests
40dc515
Implemented AuditBehavior
2b4c170
Implemented AuditBehavior and added unit tests
6da4423
@dukeofharen dukeofharen merged commit f0e62af into master Aug 10, 2023
@dukeofharen dukeofharen deleted the refactoring/project-setup-change branch August 10, 2023 20:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@dukeofharen