Skip to content
This repository has been archived by the owner on Jun 27, 2022. It is now read-only.

Fix compression param in the spec file #20

Merged
merged 1 commit into from
Jun 7, 2019
Merged

Conversation

daviddob
Copy link
Contributor

@daviddob daviddob commented Jun 5, 2019

When attempting to specify lz4-2 OpenVPN would fail to start with unknown compression options, changing to lz4-v2 resolves this.

When attempting to specify `lz4-2` OpenVPN would fail to start with unknown compression options, changing to `lz4-v2` resolves this.
@dpb587 dpb587 merged commit aefda0e into dpb587:master Jun 7, 2019
@dpb587
Copy link
Owner

dpb587 commented Jun 7, 2019

Good catch - thanks!

@dpb587
Copy link
Owner

dpb587 commented Jun 7, 2019

Out of curiosity, did you run into any bugs or concerns with the default auto behavior?

@daviddob
Copy link
Contributor Author

daviddob commented Jun 7, 2019

I didn't have any issues with it running in my previous configuration, however for security I usually disable it altogether (re: Voracle) and may suggest it become the default as such. Recently I had a use case where speed and throughput were more important in a low risk environment and after a few benchmarks I found that UDP + lz4-v2 + a few extra configs were best for the server's CPU and overall performance.

@dpb587
Copy link
Owner

dpb587 commented Jun 8, 2019

I think it makes sense to change the default as well. I'll queue it for more thought and potentially the next major version change - I have some other planned changes queued up for that as well. If you have more thoughts, I made an issue to track it.

Agreed that UDP is going to provide noticeably better throughput in stable network environments.

If you have any authoritative OpenVPN configuration references or recommendations on other default configuration changes, I'm happy to chat more.

Thanks!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants