Release 1.5.0 #5
Merged
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
- Added logging of request headers in JwtDefaultGuard to assist with debugging SSL_CERT and JWT issues. - Marked the logging line with a TODO for future removal.
- Updated the logging of request headers to use JSON.stringify for better readability. - This change aids in debugging SSL_CERT and JWT issues while maintaining the existing TODO for future removal.
chore: add restart flag "always"
- Updated package versions in package.json and package-lock.json, including major updates for eslint, prettier, and various NestJS packages. - Changed TypeScript module settings in tsconfig.json to use NodeNext. - Enhanced logging in main.ts and adjusted JWT configuration to use JWT_PUBLIC_KEY. - Updated config schema to handle JWT_PUBLIC_KEY and improved error handling for SSL_CERT. - Refactored imports and code structure for better clarity and maintainability.
- Updated the build-dev.yml to support both linux/amd64 and linux/arm64 platforms for Docker builds.
- Updated getStartMessage to include XRay version, path, and system info. - Refactored XrayController to ensure proper guard usage. - Added getXrayInfo method in XrayService to retrieve XRay details.
…esponseModel - Updated the response model to only include users with non-zero uplink or downlink values
…pendencies - Updated Xray installation script version to v25.3.6 in both Dockerfile and Dockerfile.dev. - Upgraded Node.js version to v22.14.0 in Dockerfile.dev. - Updated @nestjs/config to version 4.0.2 and @remnawave/xtls-sdk to version 0.3.0 in package.json and package-lock.json.
- Added @remnawave/supervisord-nestjs package to package.json and package-lock.json. - Configured SupervisordNestjsModule in app.module.ts for process management. - Updated supervisord.conf to include inet_http_server settings for remote access. - Refactored XrayService to utilize supervisord-nestjs for process control, replacing direct supervisorctl commands. - Enhanced error handling and logging for process management operations.
- Introduced p-retry package for improved retry handling in XrayService. - Enhanced logging format in main.ts to include custom log filter and timestamp. - Updated package.json and package-lock.json to reflect new dependencies and versions. - Refactored error handling in XrayService to provide clearer feedback on process status.
- Modified XRAY_DEFAULT_API_MODEL to include a new listen address and updated the tag. - Commented out unused inbounds configuration in generate-api-config.ts. - Introduced xtlsConfigInbounds in XrayService to extract inbound tags from the generated API config.
- Added XrayModule to HandlerModule imports for dependency injection. - Enhanced HandlerService to utilize XrayService for retrieving inbound tags. - Updated addUser method to remove users from tags using XrayService.
- Updated package version in package.json to 0.5.0. - Removed 'tags' property from IRemoveUserRequest interface and corresponding command handler. - Adjusted HandlerService to utilize inbound tags from XrayService instead of directly from the request.
Supervisord
…lication bootstrap
…ler and XrayService
- Changed host from 'http://localhost:61002' to 'http://127.0.0.1:61002' for improved compatibility in local environments.
![greptile-apps[bot]](https://avatars.githubusercontent.com/in/867647?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
PR Summary
This PR introduces version bumps, platform enhancements, and configuration updates for the release 1.5.0 of remnawave/node.
- Updated Dockerfiles (both production and dev) with Xray upgrade to v25.3.6 and a TCP-based Supervisor in supervisord.conf; verify compatibility.
- Enhanced the build workflow in
.github/workflows/build-dev.ymlto support linux/arm64 alongside amd64. - Switched JWT configuration from
SSL_CERTtoJWT_PUBLIC_KEYin bothsrc/common/config/jwt/jwt.config.tsand token validation. - Added a new ESLint configuration (
eslint.config.mjs) while renaming legacy.old.eslintrc.js, noting potential tsconfigRootDir discrepancies. - Removed the 'tags' field in remove-user commands and adjusted handler logic accordingly; check for multi-user edge cases.
35 file(s) reviewed, 8 comment(s)
Edit PR Review Bot Settings | Greptile
…controllers - Simplified constructor definitions by removing redundant assignments in InternalController, StatsController, and XrayController
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.
No description provided.