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

Add Backend custom Logger #120

Merged
merged 23 commits into from
Mar 31, 2024
Merged

Add Backend custom Logger #120

merged 23 commits into from
Mar 31, 2024

Conversation

AntonioMrtz
Copy link
Owner

Description

Add custom loggers and refactor every module except services and routers

Commit type

Feat

Issue

Solution

Proposed Changes

  • Create custom logger and formatter
  • Enable logging into files
  • Create LogPropertiesManager to manage logging settings
  • Create custom errors hierachy
  • Add ping to database and handle custom errors
  • Test database conneciton
  • Add logging into all modules except services and routers
  • Solve sending of jsons inside jsons
  • Adapt frontend and backend testing to refactor not sending jsons inside jsons
  • Delete PlaylistDTO and move PlaylistDTO logic and behaviour into Playlist
  • Simplify Genre class
  • Refactor hashed password and associated testing

Potential Impact

Tests Performed

  • [✅] Test performed

Screenshots

Additional Tasks

Assigned

@AntonioMrtz

@AntonioMrtz
feat : create Logger and add it to main
8c579d8
@AntonioMrtz
feat : add logger to PropertiesManager
0614edf
@AntonioMrtz
feat : add logger for Database connection
ff8e110
@AntonioMrtz
feat : add logger for CheckJwtMiddleware
2d04bd6
@AntonioMrtz
fix : playlist get song failing for both aws and db-blob
b4dbc3e
@AntonioMrtz
refactor : password encrypt and verification moved into methods
8800662
@AntonioMrtz
fix : typo extends on PlaylistDTO add songs method
c5b458a
@AntonioMrtz
refactor : simplify genre Enum checkings and add tests
a8292e7
@AntonioMrtz
refactor : remove PlaylistDTO and use Playlist instead
2e73eaf
@AntonioMrtz
feat : search by name uses SongDTO
f890e0b
@AntonioMrtz
test : adapt search tests to new response
90e887d
@AntonioMrtz
test : adapt explorar tests to new response
06c82b9
@AntonioMrtz
feat : remove embeded JSON from get all playlists/ endpoint
a815dbd
@AntonioMrtz
feat : remove embeded JSON from get all artists/ endpoint
1dd63c2
@AntonioMrtz
feat : remove embeded JSON from get playlists/multiple/ endpoint
69860de
@AntonioMrtz
feat : remove embeded JSON from get /generos/generos endpoint
51eea6f
@AntonioMrtz
feat : add select logging file and level
b8587f7
@AntonioMrtz
refactor : LogProperties transform into Borg Pattern
c7c5fae
@AntonioMrtz
refactor : change Database handling ping exceptions
66e39f5
@AntonioMrtz
test : add tests to database
da7d3a9
@AntonioMrtz
Copy link
Owner Author

Logger con custom clases para errores

@AntonioMrtz AntonioMrtz merged commit 95dd5d7 into master Mar 31, 2024
1 check passed
@AntonioMrtz AntonioMrtz deleted the feat/Add-Custom-Logger branch March 31, 2024 15:22
@AntonioMrtz AntonioMrtz changed the title Add custom Logger Add Backend custom Logger Jul 2, 2024
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
@AntonioMrtz