forked from Dokploy/dokploy
-
Notifications
You must be signed in to change notification settings - Fork 160
Feature/trim ssh white #13
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
Nicolas-Nogueira
wants to merge
670
commits into
codepath:canary
Choose a base branch
from
Nicolas-Nogueira:feature/trim_SSH_whit
base: canary
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Feature/trim ssh white #13
Nicolas-Nogueira
wants to merge
670
commits into
codepath:canary
from
Nicolas-Nogueira:feature/trim_SSH_whit
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
…ption-when-deleting-in-bulk feat(ui): add bulk deploy functionality for services in project dashb…
fix(organization): integrate active organization refetching on update/create
feat: Self reference env variables
feat: bump Traefik v3.2.2 and add swarm network label
bump: Traefik 3.5.0
…into feat/label-previews
…into feat/label-previews
…ing Bitbucket object directly
feat(bitbucket): Deprecate App password and replace it with API token
…nvironments - Fix sorting logic to count actual services instead of environment count - Projects now properly sort by total service count in descending order - Resolves issue where "Most services" showed ascending order instead of descending -Dokploy#2691
…cancellation logic - Updated the deployment status enum to include 'cancelled'. - Added a new utility function to handle the cancellation of deployments, setting their status to 'error'. - Enhanced the status tooltip component to display 'Cancelled' when the status is 'cancelled'. - Created a new SQL migration to add the 'cancelled' value to the deploymentStatus type.
…per status update - Updated the initCancelDeployments function to set the status of running deployments to 'cancelled' instead of 'error'. - Reintroduced the call to initCancelDeployments in the server initialization process to ensure cancellations are handled correctly.
…itialization - Added the call to initCancelDeployments back into the server initialization process to ensure that deployment cancellations are handled correctly in all environments.
…s-has-no-effect-#2691 Feat: "Most services" sorting to count total services across environments -2691
…-mark-as-running-when-they-never-ended-vps-shutdown 2670 bug deployments are mark as running when they never ended vps shutdown
- Added support for Bitbucket email and workspace name in the authentication process. - Updated the clone URL generation to use the correct format for API tokens. - Improved error handling to ensure required fields are provided for both API tokens and app passwords. - Added console logs for debugging clone URL and repository information during cloning.
…g process - Removed console logs for clone URL and repository information to clean up the output during the cloning process.
…ts-are-broken-auth-token-wont-work fix(bitbucket): enhance Bitbucket authentication handling
style: move Deployments tab after Domains tab
feat: add unsaved changes tracking and UI indication
feat(scheduler): auto-switch to 'Custom' on manual input
feat: support Arvancloud CDN detection
feat: support cloning repositories over HTTP in Gitea integration
fix: add environment in buildLink for docker compose deploy notifications
- Add automatic whitespace trimming to ipAddress field in apiCreateServer and apiUpdateServer schemas - Implement trimming in createServer and updateServerById service functions - Add comprehensive test suite for IP address trimming functionality - Include test cases for leading/trailing spaces, empty strings, and whitespace-only inputs - Add API testing script for manual validation This ensures consistent IP address formatting and prevents issues with leading/trailing whitespace in server configurations.
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.
🎯 Feature: IP Address Trimming for Server Management
Description
This PR adds automatic whitespace trimming to IP addresses when creating or updating servers, ensuring consistent data formatting and preventing issues with leading/trailing whitespace.
Changes Made
apiCreateServerandapiUpdateServerschemascreateServerandupdateServerByIdservice functionsTechnical Details
.transform()method to automatically trim whitespaceTesting
Screenshots
[You'll need to add before/after screenshots showing the IP address field behavior]
Video Demo
[You'll need to create a video showing:
Related Issues
Fixes #[issue-number] (if applicable)