Commit Graph

2733 Commits

Author SHA1 Message Date
Johannes Sjøkvist
ae2fd78580 Add support for oid 2.5.29.17 RFC822Name
This is needed to be able to retreive email from the Subject
Alternative Name from the certificate.

Signed-off-by: Johannes Sjøkvist <johannes@konsept-it.no>
Signed-off-by: Nikos Mavrogiannopoulos <nmav@gnutls.org>
2017-01-28 15:04:20 +01:00
Nikos Mavrogiannopoulos
2f65c8c4e9 radius: removed error checking from rc_avpair_add()
When a dictionary misses an element, we would previously bail
out and not send any following value pairs. With that change
we ensure that we send as many value-pairs as are available
in the dictionary.

Signed-off-by: Nikos Mavrogiannopoulos <nmav@redhat.com>
2017-01-26 15:04:00 +01:00
Nikos Mavrogiannopoulos
3feec67070 worker: avoid sending an X-Reason header
HTTP/1.x allows sending any arbitrary reason we would like
after the error code. We now do that.

Signed-off-by: Nikos Mavrogiannopoulos <nmav@redhat.com>
2017-01-19 17:40:01 +01:00
Nikos Mavrogiannopoulos
43fb150de9 sec-mod: reduced level of error processing ... in worker commands
This was providing virtually no information since more specific
errors are typically printed prior to it.

Signed-off-by: Nikos Mavrogiannopoulos <nmav@redhat.com>
2017-01-19 17:36:04 +01:00
Nikos Mavrogiannopoulos
2e65196f1e doc update 2017-01-18 15:26:33 +01:00
Nikos Mavrogiannopoulos
2ceb0ffb47 radius-test: check whether the special IP values are handled
In particular we check whether 255.255.255.254 is correctly
and the expected IP is assigned to client.
2017-01-18 15:26:31 +01:00
Nikos Mavrogiannopoulos
d2f07e7c70 tests: use fedora 25 for docker tests
Signed-off-by: Nikos Mavrogiannopoulos <nmav@redhat.com>
2017-01-18 15:18:49 +01:00
Nikos Mavrogiannopoulos
3f87a93d38 radius: handle the special Framed-IP-Address values
That is if 0xFFFFFFFF or 0xFFFFFFFE are given ignore the value
and ensure they are allocated from our pool.

Signed-off-by: Nikos Mavrogiannopoulos <nmav@redhat.com>
2017-01-18 15:18:35 +01:00
Nikos Mavrogiannopoulos
3d940695d8 Added contribution guide and require DCO
Signed-off-by: Nikos Mavrogiannopoulos <nmav@redhat.com>
2017-01-10 16:52:18 +01:00
Nikos Mavrogiannopoulos
dc2994fd40 doc update 2017-01-10 15:06:25 +01:00
John Thiltges
7e3c628785 ocserv-fw should send all traffic to the device-specific forwarding chain
After adding port-specific rules to FORWARD and creating SEC_FORWARD_CHAIN
with route-specific rules, send any remaining FORWARD traffic to
SEC_FORWARD_CHAIN.
2017-01-09 15:45:22 -06:00
John Thiltges
fa65740a4c ocserv-fw should still create a chain if restrict-user-to-routes is set
ocserv-fw only creates SEC_FORWARD_CHAIN if ports are being blocked. This
leads to an error if restrict-user-to-routes is used without any port
blocking.

Since ocserv-fw is only called if restrict-user-to-routes or -ports is set,
remove the conditional check for creating the chain.
2017-01-09 12:28:19 -06:00
Nikos Mavrogiannopoulos
83f600afda worker: increase the waiting time of cookie auth message
This allows the connect-script to run for more time than
the default socket timeout, and be limited by the configured
authentication timeout ("auth-timeout").
2017-01-05 16:08:04 +00:00
Nikos Mavrogiannopoulos
176ba796c5 doc update 2017-01-04 16:24:57 +01:00
Nikos Mavrogiannopoulos
0b47b305de improved documentation of user-profile option 2017-01-04 16:20:57 +01:00
Nikos Mavrogiannopoulos
3c8cdaedb1 Send "config client" XML field after successful auth
This allows to advertise the XML configuration file for the
client to download, in recent openconnect clients. In addition
made support for the client XML file unconditional (no longer
depending on the anyconnect client compatibility flag).
2017-01-04 16:20:29 +01:00
Nikos Mavrogiannopoulos
62bce8ddcf improved message on file limit update
Relates #85
2016-12-23 09:22:53 +01:00
Nikos Mavrogiannopoulos
442ceda8c0 when setting limits keep untouched the maximum number
Relates #85
2016-12-23 09:21:54 +01:00
Nikos Mavrogiannopoulos
1f8304fc04 Improve output of --version 2016-12-23 09:14:59 +01:00
Nikos Mavrogiannopoulos
18f003805b optimized loop in check_multiple_users
That is return early, once exceeding entries have been
detected.
2016-12-23 09:14:31 +01:00
Mike Miller
e69a07ed6e tests: allow running tests with alternate OpenConnect
Allow $OPENCONNECT in the caller's environment to override the default
openconnect system installation.

Signed-off-by: Mike Miller <mtmiller@debian.org>
2016-12-23 08:04:59 +01:00
Nikos Mavrogiannopoulos
3af590bc6c Merge branch 'preserve-user-libs' into 'master'
configure: preserve LIBS from calling environment

See merge request !27
2016-12-23 07:04:02 +00:00
Mike Miller
888cbeabc7 configure: preserve LIBS from calling environment
Use the existing oldlibs temporary variable that was used but never
populated.

Signed-off-by: Mike Miller <mtmiller@debian.org>
2016-12-22 18:48:23 -08:00
Nikos Mavrogiannopoulos
b6a2e26f1a Merge branch 'no-more-no-cert-check' into 'master'
fix test-sighup-key-change for current OpenConnect

OpenConnect 7.08 removed the option `--no-cert-check`. Pass the actual id of the newly generated server key. Fall back to the key fingerprint on CentOS 6 with an older version of GnuTLS certtool.

Resolves #81

See merge request !22
2016-12-18 12:05:32 +00:00
Mike Miller
dc222463e0 tests: fix test-sighup-key-change for current OpenConnect
OpenConnect 7.08 removed the option `--no-cert-check`. Pass the actual
id of the newly generated server key. On systems with older versions of
GnuTLS, pass the server key fingerprint instead.

Resolves #81

Signed-off-by: Mike Miller <mtmiller@debian.org>
2016-12-17 12:52:15 -08:00
Mike Miller
273024c7ce .gitlab-ci.yml: add gnutls-bin to Debian build environment
Signed-off-by: Mike Miller <mtmiller@debian.org>
2016-12-17 12:52:15 -08:00
Nikos Mavrogiannopoulos
f0c11be578 Merge branch 'tmp-host-update-test-fix' into 'master'
tests: make test-pass-script pass with new openconnect

The new versions (7.07+) do not automatically send a bogus hostname,
they require the --local-hostname parameter to be passed.

Resolves #80

See merge request !25
2016-12-16 08:53:30 +00:00
Nikos Mavrogiannopoulos
b18cd67917 tests: make test-pass-script pass with new openconnect
The new versions (7.07+) do not automatically send a bogus hostname,
they require the --local-hostname parameter to be passed.

Resolves #80
2016-12-16 09:37:44 +01:00
Mike Miller
d3370cccd0 tests: allow test-gssapi to run on Debian
Check for alternate filename of ntlmssp.conf on Debian-based systems.

Resolves #82

Signed-off-by: Mike Miller <mtmiller@debian.org>
2016-12-16 09:24:43 +01:00
Mike Miller
9f0776e6ae tests: ensure unique temporary config file names
Some unit tests share the same ocserv config file. Ensure that the file
written and used by each test script has a unique name.

Resolves #83

Signed-off-by: Mike Miller <mtmiller@debian.org>
2016-12-15 16:44:14 -08:00
Nikos Mavrogiannopoulos
cd39da8ae1 tests: coverted to bash tests which had bashishms 2016-12-15 21:23:17 +01:00
Nikos Mavrogiannopoulos
2da126a585 .gitlab-ci.yml: added Debian:stretch build 2016-12-14 09:04:04 +01:00
Mike Miller
e3ccd247a0 configure: discover pam_wrapper directory from pkg-config
Signed-off-by: Mike Miller <mtmiller@debian.org>
2016-12-14 09:04:04 +01:00
Nikos Mavrogiannopoulos
eed6ff6902 Makefile.am: corrected typo 2016-12-14 09:03:30 +01:00
Nikos Mavrogiannopoulos
98d0ae62f6 README.md: corrected name of pam-oath in debian
Also added packages where were not previously mentioned in
Debian.
2016-12-14 09:03:14 +01:00
Mike Miller
34fd11d3a8 Fix typos in man pages, config files, and comments
Signed-off-by: Mike Miller <mtmiller@debian.org>
2016-12-13 18:01:23 -08:00
Nikos Mavrogiannopoulos
2c64c99baa Reverted recursive make in src/
This simplifies the make process and allows for greater parallelism
during build.
2016-12-13 15:37:07 +01:00
Nikos Mavrogiannopoulos
1b7e00ed02 Use unique names for autogen files
This addresses issue in parallel builds caused by autogen
using the same temporary files for both occtl and ocpasswd
files.

Resolves #76
2016-12-13 09:39:51 +01:00
Nikos Mavrogiannopoulos
7fea131d00 use stamp files to allow parallel build of autogen files
Autogen seems to output on the creates files gradually, something that
makes 'make' believe that the command is complete prior to the output
file being fully populated. The current approach uses stamp files to
ensure that no incomplete files are used for compilation.
2016-12-10 14:30:15 +01:00
Nikos Mavrogiannopoulos
729b04b00b doc update 2016-11-17 08:43:41 +01:00
Nikos Mavrogiannopoulos
eb9c86218e .gitlab-ci.yml: update autogenerated files prior to make dist 2016-11-17 08:33:41 +01:00
Nikos Mavrogiannopoulos
63e924b621 .gitlab-ci.yml: cleanups 2016-11-17 08:28:15 +01:00
Nikos Mavrogiannopoulos
e42c5ed7e7 autogen: store and use auto-generated autogen .in files in git builds
This simplifies the generation of the files (delegated to configure),
and allows systems without autogen to be used to compile git sources.
2016-11-17 08:28:11 +01:00
Nikos Mavrogiannopoulos
1f940f04c5 Revert "autogen: store and use auto-generated autogen files in git builds"
This reverts commit d0908f2c52.
2016-11-16 22:24:23 +01:00
Nikos Mavrogiannopoulos
b5c39e2edf sample.config: include switch-to-tcp-timeout directive 2016-11-15 14:57:48 +01:00
Nikos Mavrogiannopoulos
fce7610aa5 released 0.11.6 ocserv_0_11_6 2016-11-14 19:17:16 +01:00
Nikos Mavrogiannopoulos
943e2fb597 bumped version 2016-11-14 19:16:50 +01:00
Nikos Mavrogiannopoulos
7606842d99 occtl.8: Added examples of usage 2016-11-14 14:15:50 +01:00
Nikos Mavrogiannopoulos
e7b1636de4 occtl.8: added more info on JSON output 2016-11-14 14:10:49 +01:00
Nikos Mavrogiannopoulos
7b0e84a36e doc update 2016-11-11 18:17:28 +01:00