Opened 6 years ago

Closed 5 years ago

#599 closed Feature Wish (fixed)

Logging should include platform info (if possible)

Reported by: hildeb Owned by: stipa
Priority: minor Milestone: release 2.3.11
Component: Generic / unclassified Version: OpenVPN 2.3.8 (Community Ed)
Severity: Not set (select this one, unless your'e a OpenVPN developer) Keywords:
Cc: stipa

Description

Currently, openvpn is logging quite a bit upon startup:

Fri Aug 21 11:45:48 2015 OpenVPN 2.3.8 x86_64-w64-mingw32 [SSL (OpenSSL)] [LZO] [PKCS11]
[IPv6] built on Aug 4 2015
Fri Aug 21 11:45:48 2015 library versions: OpenSSL 1.0.1p 9 Jul 2015, LZO 2.08

But in a support situation we need to know which version (of windows) somebody is on (pre Vista? Win10?).
Also, it would be helpful to see if somebody is running a 32bit openvpn on a 64bit platform :(

Change History (8)

comment:1 Changed 6 years ago by Gert Döring

I agree that this would be a nice feature, and I'd happily review, test and merge such a patch - but as we're a bit short on developers that know windows right now, this might not happen any time soon...

(It should also set IV_PLAT= in ssl.c accordingly, so the server side knows)

comment:2 Changed 6 years ago by Gert Döring

Found interesting bits and pieces here...

http://www.codeproject.com/Articles/678606/Part-Overcoming-Windows-s-deprecation-of-GetVe?msg=5080848#xx5080848xx

might be worth exploring eventually.

comment:3 Changed 5 years ago by hildeb

2.3.10 is logging some OS info:
Sun Jan 17 13:42:13 2016 Windows version 6.0 (Windows Vista)

comment:4 Changed 5 years ago by Gert Döring

Cc: stipa added

Lev, any idea whether we can easily find out (and log) "is this a 32bit or 64bit windows installation"?

comment:6 Changed 5 years ago by stipa

Owner: set to stipa
Status: newassigned

comment:7 Changed 5 years ago by Gert Döring

Milestone: release 2.3.11

Patch has been merged to master, is waiting for a few more tests and then merge to 2.3

comment:8 Changed 5 years ago by Gert Döring

Resolution: fixed
Status: assignedclosed

commit 15f78acfae2f99b74a72b5766559f28c2d1d3cac (master)
commit 6a8e946174509ca50c63842f155ce5c90cfe43e0 (release 2.3)
Author: Lev Stipakov <lstipakov@…>
Date: Sun Feb 7 22:21:32 2016 +0200

Report Windows bitness

... I think this is perfectly fine for windows now. For the linuxes, the need does not seem to be so big (people usually use what their distribution provides and report this)... so, closing this. If it turns out we need more detailed OS reporting for other platforms, we can always open a new ticket.

Note: See TracTickets for help on using tickets.