Fix/address comments - #29
Open
samuelmartinucci2 wants to merge 4 commits into
Open
Conversation
* feat: implement refactor-arch skill and refactor projects 1 and 2 to MVC Complete manual analysis in README.md, build refactor-arch skill, and fully refactor Python/Flask and Node.js/Express backends to safe MVC architecture. * feat: complete mvc refactoring of task-manager-api and fix all audit-project-3 findings - Extracted Flask configurations to secure config/settings.py - Refactored routes into dedicated controllers/ layer to eliminate Fat Controllers - Fixed timezone-aware vs naive comparison TypeError in task.is_overdue() - Resolved deprecated datetime.utcnow() usages in favor of timezone-aware objects - Standardized task serialization to use Task.to_dict() across endpoints - Removed generic exception masking in User update and logged/returned exact errors - Added comprehensive test_endpoints.py suite covering health, reports, and user-task flows * fix: enforce stateless services and remediate security findings * fix: refactor projects to MVC and finalize audit reports * fix: re-generate authentic, non-synthetic audit reports * fix: re-generate audit reports with strictly required finding distribution * feat: finalize MVC refactor, sync skills, and update authentic audit reports
- Replace SHA-256 with bcrypt in playbooks and ecommerce-api-legacy code. - Implement logAndCache in ecommerce-api-legacy utils. - Add additional MEDIUM and LOW issues in manual analysis tables. - Complete 'Resultados' and 'Como Executar' sections in README.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.