Skip to content

feat: Add port availability validation and improve terminal viewer experience and added new column for token revoking#237

Merged
purvangIQUD merged 3 commits into
mainfrom
feature/dp/revoke-token-existing-port
Jun 24, 2026
Merged

feat: Add port availability validation and improve terminal viewer experience and added new column for token revoking#237
purvangIQUD merged 3 commits into
mainfrom
feature/dp/revoke-token-existing-port

Conversation

@dhaval-p-iqud

@dhaval-p-iqud dhaval-p-iqud commented Jun 24, 2026

Copy link
Copy Markdown
Collaborator

📝 What does this change do?

Implemented pre-deployment port availability validation and improved terminal viewer functionality.

  • Added port availability checks before starting service deployments to prevent applications from running on already occupied ports.
  • Integrated port validation flow between Control Panel and Agent App using existing socket communication.
  • Added proper deployment failure handling and user-friendly error messages for port conflicts.
  • Enhanced terminal viewer with horizontal scrolling support to display complete command output and logs without wrapping.
  • Improved terminal fitting logic to dynamically manage content width and responsiveness across different screen sizes.
  • Updated terminal UI interactions including wheel handling and scroll behavior improvements.
  • Added revokedAt column for storing revoked token timestamp

🔗 Related Issue / Task

🧪 How was this tested?

  • [✅] Manually tested

Steps to Test:

  1. Configure a service with a port that is already occupied.
  2. Trigger deployment and verify deployment is blocked with the proper port conflict message.
  3. Configure a service with an available port and verify deployment works normally.
  4. Open terminal viewer and verify horizontal scrolling, vertical scrolling, and responsive behavior on different screen sizes.

📸 Screenshots (if UI changes)

image image image image

✅ Checklist

  • [✅] Code is self-reviewed
  • [✅] Follows project conventions
  • [✅] No unnecessary logs / debug code

@dhaval-p-iqud
feat: Implement pre-deployment port availability checks
5402e57 
- Added a new endpoint to check host port availability before deployment.
- Integrated port availability checks in the deployment workflow, ensuring ports are free before proceeding.
- Enhanced the SocketClientService to handle port check requests and responses.
- Updated the DeploymentsService to verify port availability on the target agent.
- Improved UI components to provide feedback during port checks and handle deployment failures gracefully.
- Added new column for revoked keys
@dhaval-p-iqud
feat: Enhance terminal viewer functionality and styling
0571dc9 
- Integrated horizontal scrolling support in the terminal viewer for better content management.
- Updated terminal fitting logic to synchronize content columns dynamically.
- Improved UI responsiveness with CSS adjustments for terminal components.
- Enhanced wheel trap functionality to prevent page scrolling during terminal interactions.
- Refactored terminal scroll down button for better tooltip accessibility.
- Added new utility functions for managing terminal content fitting.
@dhaval-p-iqud
fix: Simplify deployment failure messages for port conflicts
a168277 
- Removed redundant "Deployment failed:" prefix from port conflict messages.
- Streamlined messages to focus on the port issue for clarity.
@dhaval-p-iqud dhaval-p-iqud self-assigned this Jun 24, 2026
@dhaval-p-iqud dhaval-p-iqud added bug Something isn't working enhancement New feature or request labels Jun 24, 2026
@purvangIQUD purvangIQUD added this pull request to the merge queue Jun 24, 2026
Merged via the queue into main with commit 01408bf Jun 24, 2026
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@purvangIQUD purvangIQUD purvangIQUD approved these changes

Assignees

@dhaval-p-iqud dhaval-p-iqud

Labels

bug Something isn't working enhancement New feature or request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Add Port Availability Validation Before Service Deployment

2 participants

@dhaval-p-iqud @purvangIQUD