Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

ADD be explicit about the underlying limitation - OpenSwan#1426

Merged
asfgit merged 1 commit intoapache:masterapache/cloudstack:masterfrom
ntavares:ipsec-psk-rulesntavares/cloudstack:ipsec-psk-rulesCopy head branch name to clipboard
Mar 10, 2016
Merged

ADD be explicit about the underlying limitation - OpenSwan#1426
asfgit merged 1 commit intoapache:masterapache/cloudstack:masterfrom
ntavares:ipsec-psk-rulesntavares/cloudstack:ipsec-psk-rulesCopy head branch name to clipboard

Conversation

@ntavares
Copy link

@ntavares ntavares commented Mar 2, 2016

According to:
http://linux.die.net/man/5/ipsec.secrets

A preshared secret is most conveniently represented as a sequence of characters, delimited by the double-quote character ("). The sequence cannot contain a newline or double-quote.

@GabrielBrascher
Copy link
Member

It seems a fair update.
LGTM due to the lack of code.

@DaanHoogland
Copy link
Contributor

LGTM

1 similar comment
@kiwiflyer
Copy link
Contributor

LGTM

@rafaelweingartner
Copy link
Member

merging giving that it is a change only in a description of an enum and that we already have 2+ LGTM

@asfgit asfgit merged commit 783aac8 into apache:master Mar 10, 2016
asfgit pushed a commit that referenced this pull request Mar 10, 2016
ADD be explicit about the underlying limitation - OpenSwanAccording to:
http://linux.die.net/man/5/ipsec.secrets

A preshared secret is most conveniently represented as a sequence of characters, delimited by the double-quote character ("). The sequence cannot contain a newline or double-quote.

* pr/1426:
  ADD be explicit about the underlying limitation - OpenSwan

Signed-off-by: Rafael Weingärtner <rafael@apache.org>
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.

6 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.