Skip to content

Conversation

@RasPutinnn
Copy link

This documentation describes how to configure AnyConnect-Server on VyOS

Copy link
Member

@dmbaturin dmbaturin left a comment

Choose a reason for hiding this comment

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

Cisco consistently spells it "AnyConnect" in all their publications. I believe we should do the same for the sake of consistency.

Other than this minor issue, this is sure a welcome addition, I'm happy to merge once spelling consistency is fixed.

@andamasov
Copy link
Member

How about use AnyConnect-compatible server
We need to add a disclaimer about trademarks @dmbaturin

@dmbaturin
Copy link
Member

@syncer I'm only talking about capitalization: "AnyConnect-compatible" over "anyconnect-compatible" etc.

Regarding trademarks, I believe we indeed should add a disclaimer somewhere, as a CYA measure.

@RasPutinnn
Copy link
Author

RasPutinnn commented Aug 31, 2020

I performed the requested change. Please let me know if the changes are not reflected on github.

This documentation describes how to configure AnyConnect-Server on VyOS
@kroy-the-rabbit
Copy link
Contributor

I'm of the opinion here that we shouldn't say "AnyConnect" at all, except in the documentation of to say "AnyConnect compatible". Similar to what the original https://www.infradead.org/openconnect/ documentation mentions.

Unfortunately, I think the whole documentation for it needs to be refactored AND the underlying code. I think even

set vpn anyconnect ...

Is opening ourselves up to potential future issues.

Moreso since as the above link mentions, OpenConnect works with more than just AnyConnect. And 2), could we even get away with using the name:

set vpn openconnect

Wouldn't ssl-vpn or something be better?

This documentation describes how to configure OpenConnect-Server on VyOS
@RasPutinnn
Copy link
Author

I performed the requested change. Please let me know if the changes are not reflected on github.

@c-po c-po merged commit c729b98 into vyos:master Sep 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

5 participants