Asgard MVP 1.1 #11
Merged
Asgard MVP 1.1 #11
Conversation
This file contains 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
Contributor
WarriorsSami
commented
May 11, 2024
WarriorsSami
commented
- General bug fixes
- Improve submission evaluation performance using Redis and Nginx
- now only a valid refresh token is required in order to issue new access tokens, as keeping the stale access tokens in sync with the refresh token using cookies is too intricate
…o requests/responses
- the expiry time is currently hardcoded in the envoy.yaml file as I haven't figured it out how to use env vars here yet
…of ProfilePictureUrl
Added logging statements to display the request path and HTTP method. Also added error handling for cases where cookies are missing or invalid, with appropriate response headers. The response headers now include `access-control-allow-credentials` and `access-control-allow-origin` to enable CORS support.
…ser registration and update validators
…me instead of name
…ct user pagination on GetAll endpoint
…lems (both published and unpublished)
…em on problem update endpoint
…ns per user (and problem if specified)
…improve performance Add RabbitMQ pub-sub using Dapr for publishing eval metadata related events when problems/tests are upserted/deleted in order to mitigate the overhead from Anubis submission evaluation
…has been solved previously by user
# Conflicts: # anubis-eval/.env.template # enki-problems/.env.template # enki-problems/src/EnkiProblems.Application/Problems/ProblemAppService.cs # enki-problems/src/EnkiProblems.HttpApi.Host/EnkiProblemsHttpApiHostModule.cs # odin-gateway/envoy.yaml # quetzalcoatl-auth/Api/Features/Auth/Login/Endpoint.cs # quetzalcoatl-auth/Api/Features/Auth/Register/Endpoint.cs # quetzalcoatl-auth/Api/Usings.cs # quetzalcoatl-auth/Infrastructure/ApplicationDbContext.cs
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.