Skip to content
This repository has been archived by the owner on Aug 19, 2024. It is now read-only.

Fixed missing middleware and incorrect API key #66

Merged
merged 2 commits into from
Jun 7, 2024

Conversation

qstokkink
Copy link
Owner

@qstokkink qstokkink commented Jun 7, 2024

Fixes #65

This PR:

  • Fixes the RootEndpoint not initializing the middlewares.
  • Fixes the API key not being written to the config (unless it was None).
  • Fixes the validation action not finding libsodium.dll.
  • Updates the unit test runner to allow other directories.

@qstokkink
Copy link
Owner Author

validate

@qstokkink
Copy link
Owner Author

It seems the validation action will need its own python runner script in order to set the DLL import location correctly. Sadly, this means that we will have to depart from PowerShell.

@qstokkink qstokkink marked this pull request as ready for review June 7, 2024 10:13
@qstokkink qstokkink changed the title WIP: Fixed missing middleware and incorrect API key READY: Fixed missing middleware and incorrect API key Jun 7, 2024
@qstokkink qstokkink changed the title READY: Fixed missing middleware and incorrect API key Fixed missing middleware and incorrect API key Jun 7, 2024
@qstokkink qstokkink merged commit 195d581 into highlander Jun 7, 2024
5 checks passed
@qstokkink qstokkink deleted the fix_missingware branch June 7, 2024 10:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

RootEndpoint does not hook in middleware
2 participants