Skip to content

Cleanups and adjustments#103

Merged
lsipii merged 8 commits intomainfrom
fix/todos-cleanup
Nov 24, 2023
Merged

Cleanups and adjustments#103
lsipii merged 8 commits intomainfrom
fix/todos-cleanup

Conversation

@lsipii
Copy link
Copy Markdown
Contributor

@lsipii lsipii commented Nov 24, 2023

  • enable swagger only on development and local environments
  • fix the Npgsql.EnableLegacyTimestampBehavior setting related todo:
    • in dotnet use DateTime.UtcNow instead of DateTime.Now when saving to db context
  • set daily backup retention time to 30 days in prod
  • clear the search profile related TODO-flags as resolved by the current implementation
  • add error handling to the users controller occupations save phase
  • ensure the security features initialization is completed before handling of a restricted request

@lsipii lsipii requested a review from LauriGofore November 24, 2023 12:13
Copy link
Copy Markdown
Contributor

@LauriGofore LauriGofore left a comment

Choose a reason for hiding this comment

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

Siistiä!

Npgsql.EnableLegacyTimestampBehavior miksihän oli päällä alunperin?

@lsipii
Copy link
Copy Markdown
Contributor Author

lsipii commented Nov 24, 2023

Npgsql.EnableLegacyTimestampBehavior miksihän oli päällä alunperin?

Tää oli mun joskus laittama TODO jonka jätin tilanteessa jossa korjasin ef-coren antaman virheen, jonka syy ei ollut silloin ihan selvä, ja se korjaus oli siis että laittaa tuon asetuksen päälle. Nyt kun tutkin asiaa uudestaan niin se kohdentui siihen että:

  • meillä aikaleima-kolumnit on (ef coren toimesta) luotu postgresql-tyyppiin timestamp with time zone, ja se on ok
  • koodiin oli päätynyt moneen yhteen paikkaan tapa luoda aikaleima: DateTime.Now, joka on kyseisen tietokoneen lokaali aika ilman zone-tietoa
  • virhe tuli siitä että tuota lokaalia aikaa yritettiin tallentaa postgres-kolumniin joka vaatii timezone-tiedon ja se väliaikaisesti korjaantui tuolla legacy-timestamp behaviour-lipulla

Tässä PR:ssä muutettiin että DateTime.Now sijasta käytetään DateTime.UtcNow, joka tietysti on UTC-aika ja siispä pystyi sitten ottamaan tuon vanhan fiksin pois ja todo-tada!

@lsipii lsipii merged commit 9a30e37 into main Nov 24, 2023
@lsipii lsipii deleted the fix/todos-cleanup branch November 24, 2023 15:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants