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

The state should not be abbreviated #64

Merged
merged 1 commit into from
Aug 31, 2015
Merged

Conversation

coolaj86
Copy link

The command line tool instructs to not use an abbreviation, so we should expand CA to California.

The command line tool instructs to not use an abbreviation, so we should expand CA to California.
@ecrist
Copy link
Member

ecrist commented Jun 13, 2015

No. This is a standard. Not something we defined.

On Jun 12, 2015, at 6:50 PM, AJ ONeal notifications@github.com wrote:

The command line tool instructs to not use an abbreviation, so we should expand CA to California.

You can view, comment on, or merge this pull request online at:

#64

Commit Summary

The state should not be abbreviated
File Changes

M easy-rsa/2.0/vars (2)
Patch Links:

https://github.com/OpenVPN/easy-rsa/pull/64.patch
https://github.com/OpenVPN/easy-rsa/pull/64.diff

Reply to this email directly or view it on GitHub.

@ecrist ecrist closed this Jun 13, 2015
@coolaj86
Copy link
Author

Screenshot:

Imgur

Why does the tool say the standard is one thing, but you're saying the standard is something else?

@ecrist
Copy link
Member

ecrist commented Aug 31, 2015

The country should be abbreviated, but the state/province should not. I couldn't find any restriction or recommendation in any IETF RFC, but google, wikipedia, and random joe websites seem to think this is a good idea. WCGW?

@ecrist ecrist reopened this Aug 31, 2015
ecrist added a commit that referenced this pull request Aug 31, 2015
coals86: The state should not be abbreviated (according to "they" on the internet, probably really a good idea --ecrist)
@ecrist ecrist merged commit 987f3cf into OpenVPN:release/2.x Aug 31, 2015
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