Opened 9 years ago
Closed 8 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 9 years ago by
comment:2 Changed 9 years ago by
Found interesting bits and pieces here...
might be worth exploring eventually.
comment:3 Changed 8 years ago by
2.3.10 is logging some OS info:
Sun Jan 17 13:42:13 2016 Windows version 6.0 (Windows Vista)
comment:4 Changed 8 years ago by
| Cc: | stipa added |
|---|
Lev, any idea whether we can easily find out (and log) "is this a 32bit or 64bit windows installation"?
comment:5 Changed 8 years ago by
comment:6 Changed 8 years ago by
| Owner: | set to stipa |
|---|---|
| Status: | new → assigned |
comment:7 Changed 8 years ago by
| 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 8 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
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.
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)