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.

Sign up for GitHub

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

Jump to bottom

add set proxy configuration #8

Merged
merged 62 commits into from
Oct 19, 2021
Merged

add set proxy configuration #8

merged 62 commits into from
Oct 19, 2021

Conversation

paule96
Copy link
Member

@paule96 paule96 commented Oct 14, 2021

closes #7

This brings many changes with it:

  • Linux and windows support in the build pipeline
  • Set-ProxyScript (works for linux and windows)

open things

[ ] docs missing

paule96 and others added 30 commits June 15, 2021 08:13
@paule96
start the initial implementation of a set proxy script.
5284c60 
added git support.
@paule96
add npm support
c8ff956
@paule96
fixes a wired bug with npm
64623e1
@paule96
add apt configuration
46dd92f
@paule96
make the apt and npm implementation for linux more robust
22057d1
@paule96
add docker config
9838be1 
- also make the git stuff more able to handle hostnames.
@paule96
add powershell configuration
eece8bc
@paule96
add comment
7cdf657
@paule96
add noroot switch
5e78f38
@paule96
add environment configuration
75d49d3
@paule96
unset proxy if there is no proxy address is defined
da4dabe
@paule96
add working unset of proxy config
252e9f5
@paule96
add a sample for unsetting the proxy
61c3680
@paule96
fix unsetting git proxy config
252b32f
@paule96
we can unset proxies now.
b2eb9c7 
So no need for developer saveties anymore :D
@paule96
fix file names
35c6192
@paule96
add some basic tests
83ffe5b
add dev container json
c3b175d
add dev container json
839bdc6
@paule96
Change devcontainer.json
bb79e2d
@paule96
improve linux environment support
827fda5
@paule96
improve samples
b966faa
@paule96
add tests for set-proxy function.
e5bb162 
- this brings test abilitiy for linux and windows
@paule96
add test pester extension to the workspace file
3934a19
@paule96
improve comment in set proxy
9f5b7ce
@paule96
add first basic tests for set git proxy
446e7ee
@paule96
fix a git version deteciton issue
9e44ce7
@paule96
add all missing tests for set-gitproxy
60dd9ec
@paule96
cleanup some test lines
818e569
@paule96
add tests for full coverage for npm
175e8b9
@paule96 paule96 added the enhancement New feature or request label Oct 14, 2021
@paule96 paule96 requested a review from kirkone October 14, 2021 21:35
@paule96
Copy link
Member Author

paule96 commented Oct 14, 2021

cc @DanielHabenicht

@DanielHabenicht
Copy link
Collaborator

Would be pretty fun to set up some End2End Tests for this Tool :D

@paule96
Copy link
Member Author

paule96 commented Oct 15, 2021

Would be pretty fun to set up some End2End Tests for this Tool :D

make an issue for that. And then we can discuss there an infrastructure for that. It will be really complex I think. Because you would need something like this:

  • a Machine A that would be the test machine
  • a Machine B that stands in the same network as Machine A and can directly accessed by it
  • a Machine C that is in a different network and can't be reached by Machine A
  • a Machine D that is in both networks and runs the proxy

Note: The machines can be containers or full vms. shouldn't matter
Note 2: Machine A should be Windows, Linux and MacOs at the same time

@DanielHabenicht
Copy link
Collaborator

I made an issue for it: #12

@paule96 paule96 merged commit c95edcc into main Oct 19, 2021
@paule96 paule96 deleted the paule96/add_set_proxy_configuration branch October 19, 2021 19:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@DanielHabenicht DanielHabenicht DanielHabenicht approved these changes

@kirkone kirkone Awaiting requested review from kirkone

Assignees

@kirkone kirkone

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add a set proxy script
3 participants
@paule96 @DanielHabenicht @kirkone