Project

General

Profile

Actions
Copy link

Feature #5602

closed
KB JP

OpenVPN - certificate purpose validation required

Feature #5602: OpenVPN - certificate purpose validation required

Added by Kill Bill almost 11 years ago. Updated almost 11 years ago.

Status:
Resolved
Priority:
Very Low
Assignee:
Jim Pingle
Category:
OpenVPN
Target version:
2.3
Start date:
12/05/2015
Due date:
% Done:

100%

Estimated time:
Plus Target Version:
Release Notes:

Description

Way too many threads every week with users posting about "my OpenVPN does not work", boiling down to

error=unsupported certificate purpose

I can see cert_get_purpose() in certs.inc, this or some similar function should be used to validate the OpenVPN server/client configuration before letting users save it.

JP Updated by Jim Pingle almost 11 years ago Actions
Copy link
 #1

The problem with that is that it's not technically a requirement to use a server cert -- it's only required if the clients verify the server certificate type (which the export package does)

So we could be better about highlighting which certs are server certs (perhaps listing them first or styling them in some way) but we unfortunately we can't stop listing non-server certs there or it could break some setups.

We could also have the export package test the server cert type and omit the directive if they didn't use one, but I'd rather not encourage that practice too much.

JT Updated by Jim Thompson almost 11 years ago Actions
Copy link
 #2

  • Tracker changed from Bug to Feature
  • Assignee set to Jim Pingle

JP Updated by Jim Pingle almost 11 years ago Actions
Copy link
 #3

  • Assignee changed from Jim Pingle to Anonymous
  • Priority changed from Normal to Very Low
  • % Done changed from 0 to 80

I pushed a change to separate, identify, and prioritize the servers in the list of certificates on the page. Passing this off to Steve Beaver since he had an idea for printing a pretty warning if they pick the wrong kind.

I'll also look into adding a warning to the OpenVPN client export package in the meantime.

Lowering the priority since the formatting isn't a huge concern now that the rest of the work has been done.

Updated by Anonymous almost 11 years ago Actions
Copy link
 #4

  • Status changed from New to Feedback
  • Assignee changed from Anonymous to Jim Pingle

Dynamically update the help text based on the cert selected.

Updated by Anonymous almost 11 years ago Actions
Copy link
 #5

  • % Done changed from 80 to 100

Applied in changeset commit:a4ba0282e6cba7916e3955f2d44da9a902f80587.

JP Updated by Jim Pingle almost 11 years ago Actions
Copy link
 #6

  • Status changed from Feedback to Resolved

Looks great, should hopefully be obvious enough to make people realize they did something incorrectly. :-)

Actions
Copy link

Also available in: Atom