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

Use resolved_middleware for Middleware Management and Uptime Checks #6803

Closed
Ashesh3 opened this issue Dec 5, 2023 · 0 comments · Fixed by #6875
Closed

Use resolved_middleware for Middleware Management and Uptime Checks #6803

Ashesh3 opened this issue Dec 5, 2023 · 0 comments · Fixed by #6875
Assignees

Comments

@Ashesh3
Copy link
Member

Ashesh3 commented Dec 5, 2023

With the upcoming merge of PR #1741 in the coronasafe/care repository, we have an opportunity to streamline our middleware management and asset uptime verification processes. The key feature of this PR is the introduction of the resolved_middleware field, which can significantly simplify our current codebase.

Objectives:

  1. Middleware Override: Utilize the resolved_middleware field to handle client-side asset configuration overrides. This approach should replace the existing method, providing a more structured and maintainable way to manage these overrides.

  2. Asset Uptime Checking: Incorporate the resolved_middleware field in our strategy for asset uptime verification. By doing so, we can reduce the complexity of our uptime checking logic, making it more efficient and easier to understand.

Expected Outcomes:

  • A unified system for middleware management.
  • Simplified and more maintainable code for asset configuration overrides.
  • Enhanced efficiency and clarity in asset uptime checking processes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

1 participant