BC-11033 - Unify config handling in modules#6018
Open
bischofmax wants to merge 112 commits intomainfrom
Open
Conversation
…okie settings and remove unnecessary configService mocks
…ture and enhance alert status handling (#6017)
* feat(authentication): refactor logout use case to use injected config and improve error handling feat(authentication): add Valkey client session configuration for external system integration test(board): enhance collaboration gateway tests with configuration module for JWT test(fwu-learning-contents): update API tests to utilize JWT configuration for user login test(h5p-editor): modify H5P editor API tests to include JWT configuration for user authentication fix(oauth): update OAuth use case to use correct OauthConfig type for configuration service feat(oauth): extend OauthConfig to include external system logout feature flag refactor(server): clean up server configuration by removing unused authentication config test(jwt): create a test JWT module configuration for testing purposes refactor(jwt): update JWT authentication factory to accept configuration for key and domain * code review * refactor(valkey): standardize property names to camelCase in ValkeyConfig and related classes
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
* Remove NewsConfig interface and its dependencies from news.config.ts * Remove unused hostUri variable and mock implementation for Configuration in OAuthService tests * Remove NewsConfig interface and its references from deletion and news modules * Implement OAuth configuration and update related modules to support new feature flags * Refactor OAuth module to use OauthPublicApiConfig and update related configurations * Fix config type in OAuth unit tests to use OauthPublicApiConfig * Remove FEATURE_LOGIN_LINK_ENABLED from environment files and schema
…use new config structure (#6033)
…onfiguration structure (#6036)
3 tasks
* refactor(admin-api): implement configuration class for Admin API server * refactor(external-tool): remove unused Configuration import and related setup * refactor(course-rooms): remove unused Configuration imports and related setup * BC-11033 - Update timeout configuration to use class properties instead of schema definitions
…_MS` from ServerConfig interface
3 tasks
…tiple applications and remove obsolete middleware file (#6099) * Refactor request logging middleware integration across multiple applications and remove obsolete middleware file * Refactor logger imports across multiple application files to include LOGGER_CONFIG_TOKEN
…e unnecessary createMock wrapper
… enhance MailService to conditionally send emails based on configuration (#6103) * BC-11033 - Move email configuration and logging to dedicated modules; enhance MailService to conditionally send emails based on configuration * Remove redundant email sending configuration from HelpdeskConfig and related tests * Add StringToBoolean transformer to shouldSendEmail property in TestMailConfig
* Clean up server config * Remove unused STUDENT_TEAM_CREATION field from ServerConfig interface * Refactor server configuration: update to use ServerPublicApiConfig, add LICENSE_SUMMARY_URL, and clean up unused fields * Remove unused NodeEnvType enum and clean up comments in server configuration * Update video conference configuration: set featureVideoConferenceEnabled to false and enable it in tests * Enhance server configuration: add ROOM_MEMBER_INFO_URL to ConfigMap and update JWT timeout properties with StringToNumber transformation * Fix import path for AdminApiServerConfig in admin-api-server.app.ts --------- Co-authored-by: Sergej Hoffmann <97111299+SevenWaysDP@users.noreply.github.com>
…e provisioning configmap
… provisioning configmap
3 tasks
…ctor and configInjectionToken (#6107)
…to use DatabaseModule with configInjectionToken and configConstructor
…oved coverage analysis
|
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
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.
Description
Links to Tickets or other pull requests
Changes
Approval for review
generate-client:serverwas executed in vue frontend and changes were tested and put in a PR with the same branch name.