Skip to content
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

Add connect method to set destination server. #56

Merged
merged 26 commits into from
Jan 29, 2024

Conversation

amirhosseinmirmohammad
Copy link
Contributor

@amirhosseinmirmohammad amirhosseinmirmohammad commented Dec 16, 2023

submit final version of connection method in cmdlet 2023-12-16

fixes #34

catch {
Write-Error "Nvm is not installed, install it manually to continue."
}
# try {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

در Pull Request کد کامنت شده نداشته باشیم

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is not resolve

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why remove nvm installation?

Copy link
Contributor

@siadatism siadatism left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

به نظر بنده آماده است.

@siadatism siadatism changed the title change uri with serverAddress substring Add connect method to set destination server. Dec 30, 2023
@soroshsabz soroshsabz self-requested a review December 30, 2023 17:18
Description = "Target Server Address" }
}
}
Url = "http://resaa.net/",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

resaa.net?!

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Passive aggressive comment :D

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please resolve Url value

string requestUriString = request.RequestUri.ToString();
Uri newUri = new Uri(requestUriString);
string host = newUri.Host;
string finalUrl = requestUriString.Replace(host,serverAddress);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

add space after ,

Description = "Target Server Address" }
}
}
Url = "http://resaa.net/",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please resolve Url value

Copy link
Contributor

@siadatism siadatism left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I checked changes, they are OK.

@soroshsabz soroshsabz merged commit 06ebb8d into main Jan 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add connect cmdlet
3 participants