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

Major rework. #8

Closed
wants to merge 24 commits into from
Closed

Major rework. #8

wants to merge 24 commits into from

Conversation

ypid
Copy link

@ypid ypid commented Aug 16, 2015

Hi

I reworked the role. Checkout the commit messages for details.

Additional things:

ypid added 23 commits August 15, 2015 15:20
* Use consistent coding style and YAML syntax.
  https://www.reinteractive.net/posts/167-ansible-real-life-good-practices
* Don’t configure pinning for other packages. People might manage them by other means. For example: https://github.com/debops/ansible-apt_preferences
* Configure apt preferences to ensure that packages are installed from deb.torproject.org. Related: https://github.com/patrickod/docker-tor/blob/master/apt-pinning
* How did this even work with sudo: no???
* 's/proc_instances/tor_instances/g'
Options renamed:
ORBindAddress -> ORListenAddress
SocksBindAddress -> SocksListenAddress
AllowUnverifiedNodes -> AllowInvalidNodes
DirBindAddress -> DirListenAddress

Removed, not found in manual or with `tor --list-torrc-options`:
* DirServer
* HiddenServiceNodes
(All other torrc Ansible variables start with upper case)
ypid added a commit to ypid/docker-makefile that referenced this pull request Aug 16, 2015
@david415
Copy link
Owner

why not work on this https://github.com/nusenu/ansible-relayor/
this project is not being maintained... and nusenu's work is based on this and far extended.

@david415 david415 closed this Aug 23, 2015
@ypid
Copy link
Author

ypid commented Aug 23, 2015

WTF? Thanks for adding the note to the project now …

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.

None yet

2 participants