Skip to content

Commit c2e7f08

Browse files
committed
Add default timeout and max_authn_per_session settings
Relaetd to OpenConext/OpenConext-engineblock#1777 and OpenConext/OpenConext-engineblock#1345
1 parent 0371174 commit c2e7f08

File tree

2 files changed

+11
-0
lines changed

2 files changed

+11
-0
lines changed

roles/engineblock/defaults/main.yml

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -67,6 +67,12 @@ engine_minimum_execution_time_on_invalid_received_response: 5000
6767
engine_time_frame_for_authentication_loop_in_seconds: 60
6868
engine_maximum_authentication_procedures_allowed: 5
6969

70+
# maximum number of outstandig AuthN requests per session; exceeding this results in a 429
71+
engine_max_authn_per_session: 30
72+
73+
# timeout when doing external queries (e.g., to PDP, AA, SBS)
74+
engine_http_client_timeout: 10
75+
7076
# This PCRE regex is used to blacklist incoming AuthnContextClassRef attributes on. If an empty string is used
7177
# the validation is skipped. The validator will throw an exception if the used regex is invalid.
7278
engine_stepup_authn_context_class_ref_blacklist_regex: '/http:\/\/{{ base_domain | regex_escape }}\/assurance\/loa[1-3]/'

roles/engineblock/templates/parameters.yml.j2

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -147,6 +147,11 @@ parameters:
147147
## The value for guest qualifier. Can be overridden for specific environments
148148
addgueststatus_guestqualifier: '{{ guest_qualifier | default('') }}'
149149

150+
## the timeout used when querying external sources (PDP, AA, etc)
151+
http_client.timeout: "{{ engine_http_client_timeout | int }}"
152+
## maximum number of simultaneous open authentications per session (exceed this, and receive a 429)
153+
maximum_authentications_per_session: "{{ engine_max_authn_per_session | int }}"
154+
150155
## Language cookie settings
151156
cookie.path: {{ cookie_path | default('/') }}
152157
cookie.secure: true

0 commit comments

Comments
 (0)