PR: Merging in Main #63
Open
Conversation
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
…file * add X-Total-Count header, restructure and rename filters -only for endpoints that return a list -implemented by introducing a new XTotalCountResponsefilter * add dotenv to parent pom, move hardcoded origin regex to application.properties * add env file to image (default values), fix XTotalCountResponseFilterIT test, remove LoggingRequestFilterIT.java * remove old RequestLoggingFilter.java, add missing dependency to parent pom * add head request test, validate whether credential are allowed when cors is enabled closes #51
fix XTotalCountResponseFilterIT not using Authentication
-only for endpoints that return a list -implemented by introducing a new XTotalCountResponsefilter fixes #51
…IT test, remove LoggingRequestFilterIT.java fixes #51
known-issue: headers not set, has a conflict with xtotalcountfilter fixes #51
…e method for filtering and findall -add count method that counts filtered and unfiltered matches to MatchRepositoryImpl -rename some methods -change findAllFilteredOrUnfiltered query to work with pagination (issue when collections are joined), solved by subselect/query -add new tests/edit old tests using the new changes to MatchRepositoryTest and MatchResourceImplIT -add one new integration test -add back jboss.home variable, so that repository tests can be executed via intellij again (needed by arquillian) -add equals and hashcode to match issue #39
…ceding test and some ceiling issues using int issue #39
Add Pagination
add IT Tests for new Endpoint add UnitTests for new ServiceMethod used for validation add ValidateTokenCommand add properties-maven-plugin to read secret.env and set needed SECRET_JWT_HASH as envVariable edit README.md issue #58
Update CorsResponseFilterIT tests issue: #58
…rtUp instead of checking/executing it on every login request issue #58
… be faster on login and safe enough issue #58
…o 12 hours issue #58
Now all AuthUsers are removed on startup and a new user gets created based on envs without checking whether one exists issue #58
…o return users sorted by overall match count by default remove UserRepository delete method issue: #56
alekszivko
approved these changes
Apr 16, 2025
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.
Merge dev in main